package samples.study;
/**
*
* @author
*/
public class Singleton {
private Singleton(){}
private static final Singleton instance = new Singleton();
public static Singleton getInstance() {
System.out.println(Thread.currentThread().getName() + " getInstance");
return Singleton.instance;
}
}
Unit Test Case Seems to work fine and singleton objects are returned from Multiple threads. Help me spot any issues here ?
package samples.study;
import static org.junit.jupiter.api.Assertions.assertSame;
import java.util.ArrayList;
import java.util.List;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
class SingletonTest {
@BeforeEach
void setUp() throws Exception {
}
@Test
void test() throws InterruptedException {
ExecutorService s = Executors.newCachedThreadPool();
Singleton obj = Singleton.getInstance();
int threadCount = 10;
List<Singleton> objList = new ArrayList<Singleton>();
while (threadCount > 0) {
s.submit(() -> {
objList.add(getInstance(s));
});
threadCount--;
}
while (threadCount > 0) {
assertSame(obj, objList.get(threadCount));
}
}
public Singleton getInstance(ExecutorService s) {
Singleton obj = Singleton.getInstance();
System.out.println("1 = " + Thread.currentThread().getName() + obj);
return obj;
}
}
Result of UT :
- main getInstance pool-1-thread-3 getInstance pool-1-thread-2 getInstance 1 = pool-1-thread-2samples.study.Singleton@75cbce36 pool-1-thread-4 getInstance 1 = pool-1-thread-4samples.study.Singleton@75cbce36 pool-1-thread-8 getInstance 1 = pool-1-thread-8samples.study.Singleton@75cbce36 pool-1-thread-7 getInstance pool-1-thread-9 getInstance pool-1-thread-6 getInstance pool-1-thread-10 getInstance 1 = pool-1-thread-10samples.study.Singleton@75cbce36 1 = pool-1-thread-6samples.study.Singleton@75cbce36 pool-1-thread-5 getInstance 1 = pool-1-thread-5samples.study.Singleton@75cbce36 1 = pool-1-thread-9samples.study.Singleton@75cbce36 1 = pool-1-thread-7samples.study.Singleton@75cbce36 pool-1-thread-1 getInstance 1 = pool-1-thread-1samples.study.Singleton@75cbce36 1 = pool-1-thread-3samples.study.Singleton@75cbce36