测试静态方法类必须更改为单例以进行 JUnit 测试

Question

我有一个静态方法类，Utils，它基本上用于实用方法，几乎​​每个类都使用它。它做一些事情，比如获取文件和其他基本的东西。我的测试人员将这个类更改为单例，这样每个使用 Utils 的类现在都必须调用 getInstance()。原因是除非是这种情况，否则他无法测试某些东西。在我看来，这在某些方面是错误的，这可能会导致问题。

public class Utils {

/**
 * Singleton method to allow for easier testing.
 * @return an instance of
 */
public synchronized static Utils getInstance() {
    if (instance == null) {
        instance = new Utils();
    }
    return instance;
}

public synchronized static void setInstance(Utils instance) {
    Utils.instance = instance;
}

/** Singleton to make testing easier **/
private static Utils instance = null;

public static boolean checkOSTen() {
    return getInstance()._checkOSTen();
}

private boolean _checkOSTen() {
    boolean autoPair = false;
    if (android.os.Build.VERSION.SDK_INT >= 10){   
        autoPair = true;
    }
    return autoPair;
}

}

顺便说一下，我有一个非常复杂的内部消息传递系统，它使用至少 7 个线程来发送消息，并且想看看静态方法调用与 Singleton 静态方法调用是否存在影响。

除此之外还有其他方法可以进行测试吗？似乎一些Java反射会得到你需要的东西。

Answer 1

您的测试人员可能会发现此代码难以测试，因为他们无法轻松更改 android.os.Build.VERSION.SDK_INT 的返回值，因此他们无法轻松测试根据此返回值使用的不同代码路径。通过使用实例方法，他们可以使用模拟框架来伪造_checkOSTen 方法以返回true 或false，具体取决于他们要测试的内容。

进行测试的另一种方法是根本不使用诸如此类的静态方法，而是使用类（可能称为DeviceCapabilities），该类将通过setter 或构造函数注入到需要了解的所有其他类中操作系统的版本。

然后，您的测试人员可以轻松地将 DeviceCapabilities 的实例传递给需要访问此信息的任何类。

也就是说，在您构建的这个阶段，进行这样的更改可能为时已晚，因此您的测试人员所做的更改是一个合理的折衷方案。