Skip to content

Commit 5fd9e88

Browse files
committed
Remove Void wrapper registration in ReflectionUtils
Closes #4048
1 parent dde72a9 commit 5fd9e88

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

documentation/src/docs/asciidoc/release-notes/release-notes-5.12.0-M1.adoc

-1
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@ JUnit repository on GitHub.
5252
* The `ReflectionSupport.tryToLoadClass(...)` utility methods now support lookups for the
5353
`"void"` pseudo-type, which indirectly supports `String` to `Class` conversion for
5454
`"void"` in parameterized tests in JUnit Jupiter.
55-
* `ReflectionUtils` now treats `Void` as a _wrapper_ type for the `void` pseudo-type.
5655
* New `addResourceContainerSelectorResolver()` method in
5756
`EngineDiscoveryRequestResolver.Builder` which supports the discovery of class path
5857
resource based tests, analogous to the existing `addClassContainerSelectorResolver()`

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

-1
Original file line numberDiff line numberDiff line change
@@ -263,7 +263,6 @@ public enum HierarchyTraversalMode {
263263
primitivesToWrappers.put(long.class, Long.class);
264264
primitivesToWrappers.put(float.class, Float.class);
265265
primitivesToWrappers.put(double.class, Double.class);
266-
primitivesToWrappers.put(void.class, Void.class);
267266

268267
primitiveToWrapperMap = Collections.unmodifiableMap(primitivesToWrappers);
269268
}

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

+3-1
Original file line numberDiff line numberDiff line change
@@ -561,7 +561,9 @@ void isAssignableTo() {
561561
// Wrappers to Primitives
562562
assertTrue(ReflectionUtils.isAssignableTo(Integer.class, int.class));
563563
assertTrue(ReflectionUtils.isAssignableTo(Boolean.class, boolean.class));
564-
assertTrue(ReflectionUtils.isAssignableTo(Void.class, void.class));
564+
565+
// Void to void
566+
assertFalse(ReflectionUtils.isAssignableTo(Void.class, void.class));
565567

566568
// Widening Conversions from Wrappers to Primitives
567569
assertTrue(ReflectionUtils.isAssignableTo(Integer.class, long.class));

0 commit comments

Comments
 (0)