Spring环境下静态方法的mock

其实我不是很理解为什么用了Spring还需要使用静态方法来提供单例之类的服务。也许是传承自较老的代码，也许对Spring对象的生命周期不太肯定，也许要与不属于Spring上下文的代码共享逻辑。总之现实常常还是能看到这种代码的。

/**
*提供静态方法的类
*/
public class PersonPool {
    private static PersonPool instance;
    static {
        instance = new PersonPool();
    }
    private PersonPool() {
        // ...
    }
    public static PersonPool getInstance() {
        return instance;
    }
    public String next() {
        throw new RuntimeException("resource not available in test");
    }
}
/**
*使用静态方法的服务
*/
public class SampleService {
    public String greeting() {
        String name = PersonPool.getInstance().next();
        return "Hello " + name;
    }
}

一旦这些静态方法中用到了环境相关，需要在测试中隔离的资源，写单元测试就时就会遇到困难。
这类代码是无法用mockito来mock的。PowerMock可以mock静态方法，不过使用中会发现会给测试引入更多的细节，这些细节都是妨碍表达测试意图的噪音。
而且往往mock掉方法本身，又发现类初始化有个坑……，一点点解决下去，最后测试凑合能跑起来，但是充满了各种和实现紧密耦合的细节。实现代码里两行换个顺序，也许测试就不工作了。

其实这种处境，往往是因为抱着“不能为了测试修改实现代码”的信念造成的。
如果抛开这个假设，对代码稍作修改，测试就不再是个问题了。

/**
*提供静态方法的类保持不变
*/
public class PersonPool {...}
/**
*增加适配类
*/
@Component
public class PersonPoolProvider {
    public PersonPool getInstance() {
        return PersonPool.getInstance();
    }
}
/**
*改用自动注入的适配对象
*/
public class SampleService {
    @Autowired
    PersonPoolProvider personPoolProvider;

    public String greeting() {
        String name = personPoolProvider.getInstance().next();
        return "Hello " + name;
    }
}

可以看到只需要对实现代码进行影响很小的修改：

• 未改变原有静态方法的执行逻辑和生命周期。
• 不影响其它使用静态方法的代码。
• 适配类足够简单，无需测试。

希望这不是个为遗留代码补测试的权宜之计，而是进一步改进已有代码的开端。
有以下问题可以探讨：

• util类、方法是否应该有某种资源的所有权？
• 现实中单个存在的事物是否就一定用单例实现？
• 使用单例时，是在用它“单个实例”的保证，还是在利用可以从类名获得实例的“可访问性”？