Author: britter
Date: Wed Jul 16 16:06:31 2014
New Revision: 1611065

URL: http://svn.apache.org/r1611065
Log:
Make test more robust by giving the executor more time to submit all insert 
statements

Modified:
    
commons/proper/dbutils/trunk/src/test/java/org/apache/commons/dbutils/AsyncQueryRunnerTest.java

Modified: 
commons/proper/dbutils/trunk/src/test/java/org/apache/commons/dbutils/AsyncQueryRunnerTest.java
URL: 
http://svn.apache.org/viewvc/commons/proper/dbutils/trunk/src/test/java/org/apache/commons/dbutils/AsyncQueryRunnerTest.java?rev=1611065&r1=1611064&r2=1611065&view=diff
==============================================================================
--- 
commons/proper/dbutils/trunk/src/test/java/org/apache/commons/dbutils/AsyncQueryRunnerTest.java
 (original)
+++ 
commons/proper/dbutils/trunk/src/test/java/org/apache/commons/dbutils/AsyncQueryRunnerTest.java
 Wed Jul 16 16:06:31 2014
@@ -32,6 +32,7 @@ import java.sql.SQLException;
 import java.util.concurrent.ExecutionException;
 import java.util.concurrent.Executors;
 import java.util.concurrent.Future;
+import java.util.concurrent.TimeUnit;
 
 import javax.sql.DataSource;
 
@@ -450,6 +451,9 @@ public class AsyncQueryRunnerTest {
         runner.insert(conn, "3", handler);
         runner.insert(conn, "4", handler, "param1");
 
+        // give the Executor time to submit all insert statements. Otherwise 
the following verify statements will fail from time to time.
+        TimeUnit.MILLISECONDS.sleep(50);
+
         verify(mockQueryRunner).insert("1", handler);
         verify(mockQueryRunner).insert("2", handler, "param1");
         verify(mockQueryRunner).insert(conn, "3", handler);


Reply via email to