no need to check the snapshot in multithread test; it's only a sanity test

jbachorik · jbachorik · commit de98d00a5826 · 2014-11-30T21:09:47.000+01:00
diff --git a/src/test/com/sun/btrace/profiling/MethodInvocationProfilerTest.java b/src/test/com/sun/btrace/profiling/MethodInvocationProfilerTest.java
@@ -425,11 +425,9 @@ public void run() {
             Thread.sleep(13);
         }
 
-        p.recordEntry("r1");
-        p.recordExit("r1", 10);
+        finished.set(true);
 
-        Snapshot s = p.snapshot();
-        assertArrayEquals(expected, s.total);
+        t.join();
     }
 
     @Test

Original file line number	Diff line number	Diff line change
`@@ -425,11 +425,9 @@ public void run() {`
`425`	`425`	`Thread.sleep(13);`
`426`	`426`	`}`
`427`	`427`
`428`		`- p.recordEntry("r1");`
`429`		`- p.recordExit("r1", 10);`
	`428`	`+ finished.set(true);`
`430`	`429`
`431`		`- Snapshot s = p.snapshot();`
`432`		`- assertArrayEquals(expected, s.total);`
	`430`	`+ t.join();`
`433`	`431`	`}`
`434`	`432`
`435`	`433`	`@Test`