Merge pull request #1388 from elizaOS/fix/integration_tests

chore: require approval for integration test step
elizaOS · Dec 23, 2024 · f128f3b · f128f3b
2 parents 8f34bc4 + 8e7ed8a
commit f128f3b
Showing 1 changed file with 12 additions and 5 deletions.
diff --git a/.github/workflows/integrationTests.yaml b/.github/workflows/integrationTests.yaml
@@ -6,6 +6,7 @@ on:
     pull_request_target:
         branches:
             - "*"
+
 jobs:
     smoke-tests:
         runs-on: ubuntu-latest
@@ -23,8 +24,19 @@ jobs:
 
             - name: Run smoke tests
               run: pnpm run smokeTests
+
+    approval:
+        runs-on: ubuntu-latest
+        needs: smoke-tests
+        steps:
+            - name: Wait for manual approval
+              run: |
+                  echo "Please approve the workflow manually to proceed."
+                  echo "::pause::" # This creates a manual approval checkpoint.
+
     integration-tests:
         runs-on: ubuntu-latest
+        needs: approval
         env:
             OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
         steps:
@@ -46,16 +58,11 @@ jobs:
               run: pnpm build
 
             - name: Check for API key
-              id: check_api_key
               run: |
                   if [ -z "$OPENAI_API_KEY" ]; then
                       echo "Error: OPENAI_API_KEY is not set."
                       exit 1
                   fi
-              env:
-                  OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
 
             - name: Run integration tests
               run: pnpm run integrationTests
-              env:
-                  OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}