[openrewrite] add SanityCheck featuring tech.picnic.errorprone.refasterrules

Author: Vincent Potucek

build.gradle.kts

@@ -7,6 +7,7 @@ plugins {
 	id("junitbuild.jacoco-aggregation-conventions")
 	id("junitbuild.maven-central-publishing")
 	id("junitbuild.temp-maven-repo")
+	id("org.openrewrite.rewrite") version "7.17.0" apply false
 }
 
 description = "JUnit"
@@ -54,3 +55,5 @@ dependencies {
 	jacocoAggregation(projects.jupiterTests)
 	jacocoAggregation(projects.platformTests)
 }
+
+apply(from = "$rootDir/gradle/rewrite.gradle")

gradle/rewrite.gradle

@@ -0,0 +1,17 @@
+apply plugin: "org.openrewrite.rewrite"
+
+rewrite {
+	activeRecipe("org.junit.jupiter.openrewrite.SanityCheck")
+	exclusions.addAll("**.ci-temp**")
+	exportDatatables = true
+	failOnDryRunResults = true
+}
+
+dependencies {
+	rewrite(platform("org.openrewrite.recipe:rewrite-recipe-bom:3.15.0"))
+	rewrite("org.openrewrite.recipe:rewrite-java-security:3.19.0")
+	rewrite("org.openrewrite.recipe:rewrite-migrate-java:3.18.0")
+	rewrite("org.openrewrite.recipe:rewrite-rewrite:0.13.0")
+	rewrite("org.openrewrite.recipe:rewrite-static-analysis:2.18.0")
+	rewrite("org.openrewrite.recipe:rewrite-third-party:0.28.0")
+}

junit-platform-engine/src/main/java/org/junit/platform/engine/UniqueId.java

@@ -110,7 +110,7 @@ private UniqueId(UniqueIdFormat uniqueIdFormat, Segment segment) {
 	}
 
 	Optional<Segment> getRoot() {
-		return this.segments.isEmpty() ? Optional.empty() : Optional.of(this.segments.get(0));
+		return this.segments.stream().findFirst();
 	}
 
 	/**

junit-platform-engine/src/main/java/org/junit/platform/engine/support/hierarchical/DefaultParallelExecutionConfigurationStrategy.java

@@ -66,7 +66,7 @@ public ParallelExecutionConfiguration createConfiguration(ConfigurationParameter
 			BigDecimal factor = configurationParameters.get(CONFIG_DYNAMIC_FACTOR_PROPERTY_NAME,
 				BigDecimal::new).orElse(BigDecimal.ONE);
 
-			Preconditions.condition(factor.compareTo(BigDecimal.ZERO) > 0,
+			Preconditions.condition(factor.signum() == 1,
 				() -> "Factor '%s' specified via configuration parameter '%s' must be greater than 0".formatted(factor,
 					CONFIG_DYNAMIC_FACTOR_PROPERTY_NAME));
 

junit-platform-launcher/src/main/java/org/junit/platform/launcher/EngineFilter.java

@@ -126,7 +126,7 @@ public FilterResult apply(TestEngine testEngine) {
 		Preconditions.notBlank(engineId, "TestEngine ID must not be null or blank");
 
 		if (this.type == Type.INCLUDE) {
-			return includedIf(this.engineIds.stream().anyMatch(engineId::equals), //
+			return includedIf(this.engineIds.contains(engineId), //
 				() -> "Engine ID [%s] is in included list [%s]".formatted(engineId, this.engineIds), //
 				() -> "Engine ID [%s] is not in included list [%s]".formatted(engineId, this.engineIds));
 		}

junit-platform-launcher/src/main/java/org/junit/platform/launcher/listeners/OutputDir.java

@@ -140,7 +140,7 @@ private static boolean containsFilesWithExtensions(Path dir, String... extension
 			return false;
 		};
 		try (Stream<Path> pathStream = Files.find(dir, 1, matcher)) {
-			return pathStream.findFirst().isPresent();
+			return pathStream.findAny().isPresent();
 		}
 	}
 }

jupiter-tests/src/test/java/org/junit/jupiter/engine/discovery/predicates/IsTestFactoryMethodTests.java

@@ -174,7 +174,7 @@ Collection rawCollectionFactory() {
 
 		@TestFactory
 		Stream<?> unboundStreamFactory() {
-			return Stream.of();
+			return Stream.empty();
 		}
 
 	}

platform-tests/src/test/java/org/junit/platform/commons/support/conversion/ConversionSupportTests.java

@@ -268,7 +268,7 @@ void convertsStringToURL() throws Exception {
 	@Test
 	void convertsStringsToJavaTimeInstances() {
 		assertConverts("PT1234.5678S", Duration.class, Duration.ofSeconds(1234, 567800000));
-		assertConverts("1970-01-01T00:00:00Z", Instant.class, Instant.ofEpochMilli(0));
+		assertConverts("1970-01-01T00:00:00Z", Instant.class, Instant.EPOCH);
 		assertConverts("2017-03-14", LocalDate.class, LocalDate.of(2017, 3, 14));
 		assertConverts("2017-03-14T12:34:56.789", LocalDateTime.class,
 			LocalDateTime.of(2017, 3, 14, 12, 34, 56, 789_000_000));

platform-tests/src/test/java/org/junit/platform/engine/support/descriptor/AbstractTestSourceTests.java

@@ -60,7 +60,7 @@ private <T extends Serializable> void assertSerializable(T instance) {
 			var serialized = serialize(instance);
 			var deserialized = deserialize(serialized);
 
-			assertTrue(type.isAssignableFrom(deserialized.getClass()));
+			assertTrue(type.isInstance(deserialized));
 			assertEquals(instance, deserialized);
 		}
 		catch (Exception e) {

platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/LocalMavenRepo.java

@@ -69,7 +69,7 @@ public String toCliArgument() {
 	@Override
 	public void close() throws IOException {
 		try (var files = Files.walk(tempDir)) {
-			files.sorted(Comparator.<Path> naturalOrder().reversed()) //
+			files.sorted(Comparator.reverseOrder()) //
 					.forEach(path -> {
 						try {
 							Files.delete(path);