integ-runner: consistent filenames between integration tests and unit tests

[mbonig]

Describe the feature

The current default regex for finding integ tests is ^integ\..*(?<!\.d)\.ts$, putting the 'integ' at the start of the filename. However, unit tests are expecting a suffix, like ^.*(?<!\.d)\.test\.ts$.

This makes for inconsistency in filenames. It believe it would make adoption easier if the default was to look for ^.*(?<!\.integ\.d)\.ts$.

Since there are already plenty of file in the existing repos under the initial regex perhaps the default could be to look for both?

Use Case

I like my filenames to have consistent naming patterns.

Proposed Solution

Make the default TS search regex look for both the current default and ^.*(?<!\.d)\.integ\.ts$.

Other Information

No response

Acknowledgements

• [X] I may be able to implement this feature request
• [ ] This feature might incur a breaking change

CDK version used

2.77.0

Environment details (OS name and version, etc.)

All

[pahud]

Thank you for the feedback. I am making this a p2 feature request and please help us prioritize by upvotes.

[moltar]

I agree with the naming. It seems odd. I went with *.itest.ts in my own setup, after consulting with ChatGPT for the naming patterns for integration tests in the Node ecosystem :)