Stop overriding default methods in TrackingTestNGTestListener

At some point, TestNG started implementing listener methods as interface
default methods, so we no longer need to override those methods with
empty implementations.

Author: sbrannen

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 @@
  */
 public class TrackingTestNGTestListener implements ITestListener {
 
-	public final List<Throwable> throwables = new ArrayList<>();
-
 	public final AtomicInteger testStartCount = new AtomicInteger();
 
 	public final AtomicInteger testSuccessCount = new AtomicInteger();
 
 	public final AtomicInteger testFailureCount = new AtomicInteger();
 
-	public final AtomicInteger failedConfigurationsCount = new AtomicInteger();
+	public final List<Throwable> throwables = Collections.synchronizedList(new ArrayList<>());
 
+	public final AtomicInteger failedConfigurationsCount = new AtomicInteger();
 
-	@Override
-	public void onFinish(ITestContext testContext) {
-		this.failedConfigurationsCount.addAndGet(testContext.getFailedConfigurations().size());
-	}
 
 	@Override
-	public void onStart(ITestContext testContext) {
+	public void onTestStart(ITestResult testResult) {
+		this.testStartCount.incrementAndGet();
 	}
 
 	@Override
-	public void onTestFailedButWithinSuccessPercentage(ITestResult testResult) {
+	public void onTestSuccess(ITestResult testResult) {
+		this.testSuccessCount.incrementAndGet();
 	}
 
 	@Override
@@ -68,17 +66,8 @@ public void onTestFailure(ITestResult testResult) {
 	}
 
 	@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());
 	}
 
 }