change TestMeasurementGroupConfig.addGroup to deterministic implementation (#4994)

anthonyx24 · web-flow · commit e56086d1ec33 · 2025-10-27T11:40:44.000+08:00
diff --git a/foundations/foundation-metrics/src/test/java/org/apache/servicecomb/foundation/metrics/publish/spectator/TestMeasurementGroupConfig.java b/foundations/foundation-metrics/src/test/java/org/apache/servicecomb/foundation/metrics/publish/spectator/TestMeasurementGroupConfig.java
@@ -53,7 +53,7 @@ public void addGroup() {
     config.addGroup("id1", "tag1.1", "tag1.2");
     config.addGroup("id2", "tag2.1", "tag2.2");
 
-    MatcherAssert.assertThat(groups.keySet(), Matchers.contains("id2", "id1"));
+    MatcherAssert.assertThat(groups.keySet(), Matchers.containsInAnyOrder("id2", "id1"));
     MatcherAssert.assertThat(groups.get("id1").stream().map(TagFinder::getTagKey).toArray(), Matchers.arrayContaining("tag1.1", "tag1.2"));
     MatcherAssert.assertThat(groups.get("id2").stream().map(TagFinder::getTagKey).toArray(), Matchers.arrayContaining("tag2.1", "tag2.2"));
   }

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ public void addGroup() {`
`53`	`53`	`config.addGroup("id1", "tag1.1", "tag1.2");`
`54`	`54`	`config.addGroup("id2", "tag2.1", "tag2.2");`
`55`	`55`
`56`		`- MatcherAssert.assertThat(groups.keySet(), Matchers.contains("id2", "id1"));`
	`56`	`+ MatcherAssert.assertThat(groups.keySet(), Matchers.containsInAnyOrder("id2", "id1"));`
`57`	`57`	`MatcherAssert.assertThat(groups.get("id1").stream().map(TagFinder::getTagKey).toArray(), Matchers.arrayContaining("tag1.1", "tag1.2"));`
`58`	`58`	`MatcherAssert.assertThat(groups.get("id2").stream().map(TagFinder::getTagKey).toArray(), Matchers.arrayContaining("tag2.1", "tag2.2"));`
`59`	`59`	`}`