Tim te Beek
Tim te Beek
[TestsShouldNotBePublic](https://github.com/openrewrite/rewrite-testing-frameworks/blob/v1.33.0/src/main/java/org/openrewrite/java/testing/cleanup/TestsShouldNotBePublic.java#L155) should also remove public modifier from classes containing methods meta-annotated with `@Test` and other such annotations. Seen on https://github.com/openrewrite/rewrite-maven-plugin/pull/498, where the initial commit did not pick up on [the...
[JUnit 5.9.2](https://junit.org/junit5/docs/current/release-notes/index.html#release-notes) was released today, with [Document Maven Surefire/Failsafe version alignment](https://github.com/junit-team/junit5/pull/3125/files) in the listed changes. > **Use Maven Surefire/Failsafe 3.0.0-M4 or later to avoid interoperability issues** > [SUREFIRE-1585](https://issues.apache.org/jira/browse/SUREFIRE-1585) introduced support...
[3.24.0 (2023-01-06)](https://assertj.github.io/doc/#assertj-core-3-24-0-release-notes) Deprecated `ObjectAssert(AtomicReference)` ([2795](https://github.com/assertj/assertj/pull/2795)) > Instead, use `assertThat(actual.get())` or `AtomicReferenceAssert#hasValueSatisfying(Consumer)`. [3.23.0 (2022-05-31)](https://assertj.github.io/doc/#assertj-core-3-23-0-release-notes) Deprecated `getCause` and `getRootCause` for `Throwable` assertions. > Add more `assertThat…Exception` alternatives [3.22.0 (2022-01-03)](https://assertj.github.io/doc/#assertj-core-3-22-0-release-notes) Deprecated `encodedAsBase64` /...
When using static DSL methods, the replacement `WireMockExtension` in [UseWiremockExtensionshould](https://github.com/openrewrite/rewrite-testing-frameworks/blob/main/src/main/java/org/openrewrite/java/testing/junit5/UseWiremockExtension.java#L30) have [`configureStaticDsl(true)`](https://github.com/wiremock/wiremock/blob/master/src/main/java/com/github/tomakehurst/wiremock/junit5/WireMockExtension.java#L286) called on the `Builder`, otherwise the stubs will not be registered and the test will fail.
Hi! Noticed a somewhat surprising pattern after migrating from JUnit 4 to JUnit 5, which is [captured in the unit test `assertThrowsMultiLine`](https://github.com/openrewrite/rewrite-testing-frameworks/blob/main/src/test/kotlin/org/openrewrite/java/testing/junit5/UpdateTestAnnotationTest.kt#L126): ```java @Test fun assertThrowsMultiLine() = assertChanged( before =...
## What version of OpenRewrite are you using? - OpenRewrite v8.24.0 - rewrite-migrate-java v2.12.0 ## How are you running OpenRewrite? I'm running `org.openrewrite.java.migrate.net.URLConstructorsToURI.URLThreeArgumentConstructorRecipe`, via: https://github.com/openrewrite/rewrite-migrate-java/blob/f399d72046b68fd0820a6b326badfd5ff9482105/src/main/java/org/openrewrite/java/migrate/net/URLConstructorsToURI.java#L44-L58 ## What is the smallest,...
Mostly to flush out issues with the build, and to be able to add new `SequencedCollection` recipes for getFirst, getLast & addFirst.
## What's your motivation? Fixes https://github.com/openrewrite/rewrite-migrate-java/issues/390 ## Anything in particular you'd like reviewers to focus on? Added a new source set for that Java 9+ classes needed for these replacements;...
[@FieldDefaults](https://projectlombok.org/features/experimental/FieldDefaults) is an experimental Lombok feature that along with a few sibling annotations allows users to forgo adding field modifiers to individual fields. So ```java import lombok.AccessLevel; import lombok.experimental.FieldDefaults; import...
We have an existing recipe to [upgrade Gradle wrappers](https://github.com/openrewrite/rewrite/blob/v7.35.0/rewrite-gradle/src/main/java/org/openrewrite/gradle/UpdateGradleWrapper.java); When upgrading applications to Java 11, 17 and beyond, we should also upgrade the Gradle wrapper to the corresponding versions as...