Skip to content

Commit 8494ad0

Browse files
authored
chore: fix integ runner by not showing notices (#33551)
### Issue # (if applicable) Closes #<issue number here>. ### Reason for this change PR CI consistently failed due to notices showing up in the cdk list command and notice is considered as stacks. ### Description of changes Force notices flag to false ### Description of how you validated changes CI passes ### Checklist - [ ] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent be2c7d0 commit 8494ad0

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

packages/@aws-cdk/integ-runner/lib/runner/integ-test-suite.ts

+4-2
Original file line numberDiff line numberDiff line change
@@ -135,9 +135,11 @@ export class LegacyIntegTestSuite extends IntegTestSuite {
135135
if (pragma.length > 0) {
136136
tests.stacks.push(...pragma);
137137
} else {
138-
const stacks = (config.cdk.list({
138+
const options: ListOptions = {
139139
...config.listOptions,
140-
})).split('\n');
140+
notices: false,
141+
};
142+
const stacks = (config.cdk.list(options)).split('\n');
141143
if (stacks.length !== 1) {
142144
throw new Error('"cdk-integ" can only operate on apps with a single stack.\n\n' +
143145
' If your app has multiple stacks, specify which stack to select by adding this to your test source:\n\n' +

0 commit comments

Comments
 (0)