Deflake PipelineApiTest#parameterizedFreestyleTest (#2666)

Author: gbhat618

blueocean-rest-impl/pom.xml

@@ -129,6 +129,12 @@
             <artifactId>powermock-reflect</artifactId>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>org.awaitility</groupId>
+            <artifactId>awaitility</artifactId>
+            <version>4.3.0</version>
+            <scope>test</scope>
+        </dependency>
     </dependencies>
 
     <build>

blueocean-rest-impl/src/test/java/io/jenkins/blueocean/service/embedded/PipelineApiTest.java

@@ -63,6 +63,7 @@
 import java.util.concurrent.ExecutionException;
 import java.util.concurrent.TimeUnit;
 
+import static org.awaitility.Awaitility.await;
 import static org.junit.Assert.*;
 
 /**
@@ -781,10 +782,14 @@ public void parameterizedFreestyleTest() throws Exception {
                         Collections.singletonList(MapsHelper.of("name", "version", "value", "2.0"))
                 ), 200);
         assertEquals("pp", resp.get("pipeline"));
-        Thread.sleep(1000);
-        resp = get("/organizations/jenkins/pipelines/pp/runs/1/");
-        assertEquals("SUCCESS", resp.get("result"));
-        assertEquals("FINISHED", resp.get("state"));
+        await().until(() -> {
+            var build = p.getLastBuild();
+            return build != null && !build.isBuilding() && build.getResult() == Result.SUCCESS;
+        });
+        await().until(() -> {
+            Map runResp = get("/organizations/jenkins/pipelines/pp/runs/1/");
+            return "SUCCESS".equals(runResp.get("result")) && "FINISHED".equals(runResp.get("state"));
+        });
     }
 
     public static class TestStringParameterDefinition extends StringParameterDefinition{