In some corner cases we have seen TestRunStatistics from AzureEngine to be null, this fix waits a little longer in such scenarios to avoid NPE

rabelenda-abstracta · rabelenda-abstracta · commit 2b8429eff752 · 2024-03-13T15:48:50.000-03:00
diff --git a/jmeter-java-dsl-azure/src/main/java/us/abstracta/jmeter/javadsl/azure/AzureEngine.java b/jmeter-java-dsl-azure/src/main/java/us/abstracta/jmeter/javadsl/azure/AzureEngine.java
@@ -532,7 +532,8 @@ private TestRun awaitTestEnd(TestRun testRun)
 
   private TestRun awaitVirtualUsers(TestRun testRun) throws InterruptedException, IOException {
     Instant start = Instant.now();
-    while (testRun.getVirtualUsers() == null && !hasTimedOut(TEST_END_TIMEOUT, start)) {
+    while ((testRun.getVirtualUsers() == null || testRun.getTestRunStatistics() == null)
+        && !hasTimedOut(TEST_END_TIMEOUT, start)) {
       Thread.sleep(STATUS_POLL_PERIOD.toMillis());
       testRun = apiClient.findTestRunById(testRun.getId());
     }

Original file line number	Diff line number	Diff line change
`@@ -532,7 +532,8 @@ private TestRun awaitTestEnd(TestRun testRun)`
`532`	`532`
`533`	`533`	`private TestRun awaitVirtualUsers(TestRun testRun) throws InterruptedException, IOException {`
`534`	`534`	`Instant start = Instant.now();`
`535`		`- while (testRun.getVirtualUsers() == null && !hasTimedOut(TEST_END_TIMEOUT, start)) {`
	`535`	`+ while ((testRun.getVirtualUsers() == null \|\| testRun.getTestRunStatistics() == null)`
	`536`	`+ && !hasTimedOut(TEST_END_TIMEOUT, start)) {`
`536`	`537`	`Thread.sleep(STATUS_POLL_PERIOD.toMillis());`
`537`	`538`	`testRun = apiClient.findTestRunById(testRun.getId());`
`538`	`539`	`}`