Conversation
|
Looks good. No mutations were possible for these changes. |
2 similar comments
|
Looks good. No mutations were possible for these changes. |
|
Looks good. No mutations were possible for these changes. |
73ca30b to
1c76ce2
Compare
|
Looks good. No mutations were possible for these changes. |
Stephan202
left a comment
There was a problem hiding this comment.
Rebased and added two commits. Suggested commit message:
Upgrade JUnit 5.13.4 -> 6.0.0 (#1911)
See:
- https://docs.junit.org/current/release-notes/
- https://github.com/junit-team/junit-framework/releases/tag/r6.0.0
- https://github.com/junit-team/junit-framework/releases/tag/r5.14.0
- https://github.com/junit-team/junit5/compare/r5.13.3...r6.0.0
| "assertTimeoutPreemptively(Duration, ThrowingSupplier<T>)", | ||
| "assertTimeoutPreemptively(Duration, ThrowingSupplier<T>, String)", | ||
| "assertTimeoutPreemptively(Duration, ThrowingSupplier<T>, Supplier<String>)", | ||
| "assertTimeoutPreemptively(Duration, ThrowingSupplier<T>, Supplier<String>, TimeoutFailureFactory<E>)", |
There was a problem hiding this comment.
This method was annotated @API(status = INTERNAL, since = "5.9.1") and got dropped.
| static final class AssertThatBooleanArrayWithFailMessageSupplierContainsExactly { | ||
| @BeforeTemplate | ||
| void before(boolean[] actual, Supplier<String> message, boolean[] expected) { | ||
| void before(boolean[] actual, Supplier<@Nullable String> message, boolean[] expected) { | ||
| assertArrayEquals(expected, actual, message); | ||
| } | ||
|
|
||
| @AfterTemplate | ||
| @UseImportPolicy(STATIC_IMPORT_ALWAYS) | ||
| void after(boolean[] actual, Supplier<String> message, boolean[] expected) { | ||
| void after(boolean[] actual, Supplier<@Nullable String> message, boolean[] expected) { | ||
| assertThat(actual).withFailMessage(message).containsExactly(expected); | ||
| } | ||
| } |
There was a problem hiding this comment.
JUnit now explicitly allows message suppliers to yield null. I validated that AssertJ supports this a well, so I annotated the @AfterTemplate method parameter types as well.
|
Ugh: fix one thing, then other stuff pops up. The Reviewdog errors are likely addressed by #1915. The SonarCloud warnings look like false positives; will look into the best way to suppress those. |
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
1c76ce2 to
cfeb7ce
Compare
|
Looks good. No mutations were possible for these changes. |
|
I rebased and added one more commit to suppress the SonarCloud warnings. |
|
Looks good. No mutations were possible for these changes. |
1135092 to
a86cd9e
Compare
|
Looks good. No mutations were possible for these changes. |
|
|
Suggested commit message: |



This PR contains the following updates:
5.13.4->6.0.0Release Notes
junit-team/junit-framework (JUnit)
v6.0.0: JUnit 6.0.0JUnit 6.0.0 = Platform 6.0.0 + Jupiter 6.0.0 + Vintage 6.0.0
See Release Notes.
New Contributors
Full Changelog: junit-team/junit-framework@r5.14.0...r6.0.0
v5.14.0: JUnit 5.14.0JUnit 5.14.0 = Platform 1.14.0 + Jupiter 5.14.0 + Vintage 5.14.0
See Release Notes.
Full Changelog: junit-team/junit-framework@r5.13.4...r5.14.0