Merge pull request #90 from Vampire/make-test-unit-finder-thread-safe

Make the test unit finder thread-safe
pitest · May 15, 2023 · 0f30f27 · 0f30f27
2 parents b08c063 + 73391eb
commit 0f30f27
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/main/java/org/pitest/junit5/JUnit5TestUnitFinder.java b/src/main/java/org/pitest/junit5/JUnit5TestUnitFinder.java
@@ -19,6 +19,7 @@
 import java.util.List;
 
 import static java.util.Collections.emptyList;
+import static java.util.Collections.synchronizedList;
 import static java.util.Collections.unmodifiableList;
 import static java.util.stream.Collectors.toList;
 
@@ -95,15 +96,15 @@ public List<TestUnit> findTestUnits(Class<?> clazz, TestUnitExecutionListener ex
     private class TestIdentifierListener implements TestExecutionListener {
         private final Class<?> testClass;
         private final TestUnitExecutionListener l;
-        private final List<TestIdentifier> identifiers = new ArrayList<>();
+        private final List<TestIdentifier> identifiers = synchronizedList(new ArrayList<>());
 
         public TestIdentifierListener(Class<?> testClass, TestUnitExecutionListener l) {
             this.testClass = testClass;
             this.l = l;
         }
 
         List<TestIdentifier> getIdentifiers() {
-            return unmodifiableList(identifiers);
+            return unmodifiableList(new ArrayList<>(identifiers));
         }
 
         @Override