Fix flaky test (apache#2643)

in org.apache.helix.rest.server. TestResourceAccessor#testGetResources
hofi1 · Oct 13, 2023 · e1050de · e1050de
1 parent 3216a09
commit e1050de
Showing 1 changed file with 11 additions and 5 deletions.
diff --git a/helix-rest/src/test/java/org/apache/helix/rest/server/TestResourceAccessor.java b/helix-rest/src/test/java/org/apache/helix/rest/server/TestResourceAccessor.java
@@ -75,11 +75,17 @@ public void testGetResources() throws IOException {
 
     Set<String> resources = OBJECT_MAPPER.readValue(idealStates,
         OBJECT_MAPPER.getTypeFactory().constructCollectionType(Set.class, String.class));
-    Assert.assertEquals(resources.size(), _resourcesMap.get("TestCluster_0").size(), "Sizes are not equal");
-    Assert.assertTrue(resources.containsAll(_resourcesMap.get("TestCluster_0")), "resources does not contain all "
-    + "elements from _resourcesMap");
-    Assert.assertTrue(_resourcesMap.get("TestCluster_0").containsAll(resources), "_resourcesMap does not contain all "
-        + "elements from resources");
+    Assert.assertTrue(resources.size() == _resourcesMap.get("TestCluster_0").size()
+        && resources.containsAll(_resourcesMap.get("TestCluster_0"))
+        && _resourcesMap.get("TestCluster_0").containsAll(resources),
+        "Sets are not equal. _resourcesMap.size(): "
+            + _resourcesMap.size() + "; "
+            + "_resourcesMap.get(TestCluster_0).size(): "
+            + _resourcesMap.get("TestCluster_0").size() + "; "
+            + "resources.containsAll(_resourcesMap.get(TestCluster_0)): "
+            + resources.containsAll(_resourcesMap.get("TestCluster_0")) + "; "
+            + "_resourcesMap.get(TestCluster_0).containsAll(resources): "
+            + _resourcesMap.get("TestCluster_0").containsAll(resources));
     System.out.println("End test :" + TestHelper.getTestMethodName());
   }