Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(integ-tests-alpha): assertions handler is broken #26400

Merged
merged 18 commits into from
Jul 18, 2023
Merged

fix(integ-tests-alpha): assertions handler is broken #26400

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
e20be26
fix(integ-tests-alpha): assertions handler is broken
mrgrain Jul 18, 2023
2918611
fixup
mrgrain Jul 18, 2023
81c3e26
re-use shared v2-to-v3 code
mrgrain Jul 18, 2023
5cb44e8
fixup
mrgrain Jul 18, 2023
903cffe
fix broken assertions when outpath is not used
mrgrain Jul 18, 2023
796cc70
fix custom-resource tests
mrgrain Jul 18, 2023
36fff9f
update integ snapshots
mrgrain Jul 18, 2023
d923e0f
snaps
mrgrain Jul 18, 2023
9e4f7a2
do not always set output path
mrgrain Jul 18, 2023
d4e9b9f
fix broken dynamic imports
mrgrain Jul 18, 2023
9ced51c
more snaps
mrgrain Jul 18, 2023
45372d4
Fix integ-test assertions, request paramter types are incorrectly con…
mrgrain Jul 18, 2023
66a9e81
update assertions integ test
mrgrain Jul 18, 2023
cc000fe
update more snaps
mrgrain Jul 18, 2023
d2a001d
update dry-run snapshots
mrgrain Jul 18, 2023
3d8732b
final snaps
mrgrain Jul 18, 2023
a3a5f0d
Merge branch 'main' into mrgrain/fix/integ-tests-assertions
mrgrain Jul 18, 2023
92faf5e
update all snapshots
mrgrain Jul 18, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
766 changes: 654 additions & 112 deletions ...5a3a49f87ce643821594ca2b4.bundle/index.js → ...7ab4fef3d27df0cd689d4fdeb.bundle/index.js
View file
Open in desktop

Large diffs are not rendered by default.

10 changes: 5 additions & 5 deletions ...loyment.js.snapshot/integtestbucketdeploymentsDefaultTestDeployAssertCF25A2DF.assets.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,28 +1,28 @@
{
"version": "32.0.0",
"files": {
"a5c310b46637b9f23d0cd48c7ae357074563c375a3a49f87ce643821594ca2b4": {
"3e126fa5845d6a9bb767cd6134d382be1a5f3eb7ab4fef3d27df0cd689d4fdeb": {
"source": {
"path": "asset.a5c310b46637b9f23d0cd48c7ae357074563c375a3a49f87ce643821594ca2b4.bundle",
"path": "asset.3e126fa5845d6a9bb767cd6134d382be1a5f3eb7ab4fef3d27df0cd689d4fdeb.bundle",
"packaging": "zip"
},
"destinations": {
"current_account-current_region": {
"bucketName": "cdk-hnb659fds-assets-${AWS::AccountId}-${AWS::Region}",
"objectKey": "a5c310b46637b9f23d0cd48c7ae357074563c375a3a49f87ce643821594ca2b4.zip",
"objectKey": "3e126fa5845d6a9bb767cd6134d382be1a5f3eb7ab4fef3d27df0cd689d4fdeb.zip",
"assumeRoleArn": "arn:${AWS::Partition}:iam::${AWS::AccountId}:role/cdk-hnb659fds-file-publishing-role-${AWS::AccountId}-${AWS::Region}"
}
}
},
"2787dc85c864ad8be0d7a6fbb03050b38b373469a5e855d472fad293a7643bf6": {
"24181388b265eb07f3a94e1407c608fa4f24d709e1e34c306b4a7419521a6086": {
"source": {
"path": "integtestbucketdeploymentsDefaultTestDeployAssertCF25A2DF.template.json",
"packaging": "file"
},
"destinations": {
"current_account-current_region": {
"bucketName": "cdk-hnb659fds-assets-${AWS::AccountId}-${AWS::Region}",
"objectKey": "2787dc85c864ad8be0d7a6fbb03050b38b373469a5e855d472fad293a7643bf6.json",
"objectKey": "24181388b265eb07f3a94e1407c608fa4f24d709e1e34c306b4a7419521a6086.json",
"assumeRoleArn": "arn:${AWS::Partition}:iam::${AWS::AccountId}:role/cdk-hnb659fds-file-publishing-role-${AWS::AccountId}-${AWS::Region}"
}
}
Expand Down
5 changes: 3 additions & 2 deletions ...yment.js.snapshot/integtestbucketdeploymentsDefaultTestDeployAssertCF25A2DF.template.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,8 @@
}
},
"flattenResponse": "false",
"salt": "1688651249283"
"outputPaths": [],
"salt": "1689681336539"
},
"UpdateReplacePolicy": "Delete",
"DeletionPolicy": "Delete"
Expand Down Expand Up @@ -82,7 +83,7 @@
"S3Bucket": {
"Fn::Sub": "cdk-hnb659fds-assets-${AWS::AccountId}-${AWS::Region}"
},
"S3Key": "a5c310b46637b9f23d0cd48c7ae357074563c375a3a49f87ce643821594ca2b4.zip"
"S3Key": "3e126fa5845d6a9bb767cd6134d382be1a5f3eb7ab4fef3d27df0cd689d4fdeb.zip"
},
"Timeout": 120,
"Handler": "index.handler",
Expand Down
4 changes: 2 additions & 2 deletions ...ework-integ/test/aws-s3-deployment/test/integ.bucket-deployment.js.snapshot/manifest.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
"validateOnSynth": false,
"assumeRoleArn": "arn:${AWS::Partition}:iam::${AWS::AccountId}:role/cdk-hnb659fds-deploy-role-${AWS::AccountId}-${AWS::Region}",
"cloudFormationExecutionRoleArn": "arn:${AWS::Partition}:iam::${AWS::AccountId}:role/cdk-hnb659fds-cfn-exec-role-${AWS::AccountId}-${AWS::Region}",
"stackTemplateAssetObjectUrl": "s3://cdk-hnb659fds-assets-${AWS::AccountId}-${AWS::Region}/35e743b7bf5de46e55840754911f3cb567422c47dc6971ff122e32490ffac934.json",
"stackTemplateAssetObjectUrl": "s3://cdk-hnb659fds-assets-${AWS::AccountId}-${AWS::Region}/80f43f6b3b34fefa1ee2d8889ca3fc369530080f6fa4f91138bd94ce37560521.json",
"requiresBootstrapStackVersion": 6,
"bootstrapStackVersionSsmParameter": "/cdk-bootstrap/hnb659fds/version",
"additionalDependencies": [
Expand Down Expand Up @@ -484,7 +484,7 @@
"validateOnSynth": false,
"assumeRoleArn": "arn:${AWS::Partition}:iam::${AWS::AccountId}:role/cdk-hnb659fds-deploy-role-${AWS::AccountId}-${AWS::Region}",
"cloudFormationExecutionRoleArn": "arn:${AWS::Partition}:iam::${AWS::AccountId}:role/cdk-hnb659fds-cfn-exec-role-${AWS::AccountId}-${AWS::Region}",
"stackTemplateAssetObjectUrl": "s3://cdk-hnb659fds-assets-${AWS::AccountId}-${AWS::Region}/2787dc85c864ad8be0d7a6fbb03050b38b373469a5e855d472fad293a7643bf6.json",
"stackTemplateAssetObjectUrl": "s3://cdk-hnb659fds-assets-${AWS::AccountId}-${AWS::Region}/24181388b265eb07f3a94e1407c608fa4f24d709e1e34c306b4a7419521a6086.json",
"requiresBootstrapStackVersion": 6,
"bootstrapStackVersionSsmParameter": "/cdk-bootstrap/hnb659fds/version",
"additionalDependencies": [
Expand Down
4 changes: 2 additions & 2 deletions ...deployment/test/integ.bucket-deployment.js.snapshot/test-bucket-deployments-2.assets.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,15 +79,15 @@
}
}
},
"35e743b7bf5de46e55840754911f3cb567422c47dc6971ff122e32490ffac934": {
"80f43f6b3b34fefa1ee2d8889ca3fc369530080f6fa4f91138bd94ce37560521": {
"source": {
"path": "test-bucket-deployments-2.template.json",
"packaging": "file"
},
"destinations": {
"current_account-current_region": {
"bucketName": "cdk-hnb659fds-assets-${AWS::AccountId}-${AWS::Region}",
"objectKey": "35e743b7bf5de46e55840754911f3cb567422c47dc6971ff122e32490ffac934.json",
"objectKey": "80f43f6b3b34fefa1ee2d8889ca3fc369530080f6fa4f91138bd94ce37560521.json",
"assumeRoleArn": "arn:${AWS::Partition}:iam::${AWS::AccountId}:role/cdk-hnb659fds-file-publishing-role-${AWS::AccountId}-${AWS::Region}"
}
}
Expand Down
Loading