Skip to content

Commit c0a6e08

Browse files
authored
chore: add new style integration test snapshots (#19802)
The bulk of this PR is adding the new style integration test snapshots that the new `integ-runner` uses. These are snapshots of the cloud assembly and have folder names like `${testName}.integ.snapshot`. Some other minor updates were made as part of this change. 1. Updated `integ-runner` to use `synthFast` instead of `synth` to speed up performance. 2. Make sure the `integ-runner` throws an exit code if any tests fail 3. Updated the test names to include the directory if running from a parent directory (i.e. `packages/@aws-cdk`). 4. Added the ability to exempt from the update workflow (update workflow is not yet implemented). If a test is exempt then the snapshot will not contain any assets. For now this is used for tests that include very large assets that would drastically increase the size of the repo. 5. Updated some custom resources to exclude `*.ts` files from the asset bundle. `.ts` files are already excluded when we publish (`.d.ts` files are not), and `.js` files are the only thing needed by the Lambda function anyway. This was causing issues when running a build because those files in the snapshot would cause `tsc` to fail. - Updated: `aws-events-targets/lib/aws-api.ts` `aws-redshift/lib/private/database-query.ts` `aws-stepfunctions-tasks/lib/evaluate-expression.ts` `custom-resources/lib/aws-custom-resource/aws-custom-resource.ts` `custom-resources/lib/provider-framework/provider.ts` ---- ### All Submissions: * [x] Have you followed the guidelines in our [Contributing guide?](https://github.com/aws/aws-cdk/blob/master/CONTRIBUTING.md) ### Adding new Unconventional Dependencies: * [ ] This PR adds new unconventional dependencies following the process described [here](https://github.com/aws/aws-cdk/blob/master/CONTRIBUTING.md/#adding-new-unconventional-dependencies) ### New Features * [x] Have you added the new feature to an [integration test](https://github.com/aws/aws-cdk/blob/master/INTEGRATION_TESTS.md)? * [x] Did you use `cdk-integ` to deploy the infrastructure and generate the snapshot (i.e. `cdk-integ` without `--dry-run`)? *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent 8ea0a1d commit c0a6e08

File tree

3,033 files changed

+771008
-603
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

3,033 files changed

+771008
-603
lines changed

packages/@aws-cdk/alexa-ask/.gitignore

+5-1
Original file line numberDiff line numberDiff line change
@@ -14,4 +14,8 @@ tsconfig.json
1414
!.eslintrc.js
1515
!jest.config.js
1616

17-
junit.xml
17+
junit.xml
18+
!**/*.integ.snapshot/**/asset.*/*.js
19+
!**/*.integ.snapshot/**/asset.*/*.d.ts
20+
21+
!**/*.integ.snapshot/**/asset.*/**

packages/@aws-cdk/alexa-ask/.npmignore

+3-1
Original file line numberDiff line numberDiff line change
@@ -28,4 +28,6 @@ jest.config.js
2828
**/cdk.out
2929
junit.xml
3030
test/
31-
!*.lit.ts
31+
!*.lit.ts
32+
**/*.integ.snapshot
33+
**/*.integ.snapshot

packages/@aws-cdk/app-delivery/.gitignore

+5-1
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,8 @@ coverage
1212
!.eslintrc.js
1313

1414
junit.xml
15-
!jest.config.js
15+
!jest.config.js
16+
!**/*.integ.snapshot/**/asset.*/*.js
17+
!**/*.integ.snapshot/**/asset.*/*.d.ts
18+
19+
!**/*.integ.snapshot/**/asset.*/**

packages/@aws-cdk/app-delivery/.npmignore

+3-1
Original file line numberDiff line numberDiff line change
@@ -23,4 +23,6 @@ tsconfig.json
2323
junit.xml
2424
test/
2525
!*.lit.ts
26-
jest.config.js
26+
jest.config.js
27+
**/*.integ.snapshot
28+
**/*.integ.snapshot

0 commit comments

Comments
 (0)