Skip to content

Commit

Permalink
Merge pull request #726 from DFE-Digital/Auto-Fix-security-tests-Null…
Browse files Browse the repository at this point in the history
…-Env-Vars

Fixed npm run cy:run cmd in security-tests.yml to match working tests…
  • Loading branch information
dangood84 authored Nov 20, 2024
2 parents 9c730ef + 729046b commit 9291b41
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .github/workflows/security-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,11 +65,12 @@ jobs:
LOGIN_USERNAME: ${{ secrets.LOGIN_USERNAME }}
LOGIN_PASSWORD: ${{ secrets.LOGIN_PASSWORD }}
URL: ${{ secrets.URL }}
SIGNIN_URL: ${{ secrets.SIGNIN_URL }}
ZAP: true
ZAP_API_KEY: ${{ secrets.ZAP_API_KEY }}
ZAP_ADDRESS: ${{ env.ZAP_ADDRESS }}
ZAP_PORT: ${{ env.ZAP_PORT }}
run: npm run cy:run
run: npm run cy:run -- --env URL='${{ secrets.URL }}',LOGIN_USERNAME=${{ secrets.LOGIN_USERNAME }},LOGIN_PASSWORD=${{ secrets.LOGIN_PASSWORD }},SIGNIN_URL=${{ secrets.SIGNIN_URL }}

- name: Get git sha
if: '!cancelled()'
Expand Down

0 comments on commit 9291b41

Please sign in to comment.