use awaitility library when possible
Signed-off-by: Olivier Lamy <olamy@apache.org>
diff --git a/pom.xml b/pom.xml
index 776d7de..0296a1b 100644
--- a/pom.xml
+++ b/pom.xml
@@ -271,7 +271,11 @@
<artifactId>swagger-annotations</artifactId>
<version>${io.swagger.version}</version>
</dependency>
-
+ <dependency>
+ <groupId>org.awaitility</groupId>
+ <artifactId>awaitility</artifactId>
+ <version>4.2.0</version>
+ </dependency>
</dependencies>
</dependencyManagement>
<dependencies>
@@ -282,6 +286,12 @@
<scope>test</scope>
</dependency>
+ <dependency>
+ <groupId>org.awaitility</groupId>
+ <artifactId>awaitility</artifactId>
+ <scope>test</scope>
+ </dependency>
+
<!-- JUNIT 5 -->
<dependency>
<groupId>org.junit.platform</groupId>
@@ -447,15 +457,15 @@
<outputEncoding>UTF-8</outputEncoding>
<skipDeploy>true</skipDeploy>
<stagingDirectory>${site.staging.base}/target/staging${scmPublishPath}</stagingDirectory>
- <asciidoc>
- <!-- optional site-wide AsciiDoc attributes -->
- <attributes>
- <icons>font</icons>
- <source-highlighter>coderay</source-highlighter>
- <coderay-css>style</coderay-css>
- <toclevels>2</toclevels>
- </attributes>
- </asciidoc>
+<!-- <asciidoc>-->
+<!-- <!– optional site-wide AsciiDoc attributes –>-->
+<!-- <attributes>-->
+<!-- <icons>font</icons>-->
+<!-- <source-highlighter>coderay</source-highlighter>-->
+<!-- <coderay-css>style</coderay-css>-->
+<!-- <toclevels>2</toclevels>-->
+<!-- </attributes>-->
+<!-- </asciidoc>-->
</configuration>
<dependencies>
<dependency>
diff --git a/spring-cache/spring-cache-test/src/main/java/org/apache/archiva/components/cache/test/AbstractCacheTestCase.java b/spring-cache/spring-cache-test/src/main/java/org/apache/archiva/components/cache/test/AbstractCacheTestCase.java
index f096897..86ed50c 100644
--- a/spring-cache/spring-cache-test/src/main/java/org/apache/archiva/components/cache/test/AbstractCacheTestCase.java
+++ b/spring-cache/spring-cache-test/src/main/java/org/apache/archiva/components/cache/test/AbstractCacheTestCase.java
@@ -47,14 +47,6 @@
public abstract class AbstractCacheTestCase
extends TestCase
{
-// static
-// {
-// Logger logger = Logger.getLogger( "org.codehaus.plexus.cache" );
-// logger.setLevel( Level.ALL );
-// ConsoleHandler handler = new ConsoleHandler( );
-// handler.setLevel( Level.ALL );
-// logger.addHandler( handler );
-// }
private static final Logger log = LoggerFactory.getLogger( AbstractCacheTestCase.class );
diff --git a/spring-quartz/src/test/java/org/apache/archiva/components/scheduler/SchedulerTest.java b/spring-quartz/src/test/java/org/apache/archiva/components/scheduler/SchedulerTest.java
index 1ea8ee2..63c2b20 100644
--- a/spring-quartz/src/test/java/org/apache/archiva/components/scheduler/SchedulerTest.java
+++ b/spring-quartz/src/test/java/org/apache/archiva/components/scheduler/SchedulerTest.java
@@ -20,6 +20,7 @@
*/
import junit.framework.TestCase;
+import org.awaitility.Awaitility;
import org.junit.After;
import org.junit.Test;
import org.junit.runner.RunWith;
@@ -37,6 +38,7 @@
import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
import javax.inject.Inject;
+import java.util.concurrent.TimeUnit;
@RunWith ( SpringJUnit4ClassRunner.class )
@ContextConfiguration ( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } )
@@ -81,10 +83,8 @@
scheduler.scheduleJob( jobDetail, trigger );
- while ( !triggerFired )
- {
- Thread.sleep( 10 );
- }
+ Awaitility.await().atLeast( 100, TimeUnit.MILLISECONDS ).until( () -> triggerFired );
+
logger.info( "ok triggerFired" );
}
diff --git a/spring-taskqueue/src/test/java/org/apache/archiva/components/taskqueue/execution/TaskQueueExecutorTest.java b/spring-taskqueue/src/test/java/org/apache/archiva/components/taskqueue/execution/TaskQueueExecutorTest.java
index 3eded88..8ea4b12 100644
--- a/spring-taskqueue/src/test/java/org/apache/archiva/components/taskqueue/execution/TaskQueueExecutorTest.java
+++ b/spring-taskqueue/src/test/java/org/apache/archiva/components/taskqueue/execution/TaskQueueExecutorTest.java
@@ -23,6 +23,7 @@
import org.apache.archiva.components.taskqueue.BuildProjectTask;
import org.apache.archiva.components.taskqueue.TaskQueue;
import org.apache.archiva.components.taskqueue.TaskQueueException;
+import org.awaitility.Awaitility;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.springframework.test.context.ContextConfiguration;
@@ -30,6 +31,7 @@
import javax.inject.Inject;
import javax.inject.Named;
+import java.util.concurrent.TimeUnit;
/**
* @author <a href="mailto:kenney@apache.org">Kenney Westerhof</a>
@@ -90,16 +92,7 @@
{
// thread scheduling may take some time, so we want to wait until the task
// is actually running before starting to count the timeout.
- for ( int i = 0; i < 500; i++ )
- {
- if ( task.wasStarted( ) )
- {
- break;
- }
- Thread.sleep( 10 );
- }
-
- assertTrue( "Task not started in 5 seconds - heavy load?", task.isStarted( ) );
+ Awaitility.await().atMost( 5, TimeUnit.SECONDS ).until( () -> task.isStarted() );
Thread.sleep( task.getMaxExecutionTime( ) );
}