社区首页 >专栏 >测试代码的坏味道

测试代码的坏味道

Phodal

发布于 2020-01-16 03:19:25

1.6K00

代码可运行

文章被收录于专栏：phodalphodal

运行总次数：0

代码可运行

测试代码才能真正体现开发人员的水平。

追求技术卓越是采用敏捷的第一成功要素。—— Jeff Sutherland 敏捷宣言创始人之一

Phodal: “你为什么写测试？”
开发人员 A：“为了测试覆盖率”。
Phodal：“咦，这个测试没有断言”
开发人员 A 笑了笑。

某次代码重构中，我发现代码的测试覆盖率很高，过程中出了一些错误，重构手法不正确是一个问题。但是在重构的过程中，发现有些测试都是没有意义的，所以我变转向开始研究测试坏味道，顺便在 Coca 中写了个识别代码测试坏味道的工具。

测试反应开发人员的水平

与编写业务代码相比，测试代码才能真正体现开发人员的水平。你可以用测试来判断开发人员的水平：

有没有为自己的代码编写测试？
测试中有没有断言？
测试中有没有包含有效的断言？
测试的长度是否正常？
测试中的断言是否合理？

没有断言的测试意味着原本的代码写得又臭又长；测试中只包含无效断言表明开发人员在划水；测试方法的长度过长，表明原有的方法可以进一步抽象……

顺便一提，我们推荐的 TDD（测试驱动开发），它并非是银弹。但是，当你来面对一个复杂的场景时，它可以驱动出可测试的代码，辅助以重构，能帮助你写出短小的函数。借此整体上降低整一部分代码的开发 + 维护成本。

我知道你想说有人的很聪明，可以写出的代码足够的健壮。但是呢，这样的人存在吗？即使存在的话，需求是善变的，下一次接手代码的人能保证原有的功能是好的吗？

正视测试同正视 bug 一样

软件测试（英语：Software Testing），描述一种用来促进鉴定软件的正确性、完整性、安全性和质量的过程。换句话说，软件测试是一种实际输出与预期输出间的审核或者比较过程。

项目代码是日常接触最多的部门，我们会直面代码中的问题，也因此会重视它们。对于测试嘛，就呵呵了。但是，你们就这么忽略了测试的重要性。

我们编写测试是为了提升软件开发质量，一旦代码改出了问题，那么测试就会帮我们找出破坏了的原有功能。而不是在长长的软件测试反馈链之后，才发现：原来我们改出了 bug。

不过呢，当你的业务进度压力大的时候，没有时间编写测试，反而 bug 就更多了。

测试代码坏味道

代码坏味道是对应于系统中的更深层问题的表面指示。

我们一般谈论代码坏味道的时候，主体是项目代码，而测试代码坏味道则往往被人忽略了。测试代码能直观地反应出代码的设计问题，它们是 API 的使用方，它们是 API 的第一等使用方。

测试代码坏味道，是指单元测试代码中的不良编程实践（例如，测试用例的组织方式，实现方式以及彼此之间的交互方式），它们表明测试源代码中潜在的设计问题。

如 Robert C. Martin 在《代码整洁之道》所说的那样，好的测试应该是：

快速（Fast），测试应该够快。
独立（Indendent），测试应该相互独立。
可重复（Repeatable），测试应当可在任何环境中通过。
自足验证（Self-Validating），测试应该有布尔值输出。
及时（Timely），测试应该及时编写。

要我说的话，它应该还有：

同一人编写，测试应该由开发业务代码的编写。这样他/他们才知道自己代码写得烂。
边界，测试直接不影响业务代码。这里指的主要是 private -> public 的行为，又或者是业务代码中包含测试代码，而非因为测试对原有代码重构。
有效命名。测试信息应该体现在方法名上，表达某一个特定的需求。

测试代码应该遵循生产代码的质量标准。

命名在测试中也是一大难题，我们如可以采用 Roy Osherove（《单元测试的艺术》作者）推荐的 UnitOfWork_StateUnderTest_ExpectedBehavior 命名法则。几个示例如下：

Public void Sum_NegativeNumberAs1stParam_ExceptionThrown()
Public void Sum_NegativeNumberAs2ndParam_ExceptionThrown ()
Public void Sum_simpleValues_Calculated ()

测试代码坏味道示例

先让我们来看看有哪些常见的测试坏味道：

空的测试。测试是生成的，但是没有内容。
忽略的测试。即测试被 Ignore
没有断言的测试。为了测试覆盖率而出现的测试
多余的 Println。调试时留下的讯息。
多重断言。每个测试函数只应该测试一个概念。
……。

然后，再来个 Examples。

这是 Arduino 代码中的 I18NTest.java 文件，先看看文件，再看看问题：

@Test
 public void testAllLocales() {
 for (Language language : Languages.languages) {
 if (!language.getIsoCode().equals("")) {
 Locale locale = toLocale(language);
 ResourceBundle bundle = ResourceBundle.getBundle("processing.app.i18n.Resources", locale);
 if (locale.equals(bundle.getLocale())) {
 Collections.list(bundle.getKeys()).stream().map(bundle::getString).filter(key -> !key.contains("<html")).forEach(key -> {
 try {
 I18n.format(key);
 } catch (IllegalArgumentException e) {
 System.out.println(language);
 System.out.println(key);
 throw e;
 }
 });
 } else {
 System.out.println("Missing locale: " + locale);
 }
 }
 }
 }

问题有很多：

没有断言
多余的 print 函数
try...catch...
糟糕的测试命名

这个测试的正确作法之一应该是：使用容器 collection 来过滤出正确的语言，最后对比长度是否正确。

再举个例子：

@Test
public void testXmlSanitizer() {
 boolean valid = XmlSanitizer.isValid("Fritzbox");
 assertEquals("Fritzbox is valid", true, valid);
 System.out.println("Pure ASCII test - passed");

 valid = XmlSanitizer.isValid("Fritz Box");
 assertEquals("Spaces are valid", true, valid);
 System.out.println("Spaces test - passed");

 ...
}