From 4b11c764e5bfa1fd2e3b85247c0384c66b515489 Mon Sep 17 00:00:00 2001 From: contextshuffling Date: Sun, 19 Jan 2020 23:00:06 +0800 Subject: [PATCH] [LANG-1514] make test more stable by using HashSet to compare --- .../java/org/apache/commons/lang3/reflect/FieldUtilsTest.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/test/java/org/apache/commons/lang3/reflect/FieldUtilsTest.java b/src/test/java/org/apache/commons/lang3/reflect/FieldUtilsTest.java index 4e428a926..291e70a20 100644 --- a/src/test/java/org/apache/commons/lang3/reflect/FieldUtilsTest.java +++ b/src/test/java/org/apache/commons/lang3/reflect/FieldUtilsTest.java @@ -36,6 +36,7 @@ import java.lang.reflect.Modifier; import java.util.ArrayList; import java.util.Arrays; +import java.util.HashSet; import java.util.List; import static org.junit.jupiter.api.Assertions.assertArrayEquals; @@ -188,7 +189,7 @@ public void testGetAllFieldsList() { final List fieldsInteger = Arrays.asList(Integer.class.getDeclaredFields()); final List allFieldsInteger = new ArrayList<>(fieldsInteger); allFieldsInteger.addAll(fieldsNumber); - assertEquals(allFieldsInteger, FieldUtils.getAllFieldsList(Integer.class)); + assertEquals(new HashSet(allFieldsInteger), new HashSet(FieldUtils.getAllFieldsList(Integer.class))); final List allFields = FieldUtils.getAllFieldsList(PublicChild.class); // Under Jacoco,0.8.1 and Java 10, the field count is 7. int expected = 5;