From 7ab799c452a31584e52f1f33535aac80ed0c871e Mon Sep 17 00:00:00 2001 From: Richard Jones Date: Tue, 13 Oct 2015 21:23:32 -0300 Subject: [PATCH] Synchronise the list as multiple threads are calling it --- .../singleton/LazyLoadedSingletonThreadSafetyTest.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/singleton/src/test/java/com/iluwatar/singleton/LazyLoadedSingletonThreadSafetyTest.java b/singleton/src/test/java/com/iluwatar/singleton/LazyLoadedSingletonThreadSafetyTest.java index 0968cefd3..07f99005e 100644 --- a/singleton/src/test/java/com/iluwatar/singleton/LazyLoadedSingletonThreadSafetyTest.java +++ b/singleton/src/test/java/com/iluwatar/singleton/LazyLoadedSingletonThreadSafetyTest.java @@ -3,6 +3,7 @@ package com.iluwatar.singleton; import org.junit.Test; import java.util.ArrayList; +import java.util.Collections; import java.util.List; import java.util.concurrent.*; @@ -18,7 +19,7 @@ import static org.junit.Assert.assertEquals; public class LazyLoadedSingletonThreadSafetyTest { private static final int NUM_THREADS = 5; - private List threadObjects = new ArrayList<>(); + private List threadObjects = Collections.synchronizedList(new ArrayList<>()); //NullObject class so Callable has to return something private class NullObject{private NullObject(){}}