单元测试静态方法

Question

我正在尝试在这里为方法解密编写一个测试用例。

    private static Codec codec;

    static {
        try {
            codec = new Codec(encryptionType, encryptionKey, false, true, false);
        } catch (CodecException e) {
            throw new RuntimeException("Codec initialisation failed", e);
        }
    }


    public static String decrypt(final String toDecrypt) throws CodecException {
        String decrypted = codec.decryptFromBase64(toDecrypt);
        if (decrypted.endsWith(":")) {
            decrypted = decrypted.substring(0, decrypted.length() - 1);
        }
        return decrypted;
    }

测试用例：

    @Mock
    private Codec codec;
    @Test
    public void test_decrypt_Success() throws CodecException {
        when(codec.decryptFromBase64(TestConstants.toDecrypt)).thenReturn(TestConstants.decrypted);
        assertEquals(DocumentUtils.decrypt(TestConstants.toDecrypt), TestConstants.decrypted);
    }

由于这是一个静态方法，我无法在测试套件中注入该类的实例并模拟其编解码器。上述代码按预期从编解码器库中的 assert 抛出错误。

您用什么方法来测试这样的静态方法？还是我根本不应该为此编写测试？

Answer 1

根据我在这些情况下的经验，我只是在 @Before 方法中准备所有实例：

private Codec codec;

@Before
public void setup() throws CodecException {
  codec = new Codec(encryptionType, encryptionKey, false, true, false);
}

Answer 2

有许多不同的捷径可以实现相同的目标（如 cmets 和其他答案中所指出的那样），但从长远来看，并非所有这些方法都是好的。

我建议创建一个实现解密功能的单例类。所以，你不必创建多个实例，也不需要有静态方法来解密，你可以更容易地注入你的编解码器（我假设你没有多种类型的编解码器根据您的 cmets。但是，如果您这样做，则应相应地调整功能）。

更多参考：Why use a singleton instead of static methods?

关于为什么要谨慎使用静态的参考：- Why are static variables considered evil?

Answer 3

在 Java 中，静态方法并非旨在设置依赖关系。
所以把依赖切换成 mock 真的很不自然。
您可以为该字段提供一个 static 设置器，例如：

private static Codec codec;
public static void setCodec(Codec codec){
   this.codec = codec;
}

你可以用 setCodec(...) 设置一个模拟但是呃......

但是忘了，只要把事情做好：重构代码以删除所有静态并引入设置编解码器的构造函数。

private Codec codec;
public MyClassUnderTest(Codec codec){
   this.codec codec;
}

IOC 可以在这里帮助使被测类成为单例并简化依赖注入。
如果在您的情况下不可能，Java 5 枚举至少可以帮助您解决单例问题。