From db2ea77634bee2b6a00e20d462a9b46b9fadffcb Mon Sep 17 00:00:00 2001
From: Natalie Schultz <90212258+nataliejschultz@users.noreply.github.com>
Date: Tue, 19 Sep 2023 20:32:46 -0600
Subject: [PATCH] Only setting env var once

Trying out setting the environment variable just once, in the docker compose line. I have high hopes for this test :)
---
 .github/workflows/nominatim-docker-test.yml | 16 +++++-----------
 1 file changed, 5 insertions(+), 11 deletions(-)

diff --git a/.github/workflows/nominatim-docker-test.yml b/.github/workflows/nominatim-docker-test.yml
index ec83ebabd..edbf9ee45 100644
--- a/.github/workflows/nominatim-docker-test.yml
+++ b/.github/workflows/nominatim-docker-test.yml
@@ -19,10 +19,10 @@ jobs:
     # The type of runner that the job will run on
     runs-on: ubuntu-latest
     env:
-      GEOFABRIK_QUERY_URL: "https://geocoding.geofabrik.de/${{ secrets.GEOFABRIK_API }}"
-      TEST_KEY: ${{ secrets.TEST_SECRET }}
-      TEST_ENVVAR: "https://geocoding.geofabrik.de/${{ secrets.TEST_SECRET }}"
-      TEST_STR: "https://geocoding.geofabrik.de/"
+      # GEOFABRIK_QUERY_URL: "https://geocoding.geofabrik.de/${{ secrets.GEOFABRIK_API }}"
+      # TEST_KEY: ${{ secrets.TEST_SECRET }}
+      # TEST_ENVVAR: "https://geocoding.geofabrik.de/${{ secrets.TEST_SECRET }}"
+      # TEST_STR: "https://geocoding.geofabrik.de/"
 
     # Steps represent a sequence of tasks that will be executed as part of the job
     steps:
@@ -34,13 +34,7 @@ jobs:
     - name: Workflow test
       run: echo Smoke test
 
-    - name: Variable
-      run: echo "TEST_KEY=$TEST_KEY" > emission/integrationTests/.env
-
-    - name: Variable2
-      run: cat emission/integrationTests/.env
-
     - name: Test nominatim
-      run: docker-compose -f emission/integrationTests/docker-compose.yml up --exit-code-from web-server
+      run: TEST_KEY=${{ secrets.TEST_SECRET }} docker-compose -f emission/integrationTests/docker-compose.yml up --exit-code-from web-server