Skip to content

Commit

Permalink
chore(integ-tests-alpha): integ tests link is incorrect (#26402)
Browse files Browse the repository at this point in the history
This PR corrects integ tests link in an integ-runner doc and an error message.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
  • Loading branch information
go-to-k committed Jul 18, 2023
1 parent 3944f31 commit 576a851
Show file tree
Hide file tree
Showing 4 changed files with 18 additions and 3 deletions.
2 changes: 1 addition & 1 deletion packages/@aws-cdk/integ-runner/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ publishing this tool so that it can be used by the community and we would love t
on use cases that the tool should support, or issues that prevent the tool from being used in your
library.

This tool is meant to be used with the [integ-tests](https://github.com/aws/aws-cdk/tree/main/packages/%40aws-cdk/integ-tests) library.
This tool is meant to be used with the [integ-tests](https://github.com/aws/aws-cdk/tree/main/packages/%40aws-cdk/integ-tests-alpha) library.

## Usage

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -77,8 +77,8 @@ export class IntegTestRunner extends IntegRunner {
// test then point the user to the new `IntegTest` construct
if (!this.hasSnapshot() && this.isLegacyTest) {
throw new Error(`${this.testName} is a new test. Please use the IntegTest construct ` +
'to configure the test\n' +
'https://github.com/aws/aws-cdk/tree/main/packages/%40aws-cdk/integ-tests',
'to configure the test\n' +
'https://github.com/aws/aws-cdk/tree/main/packages/%40aws-cdk/integ-tests-alpha',
);
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -675,4 +675,18 @@ describe('IntegTest watchIntegTest', () => {
verbose: undefined,
}));
});

test('with error', () => {
expect(() => {
// WHEN
new IntegTestRunner({
cdk: cdkMock.cdk,
test: new IntegTest({
fileName: 'test/test-data/xxxxx.test-with-error.js',
discoveryRoot: 'test/test-data',
}),
});
// THEN
}).toThrowError('xxxxx.test-with-error is a new test. Please use the IntegTest construct to configure the test\nhttps://github.com/aws/aws-cdk/tree/main/packages/%40aws-cdk/integ-tests-alpha');
});
});
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
/// !cdk-integ test-stack

0 comments on commit 576a851

Please sign in to comment.