Mercurial > hg > truffle

changeset 22535:9b9301abe3ff

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
@Ignored broken test and corrected order of assertEquals arguments (GRAAL-1385)
author Doug Simon <doug.simon@oracle.com>
date Fri, 08 Jan 2016 21:43:03 +0100
parents 1133c68332ef
children 21f48bc8a535
files truffle/com.oracle.truffle.tools.test/src/com/oracle/truffle/tools/test/NodeExecCounterTest.java
diffstat 1 files changed, 15 insertions(+), 13 deletions(-) [+]
line wrap: on
line diff
--- a/truffle/com.oracle.truffle.tools.test/src/com/oracle/truffle/tools/test/NodeExecCounterTest.java	Fri Jan 08 11:51:54 2016 +0100
+++ b/truffle/com.oracle.truffle.tools.test/src/com/oracle/truffle/tools/test/NodeExecCounterTest.java	Fri Jan 08 21:43:03 2016 +0100
@@ -30,6 +30,7 @@
 import java.io.IOException;
 import java.lang.reflect.Field;
 
+import org.junit.Ignore;
 import org.junit.Test;
 
 import com.oracle.truffle.api.instrument.Instrumenter;
@@ -47,33 +48,34 @@
         field.setAccessible(true);
         final Instrumenter instrumenter = (Instrumenter) field.get(vm);
         final NodeExecCounter tool = new NodeExecCounter();
-        assertEquals(tool.getCounts().length, 0);
+        assertEquals(0, tool.getCounts().length);
         instrumenter.install(tool);
-        assertEquals(tool.getCounts().length, 0);
+        assertEquals(0, tool.getCounts().length);
         tool.setEnabled(false);
-        assertEquals(tool.getCounts().length, 0);
+        assertEquals(0, tool.getCounts().length);
         tool.setEnabled(true);
-        assertEquals(tool.getCounts().length, 0);
+        assertEquals(0, tool.getCounts().length);
         tool.reset();
-        assertEquals(tool.getCounts().length, 0);
+        assertEquals(0, tool.getCounts().length);
         tool.dispose();
-        assertEquals(tool.getCounts().length, 0);
+        assertEquals(0, tool.getCounts().length);
     }
 
     void checkCounts(NodeExecCounter execTool, int addCount, int valueCount) {
         NodeExecutionCount[] counts = execTool.getCounts();
-        assertEquals(counts.length, 2);
+        assertEquals(2, counts.length);
         for (NodeExecutionCount counter : counts) {
             if (counter.nodeClass() == ToolTestUtil.TestAdditionNode.class) {
-                assertEquals(counter.executionCount(), addCount);
+                assertEquals(addCount, counter.executionCount());
             } else if (counter.nodeClass() == ToolTestUtil.TestValueNode.class) {
-                assertEquals(counter.executionCount(), valueCount);
+                assertEquals(valueCount, counter.executionCount());
             } else {
                 fail("correct classes counted");
             }
         }
     }
 
+    @Ignore("GRAAL-1385")
     @Test
     public void testCounting() throws NoSuchFieldException, SecurityException, IllegalArgumentException, IllegalAccessException, IOException {
         final PolyglotEngine vm = PolyglotEngine.newBuilder().build();
@@ -84,19 +86,19 @@
         final NodeExecCounter execTool = new NodeExecCounter();
         instrumenter.install(execTool);
 
-        assertEquals(execTool.getCounts().length, 0);
+        assertEquals(0, execTool.getCounts().length);
 
-        assertEquals(vm.eval(source).get(), 13);
+        assertEquals(13, vm.eval(source).get());
 
         checkCounts(execTool, 1, 2);
 
         for (int i = 0; i < 99; i++) {
-            assertEquals(vm.eval(source).get(), 13);
+            assertEquals(13, vm.eval(source).get());
         }
         checkCounts(execTool, 100, 200);
 
         execTool.setEnabled(false);
-        assertEquals(vm.eval(source).get(), 13);
+        assertEquals(13, vm.eval(source).get());
         checkCounts(execTool, 100, 200);
 
         execTool.dispose();