From 2d0cbb8be9e6726168d4bee6b3ab93e950874d5b Mon Sep 17 00:00:00 2001 From: zhou124 Date: Sun, 1 Sep 2024 18:19:25 -0500 Subject: [PATCH] Fix flaky test through set comparison --- .../internal/managed/DisposableSupplierTest.java | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/incubator/cdi-inject-weld/src/test/java/org/glassfish/jersey/inject/weld/internal/managed/DisposableSupplierTest.java b/incubator/cdi-inject-weld/src/test/java/org/glassfish/jersey/inject/weld/internal/managed/DisposableSupplierTest.java index 478d49307c9..3d5786c546e 100644 --- a/incubator/cdi-inject-weld/src/test/java/org/glassfish/jersey/inject/weld/internal/managed/DisposableSupplierTest.java +++ b/incubator/cdi-inject-weld/src/test/java/org/glassfish/jersey/inject/weld/internal/managed/DisposableSupplierTest.java @@ -18,6 +18,8 @@ import java.lang.reflect.Type; import java.util.Objects; +import java.util.HashSet; +import java.util.Set; import java.util.concurrent.atomic.AtomicBoolean; import java.util.concurrent.atomic.AtomicInteger; import java.util.concurrent.atomic.AtomicReference; @@ -368,9 +370,17 @@ public void testDisposeComposedObjectWithPerLookupFields() { // All instances should be the same because they are request scoped. ComposedObject instance = injectionManager.getInstance(ComposedObject.class); - assertEquals("1", instance.getFirst().toString()); - assertEquals("2", instance.getSecond().toString()); - assertEquals("3", instance.getThird().toString()); + Set set1 = new HashSet() {{ + add("1"); + add("2"); + add("3"); + }}; + Set set2 = new HashSet() {{ + add(instance.getFirst().toString()); + add(instance.getSecond().toString()); + add(instance.getThird().toString()); + }}; + assertEquals(set1, set2); }); Supplier cleanedSupplier = atomicSupplier.get();