Skip to content

Commit 2193b2b

Browse files
committed
Fixed flaky test in DisposableSupplierText.java with set comparison
1 parent ff5f5a8 commit 2193b2b

File tree

1 file changed

+13
-3
lines changed

1 file changed

+13
-3
lines changed

inject/cdi2-se/src/test/java/org/glassfish/jersey/inject/cdi/se/DisposableSupplierTest.java

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,8 @@
1717
package org.glassfish.jersey.inject.cdi.se;
1818

1919
import java.lang.reflect.Type;
20+
import java.util.HashSet;
21+
import java.util.Set;
2022
import java.util.concurrent.atomic.AtomicInteger;
2123
import java.util.concurrent.atomic.AtomicReference;
2224
import java.util.function.Supplier;
@@ -368,9 +370,17 @@ public void testDisposeComposedObjectWithPerLookupFields() {
368370

369371
// All instances should be the same because they are request scoped.
370372
ComposedObject instance = injectionManager.getInstance(ComposedObject.class);
371-
assertEquals("1", instance.getFirst());
372-
assertEquals("2", instance.getSecond());
373-
assertEquals("3", instance.getThird());
373+
Set<String> set1 = new HashSet<String>() {{
374+
add("1");
375+
add("2");
376+
add("3");
377+
}};
378+
Set<String> set2 = new HashSet<String>() {{
379+
add(instance.getFirst().toString());
380+
add(instance.getSecond().toString());
381+
add(instance.getThird().toString());
382+
}};
383+
assertEquals(set1, set2);
374384
});
375385

376386
Supplier<String> cleanedSupplier = atomicSupplier.get();

0 commit comments

Comments
 (0)