svn commit: r1577136 - /jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/stats/ClockTest.java

[reschke]

Author: reschke
Date: Thu Mar 13 12:34:13 2014
New Revision: 1577136

URL: http://svn.apache.org/r1577136
Log:
OAK-1535: improve test to report actual drift on failure

Modified:
    
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/stats/ClockTest.java

Modified: 
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/stats/ClockTest.java
URL: 
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/stats/ClockTest.java?rev=1577136&r1=1577135&r2=1577136&view=diff
==============================================================================
--- 
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/stats/ClockTest.java
 (original)
+++ 
jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/stats/ClockTest.java
 Thu Mar 13 12:34:13 2014
@@ -29,6 +29,7 @@ public class ClockTest {
     public void testClockDrift() throws InterruptedException {
         ScheduledExecutorService executor =
                 Executors.newSingleThreadScheduledExecutor();
+        final long limit = 20;
         try {
             Clock[] clocks = new Clock[] {
                     Clock.SIMPLE,
@@ -38,14 +39,14 @@ public class ClockTest {
 
             for (Clock clock : clocks) {
                 long drift = clock.getTime() - System.currentTimeMillis();
-                assertTrue(Math.abs(drift) < 20); // Windows can have 15ms gaps
+                assertTrue("unexpected drift: " + Math.abs(drift) + " (limit " 
+ limit +")", Math.abs(drift) < limit); // Windows can have 15ms gaps
             }
 
             Thread.sleep(100);
 
             for (Clock clock : clocks) {
                 long drift = clock.getTime() - System.currentTimeMillis();
-                assertTrue(Math.abs(drift) < 20);
+                assertTrue("unexpected drift: " + Math.abs(drift) + " (limit " 
+ limit +")", Math.abs(drift) < limit);
             }
         } finally {
             executor.shutdown();