OQL test - do not rely on toString() implementation of Map

Author: thurka

visualvm/libs.profiler/profiler.oql/test/unit/src/org/graalvm/visualvm/lib/profiler/oql/engine/api/impl/OQLEngineTest.java

@@ -683,11 +683,11 @@ public void testComplexStatement2() throws Exception {
 
             public boolean visit(Object o) {
                 System.out.println(o);
-                rslt[0] = o.toString();
+                rslt[0] = dump(o);
                 return true;
             }
         });
-        assertEquals("{value=, key=sun.cpu.isalist}", rslt[0]);
+        assertEquals("{key=sun.cpu.isalist, value=}", rslt[0]);
     }
 
     @Test
@@ -745,4 +745,17 @@ public boolean visit(Object o) {
             }
         });
     }
+
+    private String dump(Object o) {
+        if (o instanceof Map) {
+            List<String> els = new ArrayList<>();
+            Map<?,?> m = (Map) o;
+            for (Map.Entry e : m.entrySet()) {
+                els.add(e.getKey()+"="+e.getValue());
+            }
+            els.sort(null);
+            return "{"+String.join(", ", els)+"}";
+        }
+        return o.toString();
+    }
 }