test-utils-stress: sorted StressResult output

Signed-off-by: Ketoth Xupack <ketoth.xupack@gmail.com>
no-hope · Mar 17, 2014 · 76ff71e · 76ff71e
1 parent 6e0091e
commit 76ff71e
Show file tree

Hide file tree

Showing 2 changed files with 46 additions and 2 deletions.
diff --git a/projects/test-utils/test-utils-stress/src/main/java/org/nohope/test/stress/StressResult.java b/projects/test-utils/test-utils-stress/src/main/java/org/nohope/test/stress/StressResult.java
@@ -4,6 +4,8 @@
 
 import java.util.HashMap;
 import java.util.Map;
+import java.util.Set;
+import java.util.TreeSet;
 
 import static java.util.concurrent.TimeUnit.SECONDS;
 import static org.nohope.test.stress.TimeUtils.throughputTo;
@@ -74,9 +76,13 @@ public String toString() {
                .append('\n')
                .append(separator)
                .append('\n');
-        for (final Result stats : results.values()) {
-            builder.append(stats.toString());
+
+        // sorted output
+        final Set<String> keys = new TreeSet<>(results.keySet());
+        for (final String key : keys) {
+            builder.append(results.get(key).toString());
         }
+
         return builder.append(separator)
                       .append('\n')
                       .append(pad("Total error count:"))

diff --git a/...test-utils/test-utils-stress/src/test/java/org/nohope/test/stress/StressScenarioTest.java b/...test-utils/test-utils-stress/src/test/java/org/nohope/test/stress/StressScenarioTest.java
@@ -323,4 +323,42 @@ public Long get() throws Exception {
         }
     }
 
+
+    @Test
+    public void sortedOutput() throws InterruptedException {
+        final StressResult result =
+                StressScenario.of(TimerResolution.MILLISECONDS).measure(10, 1, new Action() {
+                    @Override
+                    protected void doAction(final MeasureProvider p) throws Exception {
+                        p.invoke("action4", new Invoke() {
+                            @Override
+                            public void invoke() throws Exception {
+                            }
+                        });
+                        p.invoke("action1", new Invoke() {
+                            @Override
+                            public void invoke() throws Exception {
+                            }
+                        });
+                        p.invoke("action2", new Invoke() {
+                            @Override
+                            public void invoke() throws Exception {
+                            }
+                        });
+                        p.invoke("action3", new Invoke() {
+                            @Override
+                            public void invoke() throws Exception {
+                            }
+                        });
+                    }
+                });
+
+        final int action1 = result.toString().indexOf("action1");
+        final int action2 = result.toString().indexOf("action2");
+        final int action3 = result.toString().indexOf("action3");
+        final int action4 = result.toString().indexOf("action4");
+        assertTrue(action4 > action3);
+        assertTrue(action3 > action2);
+        assertTrue(action2 > action1);
+    }
 }