Java登錄單元測(cè)試 一種不好的做法

日志記錄是調(diào)試過(guò)程中不可避免的一部分。好吧，至少在現(xiàn)代高級(jí)編程語(yǔ)言和架構(gòu)中是這樣。這不是三十年前的事了，而是現(xiàn)在。有時(shí)我們跟蹤變量，雖然這樣做的很少。更多的時(shí)候我們只是將它們打印到控制臺(tái)。此外，我們不只是使用println控制臺(tái)打印或我們擁有的任何東西來(lái)打印它們；相反，我們將消息發(fā)送到日志框架，該框架處理控制臺(tái)或任何其他日志記錄目的地，如文件。這種框架的美妙之處在于我們不需要在調(diào)試完成后刪除日志——我們只需配置框架以抑制生產(chǎn)環(huán)境中的所有調(diào)試級(jí)別的消息。一些日志記錄可能發(fā)生在單元測(cè)試中，我們是否也把它們留下或者不留下？

這是一個(gè)示例（它是對(duì)?CalcTest.java? 中來(lái)自?Polystat?的真實(shí)單元測(cè)試的簡(jiǎn)化，?Polystat?是我們目前正在研究的靜態(tài)分析器）：

import com.jcabi.log.Logger;
import com.jcabi.xml.XML;
import org.hamcrest.MatcherAssert;
import org.hamcrest.Matchers;
import org.junit.jupiter.api.Test;
public final class FooTest {
  @Test
  public void buildsSimpleXml() {
    final XML xml = new Foo().build();
    Logger.debug(this, "This is the XML:\n%s", xml.toString());
    MatcherAssert.assertThat(
      xml,
      Matchers.notNullValue()
    );
  }
}

這是 Java，我將?JUnit5 + Hamcrest?與我自己的日志記錄框架?jcabi-log?一起使用，它是?Slf4j?的裝飾器，使用?Log4j?打印到控制臺(tái)。

這里發(fā)生了什么？有一個(gè)Foo帶有方法的類(lèi)build()，它生成一個(gè) XML 文檔（我使用的是?jcabi-xml?庫(kù)，它是?JDK DOM?的裝飾器）。然后，單元測(cè)試將 XML 文檔的內(nèi)容打印到控制臺(tái)并做出一個(gè)非常愚蠢的斷言：文檔不是 NULL。這很愚蠢，因?yàn)槿绻?NULL，那么日志語(yǔ)句在?.toString()?調(diào)用時(shí)就會(huì)失敗。

我是這段代碼的開(kāi)發(fā)者，所以我知道發(fā)生了什么：我懶得寫(xiě)一個(gè)正確的斷言，它會(huì)查看 XML 文檔并確保里面有正確的元素。我只是將它打印到控制臺(tái)，目視確認(rèn)其有效性并稱(chēng)其為一天。如果我有更多的時(shí)間，這就是我編寫(xiě)更好的單元測(cè)試的方式（我剛剛對(duì) Polystat 測(cè)試進(jìn)行了改進(jìn)）：

import com.jcabi.matchers.XhtmlMatchers;
import org.hamcrest.MatcherAssert;
import org.junit.jupiter.api.Test;
public final class FooTest {
  @Test
  public void buildsSimpleXml() {
    MatcherAssert.assertThat(
      XhtmlMatchers.xhtml(new Foo().build()),
      XhtmlMatchers.hasXPath("http://foo")
    );
  }
}

現(xiàn)在，構(gòu)建了 XML 文檔，然后測(cè)試其中是否存在?//foo XPath?。只有在斷言失敗的情況下，才會(huì)將文檔的內(nèi)容打印到控制臺(tái)。如果 XML 具有所需的 XPath，則不會(huì)有控制臺(tái)輸出，這對(duì)未來(lái)的開(kāi)發(fā)人員意味著沒(méi)有噪音。

此外，現(xiàn)在它是一個(gè)單語(yǔ)句測(cè)試，這本身就是一種很好的做法。

回顧我測(cè)試和記錄的經(jīng)驗(yàn)，我認(rèn)為記錄單元測(cè)試是一個(gè)壞主意。有時(shí)不可避免，因?yàn)槲覀儜卸杌蚋緵](méi)有足夠的時(shí)間，但仍然很糟糕。日志記錄幫助我們?cè)谝曈X(jué)上確認(rèn)輸出的正確性，但它從項(xiàng)目中帶走了這些知識(shí)。那些將在稍后進(jìn)行測(cè)試的人不會(huì)知道我們?cè)谀抢锟吹搅耸裁?。他們?huì)在控制臺(tái)看到輸出，但不知道它是否仍然符合我在撰寫(xiě)本文時(shí)的期望。

我會(huì)說(shuō)單元測(cè)試中的每個(gè)日志記錄行都是來(lái)自其作者的消息：“我對(duì)我現(xiàn)在看到的數(shù)據(jù)有所了解，但我懶得告訴你，你只需要相信我好的?！?/p>

我建議我們不要在我們的代碼中留下這樣的消息。