Synchronise the list as multiple threads are calling it

Richard Jones · Richard Jones · commit 7ab799c452a3 · 2015-10-13T21:23:32.000-03:00
diff --git a/singleton/src/test/java/com/iluwatar/singleton/LazyLoadedSingletonThreadSafetyTest.java b/singleton/src/test/java/com/iluwatar/singleton/LazyLoadedSingletonThreadSafetyTest.java
@@ -3,6 +3,7 @@
 import org.junit.Test;
 
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 import java.util.concurrent.*;
 
@@ -18,7 +19,7 @@
 public class LazyLoadedSingletonThreadSafetyTest {
 
     private static final int NUM_THREADS = 5;
-    private List<ThreadSafeLazyLoadedIvoryTower> threadObjects = new ArrayList<>();
+    private List<ThreadSafeLazyLoadedIvoryTower> threadObjects = Collections.synchronizedList(new ArrayList<>());
 
     //NullObject class so Callable has to return something
     private class NullObject{private NullObject(){}}
