Google Guice中Singleton作用域配置问题：测试单例复用致测试失败

方案1：让Guice完全管理ContactList的生命周期，实现测试隔离 #

你现在的问题根源是：原来的ContactList自己实现了单例模式（getInstance()），就算用Guice的@Singleton注解，其实只是把这个全局单例绑定到注入器里，导致所有测试共享同一个实例。要解决测试隔离问题，你可以：

1. 改造ContactList，移除自己的单例逻辑（比如去掉私有构造、getInstance()静态方法），把它改成普通类。
2. 在你的Guice模块里，直接绑定ContactList为单例：

public class ContactFactoryModule extends AbstractModule {
  @Override
  protected void configure() {
    bind(ContactList.class).in(Singleton.class);
    // 其他绑定逻辑...
  }
}

或者用@Provides方法：

@Provides @Singleton
ContactList provideContactList() {
  return new ContactList(); // 这里直接新建实例，不再调用原有的getInstance()
}

3. 测试类里通过Guice注入ContactList，并且保持每个测试用例创建新Injector的逻辑（你当前的setUp已经在做这个）：

public class ContactTest {
    ContactFactory factory;
    ContactList contacts;
    Injector injector;

    @BeforeEach
    public void setUp(){
        injector = Guice.createInjector(new ContactFactoryModule());
        factory = injector.getInstance(ContactFactory.class);
        contacts = injector.getInstance(ContactList.class); // 从Injector获取实例，替代直接调用单例方法
    }

    // 测试用例代码不变...

    @AfterEach
    public void tearDown(){
        // 每个测试的ContactList都是独立实例，若没有额外资源需要清理，这一步可以省略
        // contacts.clear(); 
    }
}

这样每个测试用例都会得到一个专属的ContactList单例（绑定到当前测试的Injector），测试之间的状态完全隔离，第二个测试就不会受第一个测试的影响。

方案2：保留全局单例，优化测试清理逻辑 #

如果生产环境必须保留ContactList的全局单例，那测试中还是需要清理状态，但可以优化成更符合依赖注入的方式：

1. 测试类里的ContactList通过Guice注入，替代手动调用getInstance()：

@BeforeEach
public void setUp(){
    injector = Guice.createInjector(new ContactFactoryModule());
    factory = injector.getInstance(ContactFactory.class);
    contacts = injector.getInstance(ContactList.class); // 用注入方式获取全局单例
}

2. 继续在tearDown里调用contacts.clear()，确保每个测试后状态重置。这种方式虽然需要手动清理，但比直接依赖单例方法更灵活，也符合你引入Guice的初衷。

总结 #

如果测试隔离是优先需求，推荐方案1，让Guice完全接管实例生命周期，彻底避免状态残留问题；如果必须保留全局单例，方案2是退而求其次的选择。

内容的提问来源于stack exchange，提问作者Anders