Skip to content

Commit

Permalink
fix: remove e2e test account credentials
Browse files Browse the repository at this point in the history
  • Loading branch information
David Lopez committed Jan 5, 2023
1 parent f2311c3 commit f7f2bca
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 9 deletions.
19 changes: 13 additions & 6 deletions .github/workflows/canaries.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,16 +31,19 @@ jobs:
run: |
npm i aws-amplify @aws-amplify/ui-react
npm i --save-dev cypress
- name: Configure AWS credentials
uses: aws-actions/configure-aws-credentials@v1
with:
role-to-assume: ${{ secrets.E2E_TEST_ROLE_ARN }}
aws-region: us-west-2
- name: Run CLI Pull in test app
working-directory: e2e-test-app
run: |
FORCE_RENDER=1 amplify pull --appId ${{ secrets.E2E_TEST_APP_ID }} --envName staging -y --providers "{\
\"awscloudformation\":{\
\"configLevel\":\"project\",\
\"useProfile\":false,\
\"useProfile\":true,\
\"profileName\":\"default\",\
\"accessKeyId\":\"${{ secrets.E2E_TEST_ACCESS_KEY }}\",\
\"secretAccessKey\":\"${{ secrets.E2E_TEST_SECRET_KEY }}\",\
\"region\":\"us-west-2\"\
}\
}"
Expand Down Expand Up @@ -103,16 +106,19 @@ jobs:
run: |
npm i aws-amplify @aws-amplify/ui-react
npm i --save-dev cypress
- name: Configure AWS credentials
uses: aws-actions/configure-aws-credentials@v1
with:
role-to-assume: ${{ secrets.E2E_TEST_ROLE_ARN }}
aws-region: us-west-2
- name: Run CLI Pull in test app
working-directory: e2e-test-app
run: |
FORCE_RENDER=1 amplify pull --appId ${{ secrets.E2E_TEST_APP_ID }} --envName staging -y --providers "{\
\"awscloudformation\":{\
\"configLevel\":\"project\",\
\"useProfile\":false,\
\"useProfile\":true,\
\"profileName\":\"default\",\
\"accessKeyId\":\"${{ secrets.E2E_TEST_ACCESS_KEY }}\",\
\"secretAccessKey\":\"${{ secrets.E2E_TEST_SECRET_KEY }}\",\
\"region\":\"us-west-2\"\
}\
}"
Expand All @@ -131,6 +137,7 @@ jobs:
env:
REACT_APP_USER_EMAIL: ${{ secrets.E2E_TEST_USER_EMAIL }}
REACT_APP_USER_PASSWORD: ${{ secrets.E2E_TEST_USER_PASSWORD }}

write-release-canary-failure-metric:
runs-on: ubuntu-latest
needs: release-canary
Expand Down
14 changes: 11 additions & 3 deletions .github/workflows/check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,11 @@ on:
pull_request:
branches: [main, develop, feature/*, tagged-release/*]

# These permissions are needed to interact with GitHub's OIDC Token endpoint.
permissions:
id-token: write # This is required for requesting the JWT
contents: read # This is required for actions/checkout

jobs:
amplify-cli-tests:
runs-on: ubuntu-latest
Expand Down Expand Up @@ -52,16 +57,19 @@ jobs:
run: |
npm i aws-amplify @aws-amplify/ui-react
npm i --save-dev cypress
- name: Configure AWS credentials
uses: aws-actions/configure-aws-credentials@v1
with:
role-to-assume: ${{ secrets.E2E_TEST_ROLE_ARN }}
aws-region: us-west-2
- name: Run CLI Pull in test app
working-directory: e2e-test-app
run: |
FORCE_RENDER=1 amplify-dev pull --appId ${{ secrets.E2E_TEST_APP_ID }} --envName staging -y --providers "{\
\"awscloudformation\":{\
\"configLevel\":\"project\",\
\"useProfile\":false,\
\"useProfile\":true,\
\"profileName\":\"default\",\
\"accessKeyId\":\"${{ secrets.E2E_TEST_ACCESS_KEY }}\",\
\"secretAccessKey\":\"${{ secrets.E2E_TEST_SECRET_KEY }}\",\
\"region\":\"us-west-2\"\
}\
}"
Expand Down

0 comments on commit f7f2bca

Please sign in to comment.