始终抛出异常的方法后面的行的代码覆盖率

Question

我知道 100% 的代码覆盖率只是一个目标，但是将包含右大括号的行算作 未覆盖 很烦人，因为它遵循一个方法调用，其唯一目的是抛出异常。这是我的基本测试用例类中的一个简单示例来演示：

function checkForSkipAllTests() {
    if (self::$_skipAllTests) {
        self::markTestSkipped();   // [1] always throws an exception
    }                              // [2] shown as executable but not covered
}

由于 [1] 总是退出该方法，因此行 [2] 实际上无法到达。有没有办法通过注释markTestSkipped() 方法本身来告诉 Xdebug？

Answer 1

您的pull request 已合并，因此从 php-code-coverage 1.1.2 开始，应该很快就会出现（使用 PHPUnit 3.6.3 或 3.6.4），人们将能够编写：

private static function checkForSkipAllTests() {
    if (self::$_skipAllTests) {
        self::markTestSkipped();
    } // @codeCoverageIgnore
}

在更远的将来，当 xDebug 将能够提供“条件”覆盖时，我想我记得讨论过通过重构使整个问题消失，因为在最后一个语句时，右括号将被视为“覆盖”在函数中终止函数......但我可能错了

Answer 2

您可以用stard/end cmets 包围该行，让PHP_CodeCoverage 忽略它，但这意味着在调用该方法的任何地方都这样做。

function checkForSkipAllTests() {
    if (self::$_skipAllTests) {
        self::markTestSkipped();
    // @codeCoverageIgnoreStart
    }
    // @codeCoverageIgnoreEnd
}

这是一场维护噩梦，容易出错。我真的很想避免这种解决方案。

Answer 3

我知道 100% 的代码覆盖率只是一个目标，但是将包含右括号的行算作未覆盖是很烦人的，因为它遵循一个唯一目的是引发异常的方法调用。这是我的基本测试用例类中的一个简单示例来演示：

确实，100% 的代码覆盖率并不是一个目标，但它很高兴，特别是如果你需要零时间来实现它。我确实想知道；您的测试不是要测试的文件。我从不测试我的测试，也不对它们的代码覆盖率感兴趣。我已经知道哪些测试完成了，哪些成功了，哪些失败了，哪些被跳过了。这就是 PHPUnit 为我带来的； .....S...F 已经足够反馈了。

我的测试在一个单独的目录中，不包含在代码覆盖范围内；在我看来，这样做似乎毫无用处。无论如何，如果您对测试用例的代码覆盖率报告感到满意，您可能想简单地摆脱}，如下所示：

function checkForSkipAllTests() {
    if (self::$_skipAllTests)
        self::markTestSkipped();
}

是的，我知道没有大括号的 if 会让我成为回答你问题的最不酷的人，但它似乎比拥有一些神奇的注释更容易解决。