Skip to content

Commit

Permalink
fix: inject variable/secret TEST_SNYK_TOKEN into workflow step
Browse files Browse the repository at this point in the history
  • Loading branch information
bastiandoetsch committed Oct 9, 2023
1 parent 2dab7ba commit 146c974
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/smoke-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ jobs:
- name: Run npm test with Root user
if: ${{ matrix.snyk_install_method == 'npm-root-user' }}
env:
SMOKE_TESTS_SNYK_TOKEN: ${{ secrets.SMOKE_TESTS_SNYK_TOKEN }}
TEST_SNYK_TOKEN: ${{ secrets.TEST_SNYK_TOKEN }}
run: |
docker build -t snyk-docker-root -f ./test/smoke/docker-root/Dockerfile ./test
docker run -eCI=1 -eSMOKE_TESTS_SNYK_TOKEN snyk-docker-root
Expand Down
4 changes: 2 additions & 2 deletions test/smoke/spec/snyk_auth_spec.sh
Original file line number Diff line number Diff line change
Expand Up @@ -47,11 +47,11 @@ Describe "Snyk CLI Authorization"
End

It "updates config file if given legit token"
When run snyk auth "${SMOKE_TESTS_SNYK_TOKEN}"
When run snyk auth "${TEST_SNYK_TOKEN}"
The output should include "Your account has been authenticated. Snyk is now ready to be used."
The status should be success
# TODO: unusable with our current docker issues
The stderr should equal ""
The result of "print_snyk_config()" should include "api: ${SMOKE_TESTS_SNYK_TOKEN}"
The result of "print_snyk_config()" should include "api: ${TEST_SNYK_TOKEN}"
End
End

0 comments on commit 146c974

Please sign in to comment.