如果你做 TDD,你应该为 getter 和 setter 编写一个测试。也。不要在没有测试的情况下编写一行代码——即使你的代码非常简单。
这是一种宗教战争,使用串联的 getter 和 setter 进行测试,或者通过使用单元测试框架功能访问受保护的类成员来隔离每个。作为一个黑盒测试人员,我更喜欢将我的单元测试代码绑定到公共 api,而不是将其绑定到具体的实现细节。我期待改变。我想鼓励开发人员重构现有代码。并且类内部不应该影响“外部代码”(在这种情况下是单元测试)。我不想在内部发生变化时中断单元测试,我希望它们在公共 api 发生变化或行为发生变化时中断。好的,好的,如果单元测试失败,请不要指出唯一的问题来源。我确实必须查看 getter 和 setter 以找出导致问题的原因。大多数时候,您的 getter 非常简单(少于 5 行代码:例如,返回和可选的带有异常的空检查)。因此,首先检查这一点没什么大不了的,也不费时。并且大多数时候检查 setter 的快乐路径只是稍微复杂一点(即使您有一些验证检查)。
尝试隔离您的测试用例 - 为 SUT(被测对象)编写一个测试,以验证其正确性,而无需依赖其他方法(我上面的示例除外)。您隔离测试的次数越多,您的测试发现问题的次数就越多。
根据您的测试策略,您可能只想涵盖快乐的路径(务实的程序员)。或者悲伤的道路,也是。我更喜欢涵盖所有执行路径。当我认为我发现了所有执行路径时,我会检查代码覆盖率以识别死代码(而不是识别是否存在未发现的执行路径 - 100% 的代码覆盖率是一个误导性指标)。
黑盒测试人员的最佳实践是在严格模式下使用 phpunit 并使用 @covers 隐藏附带覆盖。
当您编写单元测试时,您对 A 类的测试应独立于 B 类执行。因此,您对 A 类的单元测试不应调用/覆盖 B 类的方法。
如果您想识别过时的 getter/setter 和其他“死”方法(生产代码不使用),请使用静态代码分析。您感兴趣的指标称为“方法级别的传入耦合(MethodCa)”。不幸的是,此指标 (ca) 在 PHP Depend 中的方法级别不可用(请参阅: http: //pdepend.org/documentation/software-metrics/index.html和http://pdepend.org/documentation/software-metrics /传入耦合.html)。如果您真的需要它,请随时将其贡献给 PHP Depend。排除来自同一类的调用的选项将有助于获得没有“附带”调用的结果。如果您确定一个“死方法”,请尝试确定它是否打算在不久的将来使用(与具有 @depriated 注释的其他方法的对应物),否则将其删除。如果它仅在同一类中使用,请将其设为私有/受保护。不要将此规则应用于库代码。
计划 B:如果您有验收测试(集成测试、回归测试等),您可以在不同时运行单元测试和 phpunits 严格模式的情况下运行该测试。这可能会导致非常相似的代码覆盖率结果,就像您分析了生产代码一样。但在大多数情况下,您的非单元测试不如您的生产代码那么强大。如果此计划 B 与生产代码“足够”以获得有意义的结果,则取决于您的纪律。
进一步阅读: - 书籍:实用程序员 - 书籍:清洁代码