Tests failed randomly 63/68363/1
authorromaingimbert <romain.gimbert@orange.com>
Fri, 21 Sep 2018 08:31:24 +0000 (10:31 +0200)
committerromaingimbert <romain.gimbert@orange.com>
Fri, 21 Sep 2018 08:31:24 +0000 (10:31 +0200)
-remove thread.sleep in tests

Change-Id: Ia7eb1a44725915648d3aa77bfcd93e42a93310d8
Issue-ID: EXTAPI-153
Signed-off-by: romaingimbert <romain.gimbert@orange.com>
src/test/java/org/onap/nbi/apis/ApiTest.java

index 3ac2f7e..696aa57 100644 (file)
@@ -21,6 +21,7 @@ import com.github.tomakehurst.wiremock.http.ResponseDefinition;
 import com.github.tomakehurst.wiremock.stubbing.ListStubMappingsResult;
 import com.github.tomakehurst.wiremock.stubbing.StubMapping;
 import java.util.ArrayList;
+import java.util.Calendar;
 import java.util.Date;
 import java.util.LinkedHashMap;
 import java.util.List;
@@ -1002,8 +1003,10 @@ public class ApiTest {
             }
         }
         executionTaskA = getExecutionTask("A");
-        assertThat(executionTaskA.getLastAttemptDate().getTime()>executionTaskA.getCreateDate().getTime()).isTrue();
-        Thread.sleep((long)(pollingDurationInMins*60000 + 1));
+        Date createDate = executionTaskA.getCreateDate();
+        assertThat(executionTaskA.getLastAttemptDate().getTime()> createDate.getTime()).isTrue();
+
+        changeCreationDate(executionTaskA);
         SoTaskProcessor.processOrderItem(executionTaskA);
 
         serviceOrderChecked = serviceOrderRepository.findOne("test");
@@ -1018,6 +1021,13 @@ public class ApiTest {
 
     }
 
+    private void changeCreationDate(ExecutionTask executionTaskA) {
+        Calendar cal = Calendar.getInstance();
+        cal.setTime(executionTaskA.getCreateDate());
+        cal.add(Calendar.SECOND, -30);
+        executionTaskA.setCreateDate(cal.getTime());
+    }
+
 
     @Test
     public void testE2EExecutionTaskFailed() throws Exception {
@@ -1040,7 +1050,7 @@ public class ApiTest {
         }
         executionTaskA = getExecutionTask("A");
         assertThat(executionTaskA.getLastAttemptDate().getTime()>executionTaskA.getCreateDate().getTime()).isTrue();
-        Thread.sleep((long)(pollingDurationInMins*60000 + 1));
+        changeCreationDate(executionTaskA);
         SoTaskProcessor.processOrderItem(executionTaskA);
 
         serviceOrderChecked = serviceOrderRepository.findOne("test");
@@ -1227,7 +1237,7 @@ public class ApiTest {
         }
         executionTaskA = getExecutionTask("A");
         assertThat(executionTaskA.getLastAttemptDate().getTime()>executionTaskA.getCreateDate().getTime()).isTrue();
-        Thread.sleep((long)(pollingDurationInMins*60000 + 1));
+        changeCreationDate(executionTaskA);
         SoTaskProcessor.processOrderItem(executionTaskA);
 
         serviceOrderChecked = serviceOrderRepository.findOne("test");