Skip to content

Commit 9a62d72

Browse files
authored
Fix flaky test caused by .toArray() (#403)
1 parent a4c3c85 commit 9a62d72

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

apm-commons/apm-util/src/test/java/org/apache/skywalking/apm/util/ConfigInitializerTest.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
import org.junit.Assert;
2222
import org.junit.Before;
2323
import org.junit.Test;
24+
import org.mockito.internal.util.collections.Sets;
2425

2526
import java.util.Arrays;
2627
import java.util.HashMap;
@@ -67,8 +68,8 @@ public void testInitialize() throws IllegalAccessException {
6768
Assert.assertArrayEquals(new Integer[]{1, 2, 3, 4}, TestPropertiesObject.Level1Object.LIST_INT_ATTR_ED2.toArray());
6869
Assert.assertArrayEquals(new String[]{}, TestPropertiesObject.Level1Object.SET_STR_EMPTY_ATTR.toArray());
6970
Assert.assertArrayEquals(new Boolean[]{true, false}, TestPropertiesObject.Level1Object.LIST_BOOL_ATTR.toArray());
70-
Assert.assertArrayEquals(new String[]{"a", "b", "c", "d"}, TestPropertiesObject.Level1Object.SET_STR_ATTR.toArray());
71-
Assert.assertArrayEquals(new Integer[]{1, 2, 3, 4}, TestPropertiesObject.Level1Object.SET_INT_ATTR.toArray());
71+
Assert.assertEquals(Sets.newSet("a", "b", "c", "d"), TestPropertiesObject.Level1Object.SET_STR_ATTR);
72+
Assert.assertEquals(Sets.newSet(1, 2, 3, 4), TestPropertiesObject.Level1Object.SET_INT_ATTR);
7273
Assert.assertArrayEquals(new Boolean[]{true}, TestPropertiesObject.Level1Object.SET_BOOL_ATTR.toArray());
7374
Assert.assertEquals(TestColorEnum.RED, TestPropertiesObject.Level1Object.Level2Object.ENUM_ATTR);
7475
//make sure that when descs is empty,toString() work right;

0 commit comments

Comments
 (0)