单元测试辅助方法的正确方法

Question

我有以下 class

public class Multiplier {
    private static Map<Integer, Float> map;
    private static final float DEFAULT_MULTIPLIER = 4.4F;

    static {
        // This map is actually populated by reading from a file. This is an example on how the map looks like.
        map = new HashMap<>();
        map.put(1, "3.5F");
        map.put(2, "5.8F");
        map.put(3, "2.7F");
    }

    public static float getMultiplier(Integer id) {
        return map.getOrDefault(id, DEFAULT_MULTIPLIER);
    }
}

我想测试 getMultiplier 方法。 最好的办法是go一下吗？

1st - map 中有 1000 个条目。我是否应该只使用几个不同的 ID 调用 getMultiplier 方法并确保返回的值匹配？ 文件是static，以后偶尔会变，不过就是更新测试，没关系。

第二 - 我也想测试默认值。 我应该做类似的事情吗

Assert.assertEquals(4.4F, Multiplier.getMultiplier(<invalidId/null>));

我在 assertEquals 的预期参数中硬编码 DEFAULT_MULTIPLIER 值。 这甚至有意义吗？

我想知道对此方法进行单元测试的正确方法是什么？ 谢谢！

Answer 1

你不应该测试任何这些——它有太多的“给你自己的考试打分”的氛围。 这里可能出现的错误涉及您弄乱了常量或 static 文件，测试不太可能发现这些错误。 您将为将来的一系列更新做好准备（在这里您更新默认值，然后更新 go：哦，对，当然，我在测试中重复了自己，但我忘记更新那里的值）。 你没有在测试任何东西——你在给未来的自己做毫无意义的功课。

发挥创意——未来有什么可能会失败？ 不多——这是相当简单的代码。 测试可能的失败途径。

如果我是你，我会测试以下情况：

• 我会检查我确定永远不会在 map 中的键的值（现在不会，将来也不太可能）返回非 0。 这避免了必须对 4.4F 进行硬编码，或者必须将常量值标记为 package-private，但仍然确保了 2 个不太可能但可能出现的未来失败途径： .getMultiplier方法由于未来的错误而抛出异常，或者它返回 0 而不是实际值。 如果您稍后更改此默认值，它将避免必须更新测试。 除非您将其更改为 0 或“如果您要求未列出的数字的乘数，该方法现在会抛出”，这是对功能的根本更改，因此确实会导致测试失败。

• 我不会测试 static 文件中的任何值，因为要么你只是再次测试你是否正确地将它的值复制并粘贴到你的测试源文件中，并在你更新文件时给自己未来的作业来更新测试 - 或者如果您还通过从文件中获取它来动态调整您测试的 k/v 对，那么您就是在标记自己的考试（无论此 static 文件中读取的代码中可能存在什么错误，它也会出现在测试代码中） . 相反，我会采用能够读取 static 文件并将其转换为 static“常量映射”的东西并测试它。 在测试代码中，我可以有一个小示例“输入 static 文件”，然后我将测试乘法器 class 是否在给定此已知输入的情况下正确响应。 这使您免于将来的更新（测试代码始终向代码提供测试 static 数据，它不会查看将来可能更新的内容，因此本质上是“不稳定的”），并让您测试那些可能会在这里中断 - 将 static 文件变成填充的 map 的部分。

一旦测试确保“将 static 文件转换为 static 地图”代码有效，并且我测试了 getMultiplier 方法不会以两种明显的方式（抛出或返回 0）失败，那么您就测试了所有相关的内容。