Skip to content

Commit

Permalink
ci: split some migration tests, fix retry issues, remove per-job clea…
Browse files Browse the repository at this point in the history
…nup (#8340)
  • Loading branch information
johnpc authored Oct 5, 2021
1 parent 3e1ce0d commit c936570
Show file tree
Hide file tree
Showing 6 changed files with 2,315 additions and 2,229 deletions.
38 changes: 19 additions & 19 deletions .circleci/config.base.yml
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,6 @@ jobs:
name: Run GraphQL end-to-end tests
command: |
source .circleci/local_publish_helpers.sh
setAwsAccountCredentials
cd packages/graphql-transformers-e2e-tests/
retry yarn e2e --maxWorkers=3 $TEST_SUITE
environment:
Expand Down Expand Up @@ -288,8 +287,9 @@ jobs:
path: packages/amplify-e2e-tests/
- store_artifacts:
path: packages/amplify-e2e-tests/amplify-e2e-reports
- clean_e2e_resources:
os: << parameters.os >>
# - clean_e2e_resources:
# os: << parameters.os >>

amplify_migration_tests_v4:
<<: *defaults
steps:
Expand All @@ -305,7 +305,7 @@ jobs:
source .circleci/local_publish_helpers.sh
changeNpmGlobalPath
cd packages/amplify-migration-tests
yarn run migration_v4.0.0 --maxWorkers=3 $TEST_SUITE
retry yarn run migration_v4.0.0 --maxWorkers=3 $TEST_SUITE
no_output_timeout: 90m
- run: *scan_e2e_test_artifacts
- store_test_results:
Expand Down Expand Up @@ -412,7 +412,7 @@ jobs:
source .circleci/local_publish_helpers.sh
changeNpmGlobalPath
cd packages/amplify-migration-tests
yarn run migration --maxWorkers=3 $TEST_SUITE
retry yarn run migration --maxWorkers=3 $TEST_SUITE
no_output_timeout: 90m
- run: *scan_e2e_test_artifacts
- store_test_results:
Expand Down Expand Up @@ -767,8 +767,8 @@ workflows:
- cleanup-resources
- e2e-auth-credentials
- e2e-test-context
post-steps:
- run: *clean_e2e_resources
# post-steps:
# - run: *clean_e2e_resources
filters:
branches:
only:
Expand Down Expand Up @@ -805,8 +805,8 @@ workflows:
- /run-e2e\/.*/
requires:
- build
post-steps:
- run: *clean_e2e_resources
# post-steps:
# - run: *clean_e2e_resources
- amplify_migration_tests_v4:
context:
- amplify-ecr-image-pull
Expand All @@ -821,8 +821,8 @@ workflows:
- /run-e2e\/.*/
requires:
- build
post-steps:
- run: *clean_e2e_resources
# post-steps:
# - run: *clean_e2e_resources
- amplify_migration_tests_v4_30_0:
context:
- amplify-ecr-image-pull
Expand All @@ -837,8 +837,8 @@ workflows:
- /run-e2e\/.*/
requires:
- build
post-steps:
- run: *clean_e2e_resources
# post-steps:
# - run: *clean_e2e_resources
- amplify_migration_tests_non_multi_env_layers:
context:
- amplify-ecr-image-pull
Expand All @@ -853,8 +853,8 @@ workflows:
- /run-e2e\/.*/
requires:
- build
post-steps:
- run: *clean_e2e_resources
# post-steps:
# - run: *clean_e2e_resources
- amplify_migration_tests_multi_env_layers:
context:
- amplify-ecr-image-pull
Expand All @@ -869,17 +869,17 @@ workflows:
- /run-e2e\/.*/
requires:
- build
post-steps:
- run: *clean_e2e_resources
# post-steps:
# - run: *clean_e2e_resources
- amplify_console_integration_tests:
context:
- amplify-ecr-image-pull
- e2e-auth-credentials
- cleanup-resources
- console-e2e-test
- e2e-test-context
post-steps:
- run: *clean_e2e_resources
# post-steps:
# - run: *clean_e2e_resources
filters:
branches:
only:
Expand Down
Loading

0 comments on commit c936570

Please sign in to comment.