Skip to content

Commit 1ff6e39

Browse files
mp911demp911de
committed
Adopt to JUnit 5.9 changes.
Avoid duplicate parameter factory names. Closes #2702
1 parent 7027df9 commit 1ff6e39

File tree

2 files changed

+46
-44
lines changed

2 files changed

+46
-44
lines changed

‎src/test/java/org/springframework/data/mapping/model/ClassGeneratingPropertyAccessorFactoryDatatypeTests.java

Lines changed: 36 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@
3939
* @author Mark Paluch
4040
* @author Oliver Gierke
4141
*/
42+
@SuppressWarnings("WeakerAccess") // public required for class generation due to visibility rules
4243
public class ClassGeneratingPropertyAccessorFactoryDatatypeTests {
4344

4445
private final ClassGeneratingPropertyAccessorFactory factory = new ClassGeneratingPropertyAccessorFactory();
@@ -50,45 +51,45 @@ static List<Object[]> parameters() throws Exception {
5051
var types = Arrays.asList(FieldAccess.class, PropertyAccess.class, PrivateFinalFieldAccess.class,
5152
PrivateFinalPropertyAccess.class);
5253

53-
parameters.addAll(parameters(types, "primitiveInteger", Integer.valueOf(1)));
54-
parameters.addAll(parameters(types, "primitiveIntegerArray", new int[] { 1, 2, 3 }));
55-
parameters.addAll(parameters(types, "boxedInteger", Integer.valueOf(1)));
56-
parameters.addAll(parameters(types, "boxedIntegerArray", new Integer[] { Integer.valueOf(1) }));
57-
parameters.addAll(parameters(types, "primitiveShort", Short.valueOf("1")));
58-
parameters.addAll(parameters(types, "primitiveShortArray", new short[] { 1, 2, 3 }));
59-
parameters.addAll(parameters(types, "boxedShort", Short.valueOf("1")));
60-
parameters.addAll(parameters(types, "boxedShortArray", new Short[] { Short.valueOf("1") }));
61-
parameters.addAll(parameters(types, "primitiveByte", Byte.valueOf("1")));
62-
parameters.addAll(parameters(types, "primitiveByteArray", new byte[] { 1, 2, 3 }));
63-
parameters.addAll(parameters(types, "boxedByte", Byte.valueOf("1")));
64-
parameters.addAll(parameters(types, "boxedByteArray", new Byte[] { Byte.valueOf("1") }));
65-
parameters.addAll(parameters(types, "primitiveChar", Character.valueOf('c')));
66-
parameters.addAll(parameters(types, "primitiveCharArray", new char[] { 'a', 'b', 'c' }));
67-
parameters.addAll(parameters(types, "boxedChar", Character.valueOf('c')));
68-
parameters.addAll(parameters(types, "boxedCharArray", new Character[] { Character.valueOf('c') }));
69-
parameters.addAll(parameters(types, "primitiveBoolean", Boolean.valueOf(true)));
70-
parameters.addAll(parameters(types, "primitiveBooleanArray", new boolean[] { true, false }));
71-
parameters.addAll(parameters(types, "boxedBoolean", Boolean.valueOf(true)));
72-
parameters.addAll(parameters(types, "boxedBooleanArray", new Boolean[] { Boolean.valueOf(true) }));
73-
parameters.addAll(parameters(types, "primitiveFloat", Float.valueOf(1f)));
74-
parameters.addAll(parameters(types, "primitiveFloatArray", new float[] { 1f, 2f }));
75-
parameters.addAll(parameters(types, "boxedFloat", Float.valueOf(1f)));
76-
parameters.addAll(parameters(types, "boxedFloatArray", new Float[] { Float.valueOf(1f) }));
77-
parameters.addAll(parameters(types, "primitiveDouble", Double.valueOf(1d)));
78-
parameters.addAll(parameters(types, "primitiveDoubleArray", new double[] { 1d, 2d }));
79-
parameters.addAll(parameters(types, "boxedDouble", Double.valueOf(1d)));
80-
parameters.addAll(parameters(types, "boxedDoubleArray", new Double[] { Double.valueOf(1d) }));
81-
parameters.addAll(parameters(types, "primitiveLong", Long.valueOf(1L)));
82-
parameters.addAll(parameters(types, "primitiveLongArray", new long[] { 1L, 2L }));
83-
parameters.addAll(parameters(types, "boxedLong", Long.valueOf(1L)));
84-
parameters.addAll(parameters(types, "boxedLongArray", new Long[] { Long.valueOf(1L) }));
85-
parameters.addAll(parameters(types, "string", "hello"));
86-
parameters.addAll(parameters(types, "stringArray", new String[] { "hello", "world" }));
54+
parameters.addAll(create(types, "primitiveInteger", Integer.valueOf(1)));
55+
parameters.addAll(create(types, "primitiveIntegerArray", new int[] { 1, 2, 3 }));
56+
parameters.addAll(create(types, "boxedInteger", Integer.valueOf(1)));
57+
parameters.addAll(create(types, "boxedIntegerArray", new Integer[] { Integer.valueOf(1) }));
58+
parameters.addAll(create(types, "primitiveShort", Short.valueOf("1")));
59+
parameters.addAll(create(types, "primitiveShortArray", new short[] { 1, 2, 3 }));
60+
parameters.addAll(create(types, "boxedShort", Short.valueOf("1")));
61+
parameters.addAll(create(types, "boxedShortArray", new Short[] { Short.valueOf("1") }));
62+
parameters.addAll(create(types, "primitiveByte", Byte.valueOf("1")));
63+
parameters.addAll(create(types, "primitiveByteArray", new byte[] { 1, 2, 3 }));
64+
parameters.addAll(create(types, "boxedByte", Byte.valueOf("1")));
65+
parameters.addAll(create(types, "boxedByteArray", new Byte[] { Byte.valueOf("1") }));
66+
parameters.addAll(create(types, "primitiveChar", Character.valueOf('c')));
67+
parameters.addAll(create(types, "primitiveCharArray", new char[] { 'a', 'b', 'c' }));
68+
parameters.addAll(create(types, "boxedChar", Character.valueOf('c')));
69+
parameters.addAll(create(types, "boxedCharArray", new Character[] { Character.valueOf('c') }));
70+
parameters.addAll(create(types, "primitiveBoolean", Boolean.valueOf(true)));
71+
parameters.addAll(create(types, "primitiveBooleanArray", new boolean[] { true, false }));
72+
parameters.addAll(create(types, "boxedBoolean", Boolean.valueOf(true)));
73+
parameters.addAll(create(types, "boxedBooleanArray", new Boolean[] { Boolean.valueOf(true) }));
74+
parameters.addAll(create(types, "primitiveFloat", Float.valueOf(1f)));
75+
parameters.addAll(create(types, "primitiveFloatArray", new float[] { 1f, 2f }));
76+
parameters.addAll(create(types, "boxedFloat", Float.valueOf(1f)));
77+
parameters.addAll(create(types, "boxedFloatArray", new Float[] { Float.valueOf(1f) }));
78+
parameters.addAll(create(types, "primitiveDouble", Double.valueOf(1d)));
79+
parameters.addAll(create(types, "primitiveDoubleArray", new double[] { 1d, 2d }));
80+
parameters.addAll(create(types, "boxedDouble", Double.valueOf(1d)));
81+
parameters.addAll(create(types, "boxedDoubleArray", new Double[] { Double.valueOf(1d) }));
82+
parameters.addAll(create(types, "primitiveLong", Long.valueOf(1L)));
83+
parameters.addAll(create(types, "primitiveLongArray", new long[] { 1L, 2L }));
84+
parameters.addAll(create(types, "boxedLong", Long.valueOf(1L)));
85+
parameters.addAll(create(types, "boxedLongArray", new Long[] { Long.valueOf(1L) }));
86+
parameters.addAll(create(types, "string", "hello"));
87+
parameters.addAll(create(types, "stringArray", new String[] { "hello", "world" }));
8788

8889
return parameters;
8990
}
9091

91-
private static List<Object[]> parameters(List<Class<?>> types, String propertyName, Object value) throws Exception {
92+
private static List<Object[]> create(List<Class<?>> types, String propertyName, Object value) throws Exception {
9293

9394
List<Object[]> parameters = new ArrayList<>();
9495

‎src/test/java/org/springframework/data/mapping/model/ClassGeneratingPropertyAccessorFactoryTests.java

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@
3939
*
4040
* @author Mark Paluch
4141
*/
42+
@SuppressWarnings("WeakerAccess") // public required for class generation due to visibility rules
4243
public class ClassGeneratingPropertyAccessorFactoryTests {
4344

4445
private final static ClassGeneratingPropertyAccessorFactory factory = new ClassGeneratingPropertyAccessorFactory();
@@ -53,17 +54,17 @@ public static List<Object[]> parameters() throws ReflectiveOperationException {
5354
"privateProperty", "packageDefaultProperty", "protectedProperty", "publicProperty", "syntheticProperty",
5455
"immutable", "wither");
5556

56-
parameters.addAll(parameters(new InnerPrivateType(), propertyNames, Object.class));
57+
parameters.addAll(create(new InnerPrivateType(), propertyNames, Object.class));
5758
parameters
58-
.addAll(parameters(new InnerTypeWithPrivateAncestor(), propertyNames, InnerTypeWithPrivateAncestor.class));
59-
parameters.addAll(parameters(new InnerPackageDefaultType(), propertyNames, InnerPackageDefaultType.class));
60-
parameters.addAll(parameters(new InnerProtectedType(), propertyNames, InnerProtectedType.class));
61-
parameters.addAll(parameters(new InnerPublicType(), propertyNames, InnerPublicType.class));
62-
parameters.addAll(parameters(new ClassGeneratingPropertyAccessorPackageDefaultType(), propertyNames,
59+
.addAll(create(new InnerTypeWithPrivateAncestor(), propertyNames, InnerTypeWithPrivateAncestor.class));
60+
parameters.addAll(create(new InnerPackageDefaultType(), propertyNames, InnerPackageDefaultType.class));
61+
parameters.addAll(create(new InnerProtectedType(), propertyNames, InnerProtectedType.class));
62+
parameters.addAll(create(new InnerPublicType(), propertyNames, InnerPublicType.class));
63+
parameters.addAll(create(new ClassGeneratingPropertyAccessorPackageDefaultType(), propertyNames,
6364
ClassGeneratingPropertyAccessorPackageDefaultType.class));
64-
parameters.addAll(parameters(new ClassGeneratingPropertyAccessorPublicType(), propertyNames,
65+
parameters.addAll(create(new ClassGeneratingPropertyAccessorPublicType(), propertyNames,
6566
ClassGeneratingPropertyAccessorPublicType.class));
66-
parameters.addAll(parameters(new SubtypeOfTypeInOtherPackage(), propertyNames, SubtypeOfTypeInOtherPackage.class));
67+
parameters.addAll(create(new SubtypeOfTypeInOtherPackage(), propertyNames, SubtypeOfTypeInOtherPackage.class));
6768

6869
Class<Object> defaultPackageClass = (Class) Class.forName("TypeInDefaultPackage");
6970

@@ -73,7 +74,7 @@ public static List<Object[]> parameters() throws ReflectiveOperationException {
7374
return parameters;
7475
}
7576

76-
private static List<Object[]> parameters(Object bean, List<String> propertyNames, Class<?> expectedConstructorType) {
77+
private static List<Object[]> create(Object bean, List<String> propertyNames, Class<?> expectedConstructorType) {
7778

7879
List<Object[]> parameters = new ArrayList<>();
7980

0 commit comments

Comments
 (0)