如何确保开发人员知道要测试的不重要内容？ [关闭]

Question

在提出这个问题之前，我阅读了以下问题及其答案：

• 单元测试的合理代码覆盖率是多少（为什么）？
• 在进行单元测试时，100％的代码覆盖率真的是一件好事吗？
• 单元测试代码覆盖率-您有100％的覆盖率吗？

并阅读其他博客，例如Martin Fowler-TestCoverage 。

我的结论-当然是简历-社区说：

• 不要浪费时间（时间就是金钱）来创建测试，只是为了获得100％的代码覆盖率。
• 覆盖率测试的80％或90％之类的神奇数字可能涵盖99.99999％的功能。 那么，为什么要浪费时间来完成0.000001％的功能？

我同意。 但是我担心给开发人员提供不创建测试的机会，因为他认为这并不重要。 我知道我们可以通过在发布代码之前先让另一个人验证代码来避免这些错误。

问题

以一种控制开发人员知道不应该进行测试的方式进行思考的方法，将是一种很好的做法，即在代码中为开发人员明确标记创建一种//special comment ，他在哪里知道该不值得经过测试？ 还是那是与代码无关的无关紧要的信息？ 有人可以提出另一种想法来实现它吗？

在阅读该问题的任何答案之前，我认为这是一个好习惯，因为第三方可以检查（是否同意）开发人员未涵盖该代码的原因。

Java示例：

public String encodeToUTF8(String value){
    String encodedValue = null;

    try {
        encodedValue = URLEncoder.encode(value, "UTF-8");
    }
    catch (UnsupportedEncodingException ignore) {
        // [coverage-not-need] this exception will never occur because UTF-8 is a supported encoding type
    }
    return encodedValue;
}

术语：100％的代码覆盖率意味着覆盖所有分支，而不仅仅是所有行。

Answer 1

大多数覆盖率工具都具有这一点，在特殊注释中可以声明该代码不包含覆盖率。 例如，Perl的Devel :: Cover使用# uncoverable覆盖，而Ruby的# :nocov:具有# :nocov:

但是，我会警告开发人员不要过早地宣布事情是无法发现的，或者过分依赖它。 编写代码的开发人员可能对测试机会视而不见。 就像任何注释一样，如果周围的代码更改，它可能会过时。 过多使用它会给测试覆盖率带来错误的信心。

完成测试，运行覆盖范围并确定该语句实际上几乎无法测试之后，请使用它作为最后的手段。 同样，我告诫不要以它为借口在难以测试的事物上作文章。 通常，这表明需要重新设计，而不是真正无法测试的代码。

---

您的示例代码是滥用“无法覆盖的”代码的完美案例。 如果永远不会发生例外，我想知道为什么根本就没有捕获块。 如所写，如果确实发生，它将被静音，并且用户将不知道为什么稍后在代码中某个地方会收到NullPointerException。 相反，应该没有try/catch块。 在编码失败的特殊情况下 ，应允许该异常引发错误。

public String encodeToUTF8(String value){
    return URLEncoder.encode(value, "UTF-8");
}

我不是Java程序员，但我知道它不喜欢未经检查的异常。 如果Java要求您捕获所有可能的异常（ugh），则该catch应声明或重新抛出； 不会使异常消失的东西。

这就是为什么您要非常非常谨慎地并且仅在经过大量审查之后才使用“无法覆盖的”标记。 检查发现的代码通常会导致发现隐藏的错误或设计不良的代码。