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
0

There are 0 answers