Remove unused tryToLoadResource

junit-platform-commons/src/main/java/org/junit/platform/commons/support/ReflectionSupport.java

@@ -19,7 +19,6 @@
 import java.net.URI;
 import java.util.List;
 import java.util.Optional;
-import java.util.function.Function;
 import java.util.function.Predicate;
 import java.util.stream.Stream;
 
@@ -114,27 +113,6 @@ public static Try<Class<?>> tryToLoadClass(String name, ClassLoader classLoader)
 		return ReflectionUtils.tryToLoadClass(name, classLoader);
 	}
 
-	/**
-	 * Tries to load the {@link Resource} for the supplied classpath resource name.
-	 *
-	 * <p>The name of a <em>classpath resource</em> must follow the semantics
-	 * for resource paths as defined in {@link ClassLoader#getResource(String)}.
-	 *
-	 * <p>If the supplied classpath resource name is prefixed with a slash
-	 * ({@code /}), the slash will be removed.
-	 *
-	 * @param classpathResourceName the name of the resource to load; never {@code null} or blank
-	 * @param classLoader the {@code ClassLoader} to use; never {@code null}
-	 * @return a successful {@code Try} containing the loaded class or a failed
-	 * {@code Try} containing the exception if no such resource could be loaded;
-	 * never {@code null}
-	 * @since 1.11
-	 */
-	@API(status = EXPERIMENTAL, since = "1.11")
-	public static Try<Resource> tryToLoadResource(String classpathResourceName, ClassLoader classLoader) {
-		return ReflectionUtils.tryToLoadResource(classpathResourceName, classLoader);
-	}
-
 	/**
 	 * Find all {@linkplain Class classes} in the supplied classpath {@code root}
 	 * that match the specified {@code classFilter} and {@code classNameFilter}

junit-platform-commons/src/main/java/org/junit/platform/commons/util/ReflectionUtils.java

@@ -35,7 +35,6 @@
 import java.lang.reflect.Type;
 import java.lang.reflect.TypeVariable;
 import java.net.URI;
-import java.net.URL;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.nio.file.Paths;
@@ -891,33 +890,6 @@ public static Try<Class<?>> tryToLoadClass(String name, ClassLoader classLoader)
 		});
 	}
 
-	/**
-	 * Tries to load the {@link Resource} for the supplied classpath resource name.
-	 *
-	 * <p>See {@link org.junit.platform.commons.support.ReflectionSupport#tryToLoadResource(String, ClassLoader)}
-	 * for details.
-	 *
-	 * @param classpathResourceName the name of the resource to load; never {@code null} or blank
-	 * @param classLoader the {@code ClassLoader} to use; never {@code null}
-	 * @since 1.11
-	 */
-	@API(status = INTERNAL, since = "1.11")
-	public static Try<Resource> tryToLoadResource(String classpathResourceName, ClassLoader classLoader) {
-		Preconditions.notBlank(classpathResourceName, "Resource name must not be null or blank");
-		Preconditions.notNull(classLoader, "ClassLoader must not be null");
-		boolean startsWithSlash = classpathResourceName.startsWith("/");
-		String canonicalClasspathResourceName = (startsWithSlash ? classpathResourceName.substring(1)
-				: classpathResourceName);
-
-		return Try.call(() -> {
-			URL resource = classLoader.getResource(canonicalClasspathResourceName);
-			if (resource == null) {
-				throw new NullPointerException("classLoader.getResource returned null");
-			}
-			return new ClasspathResource(canonicalClasspathResourceName, resource.toURI());
-		});
-	}
-
 	private static Class<?> loadArrayType(ClassLoader classLoader, String componentTypeName, int dimensions)
 			throws ClassNotFoundException {
 

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

@@ -91,17 +91,6 @@ void tryToLoadClassWithExplicitClassLoaderDelegates() {
 			ReflectionSupport.tryToLoadClass("java.nio.Bits", classLoader));
 	}
 
-	/**
-	 * @since 1.11
-	 */
-	@Test
-	void tryToLoadResourcesWithExplicitClassLoaderDelegates() {
-		var classLoader = getClass().getClassLoader();
-		var resource = "org/junit/platform/commons/example.resource";
-		assertEquals(ReflectionUtils.tryToLoadResource(resource, classLoader).toOptional(),
-			ReflectionSupport.tryToLoadResource(resource, classLoader).toOptional());
-	}
-
 	/**
 	 * @since 1.10
 	 */
@@ -115,21 +104,6 @@ void tryToLoadClassWithExplicitClassLoaderPreconditions() {
 		assertPreconditionViolationException("ClassLoader", () -> ReflectionSupport.tryToLoadClass("int", null));
 	}
 
-	/**
-	 * @since 1.11
-	 */
-	@Test
-	void tryToLoadResourceWithExplicitClassLoaderPreconditions() {
-		ClassLoader cl = getClass().getClassLoader();
-
-		assertPreconditionViolationExceptionForString("Resource name",
-			() -> ReflectionSupport.tryToLoadResource(null, cl));
-		assertPreconditionViolationExceptionForString("Resource name",
-			() -> ReflectionSupport.tryToLoadResource("", cl));
-
-		assertPreconditionViolationException("ClassLoader", () -> ReflectionSupport.tryToLoadResource("int", null));
-	}
-
 	@TestFactory
 	List<DynamicTest> findAllClassesInClasspathRootDelegates() throws Throwable {
 		List<DynamicTest> tests = new ArrayList<>();

platform-tests/src/test/java/org/junit/platform/commons/util/ReflectionUtilsTests.java

@@ -710,46 +710,6 @@ private void privateMethod() {
 
 	}
 
-	@Nested
-	class ResourceLoadingTests {
-
-		@Test
-		void tryToLoadResourcePreconditions() {
-			assertThrows(PreconditionViolationException.class, () -> ReflectionUtils.tryToLoadResource("", null));
-			assertThrows(PreconditionViolationException.class, () -> ReflectionUtils.tryToLoadResource("   ", null));
-
-			assertThrows(PreconditionViolationException.class, () -> ReflectionUtils.tryToLoadResource(null, null));
-			assertThrows(PreconditionViolationException.class,
-				() -> ReflectionUtils.tryToLoadResource("org/junit/platform/commons/example.resource", null));
-		}
-
-		@Test
-		void tryToLoadResource() {
-			var classLoader = ClassLoaderUtils.getDefaultClassLoader();
-			var tryToLoadResource = ReflectionUtils.tryToLoadResource("org/junit/platform/commons/example.resource",
-				classLoader);
-			var resource = assertDoesNotThrow(tryToLoadResource::get);
-			assertThat(resource.getName()).isEqualTo("org/junit/platform/commons/example.resource");
-		}
-
-		@Test
-		void tryToLoadResourceWithPrefixedSlash() {
-			var classLoader = ClassLoaderUtils.getDefaultClassLoader();
-			var tryToLoadResource = ReflectionUtils.tryToLoadResource("/org/junit/platform/commons/example.resource",
-				classLoader);
-			var resource = assertDoesNotThrow(tryToLoadResource::get);
-			assertThat(resource.getName()).isEqualTo("org/junit/platform/commons/example.resource");
-		}
-
-		@Test
-		void tryToLoadResourceWhenResourceNotFound() {
-			var classLoader = ClassLoaderUtils.getDefaultClassLoader();
-			var tryToLoadResource = ReflectionUtils.tryToLoadResource("org/junit/platform/commons/no-such.resource",
-				classLoader);
-			assertThrows(NullPointerException.class, tryToLoadResource::get);
-		}
-	}
-
 	@Nested
 	class ClassLoadingTests {