-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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.
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
Exclude anonymous class in JUnit Platform #4774
Conversation
We should exclude them on the daemon side already. I don't think passing anononymous classes makes sense for any test framework. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I was gonna propose using a test class processor, but by filtering these in the scanner we avoid visiting the class, which saves some additional time 👍
@oehme Your thoughts are exactly same as me :) |
abbb871
to
cbec130
Compare
cbec130
to
2d39c91
Compare
This reverts commit abeb7eb.
* master: (30 commits) Fix NPE in capabilities conflict resolution By default don't render the welcome message to avoid test flakiness (#4796) Change console tests to require parallel execution only for specific tests Update user guide to reflect new plain console format Ensure console tests run with plain, rich and verbose modes Revert "Exclude anonymous class in JUnit Platform (#4774)" Add @ToBeImplemented integration tests for #4799 Always report Checkstyle violations-summary from xml report (#3901) Remove the verbose flag by default for `swiftc` Fix failing unit tests Fix test failure due to missing API export from application Fix tests and share code between C++ and Swift Wire operatingSystem into Swift binary types Remove spurious newline from tests that check output Fix issue with timeout in test Tasks that print output to the console hold it for as long as possible Up long running task timeout to 30s Create directory Fix path for Windows Create marker file to prevent rendering of initial welcome message ...
This fixes #4544 #4618
Previously we send anonymous class to JUnit Platform test worker, which caused some issues.
Now they're excluded by name pattern
.*\$\d+