diff --git a/testng-core/src/main/java/org/testng/SuiteRunner.java b/testng-core/src/main/java/org/testng/SuiteRunner.java index 41244e821..452e57112 100644 --- a/testng-core/src/main/java/org/testng/SuiteRunner.java +++ b/testng-core/src/main/java/org/testng/SuiteRunner.java @@ -34,8 +34,7 @@ public class SuiteRunner implements ISuite, ISuiteRunnerListener { private static final String DEFAULT_OUTPUT_DIR = "test-output"; - private final Map suiteResults = - Collections.synchronizedMap(Maps.newLinkedHashMap()); + private final Map suiteResults = Maps.newLinkedHashMap(); private final List testRunners = Lists.newArrayList(); private final Map, ISuiteListener> listeners = Maps.newLinkedHashMap(); @@ -514,7 +513,9 @@ public String getOutputDirectory() { @Override public Map getResults() { - return suiteResults; + // Just to ensure that we guard the internals of the suite results we now wrap it + // around with an unmodifiable map. + return Collections.unmodifiableMap(suiteResults); } /**