diff --git a/spring-test/src/test/java/org/springframework/test/context/testng/TrackingTestNGTestListener.java b/spring-test/src/test/java/org/springframework/test/context/testng/TrackingTestNGTestListener.java index b529e29fc0..5ce4dd8b80 100644 --- a/spring-test/src/test/java/org/springframework/test/context/testng/TrackingTestNGTestListener.java +++ b/spring-test/src/test/java/org/springframework/test/context/testng/TrackingTestNGTestListener.java @@ -17,6 +17,7 @@ package org.springframework.test.context.testng; import java.util.ArrayList; +import java.util.Collections; import java.util.List; import java.util.concurrent.atomic.AtomicInteger; @@ -33,28 +34,25 @@ import org.testng.ITestResult; */ public class TrackingTestNGTestListener implements ITestListener { - public final List throwables = new ArrayList<>(); - public final AtomicInteger testStartCount = new AtomicInteger(); public final AtomicInteger testSuccessCount = new AtomicInteger(); public final AtomicInteger testFailureCount = new AtomicInteger(); + public final List throwables = Collections.synchronizedList(new ArrayList<>()); + public final AtomicInteger failedConfigurationsCount = new AtomicInteger(); @Override - public void onFinish(ITestContext testContext) { - this.failedConfigurationsCount.addAndGet(testContext.getFailedConfigurations().size()); + public void onTestStart(ITestResult testResult) { + this.testStartCount.incrementAndGet(); } @Override - public void onStart(ITestContext testContext) { - } - - @Override - public void onTestFailedButWithinSuccessPercentage(ITestResult testResult) { + public void onTestSuccess(ITestResult testResult) { + this.testSuccessCount.incrementAndGet(); } @Override @@ -68,17 +66,8 @@ public class TrackingTestNGTestListener implements ITestListener { } @Override - public void onTestSkipped(ITestResult testResult) { - } - - @Override - public void onTestStart(ITestResult testResult) { - this.testStartCount.incrementAndGet(); - } - - @Override - public void onTestSuccess(ITestResult testResult) { - this.testSuccessCount.incrementAndGet(); + public void onFinish(ITestContext testContext) { + this.failedConfigurationsCount.addAndGet(testContext.getFailedConfigurations().size()); } }