Skip to content

Add junit-extensions to test classpaths#19797

Merged
findepi merged 1 commit intotrinodb:masterfrom
findepi:findepi/add-junit-extensions-to-test-classpaths-8c631b
Nov 17, 2023
Merged

Add junit-extensions to test classpaths#19797
findepi merged 1 commit intotrinodb:masterfrom
findepi:findepi/add-junit-extensions-to-test-classpaths-8c631b

Conversation

@findepi
Copy link
Copy Markdown
Member

@findepi findepi commented Nov 17, 2023

junit-extensions is necessary safety extensions to JUnit, preventing JUnit "UX trap" which makes overridden tests silently skipped. It should be on every module's classpath.

For the record, the modules were identified using

find -name pom.xml | grep -vFxf <(find -name pom.xml -exec grep -l junit-extensions {} +)

@cla-bot cla-bot bot added the cla-signed label Nov 17, 2023
@findepi findepi force-pushed the findepi/add-junit-extensions-to-test-classpaths-8c631b branch from 5dcc6f0 to 6e94622 Compare November 17, 2023 14:27
`junit-extensions` is necessary safety extensions to JUnit, preventing
JUnit "UX trap" which makes overridden tests silently skipped. It should
be on every module's classpath.

For the record, the modules were identified using

    find -name pom.xml | grep -vFxf <(find -name pom.xml -exec grep -l junit-extensions {} +)

And some left out.
@findepi findepi force-pushed the findepi/add-junit-extensions-to-test-classpaths-8c631b branch from 6e94622 to 3b28739 Compare November 17, 2023 15:04
@findepi
Copy link
Copy Markdown
Member Author

findepi commented Nov 17, 2023

CI #19799

@findepi findepi added test no-release-notes This pull request does not require release notes entry labels Nov 17, 2023
@findepi findepi merged commit d4d7d16 into trinodb:master Nov 17, 2023
@findepi findepi deleted the findepi/add-junit-extensions-to-test-classpaths-8c631b branch November 17, 2023 16:16
@github-actions github-actions bot added this to the 434 milestone Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed no-release-notes This pull request does not require release notes entry test

Development

Successfully merging this pull request may close these issues.

2 participants