Add unit test showing that we are not there yet.

1 files changed, 14 insertions, 2 deletions

diff --git a/metrics-proxy/src/test/java/ai/vespa/metricsproxy/service/MetricsParserTest.java b/metrics-proxy/src/test/java/ai/vespa/metricsproxy/service/MetricsParserTest.java
index 659305a8543..5fc260e9912 100644
--- a/metrics-proxy/src/test/java/ai/vespa/metricsproxy/service/MetricsParserTest.java
+++ b/metrics-proxy/src/test/java/ai/vespa/metricsproxy/service/MetricsParserTest.java
@@ -34,12 +34,24 @@ public class MetricsParserTest {
                 new MetricsParser.Dimension("cluster", "CLUSTER-2"),
                 new MetricsParser.Dimension("clusterid", "CLUSTER-2"));
 
-        System.out.println(dimensionsHashCode(dimensions1));
-        System.out.println(dimensionsHashCode(dimensions2));
         assertNotEquals(dimensionsHashCode(dimensions1), dimensionsHashCode(dimensions2));
     }
 
     @Test
+    public void dimensions_hashcode_is_different_for_distinct_with_swapped_dimension_values() {
+        var dimensions1 = List.of(
+                new MetricsParser.Dimension("cluster", "CLUSTER-1"),
+                new MetricsParser.Dimension("clusterid", "CLUSTER-2"));
+
+        var dimensions2 = List.of(
+                new MetricsParser.Dimension("cluster", "CLUSTER-2"),
+                new MetricsParser.Dimension("clusterid", "CLUSTER-1"));
+
+        assertNotEquals(dimensionsHashCode(dimensions1), dimensionsHashCode(dimensions2));
+    }
+
+
+    @Test
     public void different_dimension_values_are_not_treated_as_equal() throws Exception {
         var collector = new MetricsCollector();
         MetricsParser.parse(metricsJson(), collector);