assertionのないテスト-例外のテスト

以前参加した勉強会でassertionのないテストはアンチパターンだ、との話がありました。

本書の第２９章の「例外のテスト」がまさにassertがないテストでした。ですのでこの話題をあげて見ました。

@Test
public void testMissingRate() {
    try {
        exchange.findRate("USD", "GBP");
        fail();//findRateで例外が発生すればこの行に達しない。
    } catch (IllegalArgumentException expected) {
    }
}

例外のテスト

帰宅後調べて見たら例外をテストする方法は

try-fail-catchのイディオム(TDD本記載)
例外をcatchのブロックでassertionする
アノテーションで確認する

の３つがあることがわかりました。(なぜか最後の方法は成功しませんでした) サンプルを作って試して見ました。

※以下のテストの中のkti.toInteger()はString型の数値を表す数字または漢字を引数にとり Integer型を返しますkti.toInteger("一九"); //19

@DisplayName("例外系のテスト")
class 例外{
  @DisplayName("例外確認(TDD本にあったver)")
  @Test
  void 数値でない漢字が来た時の例外のテスト() {
    try{
      kti.toInteger("数値でない");
      fail();
    }catch(Exception e) {
    }
  }

  @DisplayName("例外確認(Assertを使うver)")
  @Test
  void 数値でない漢字が来た時の例外のテスト2() {
    try{
      kti.toInteger("数値でない");
      fail();
    }catch(Exception e) {
      assertEquals(NumberFormatException.class, e.getClass());
    }
  }

  @Rule
  public ExpectedException expectedException = ExpectedException.none();
  @DisplayName("例外確認(annotationを使うver)")
  @Test
  void 数値でない漢字が来た時の例外のテスト3() {
    expectedException.expect(java.lang.NumberFormatException.class);
    //throw new NumberFormatException();
    kti.toInteger("数値でない");
  }

Assertionのないテストがダメな理由

私が件の勉強会できちんと話を理解して聞いていなかったため、話題はそこで中断しました。

今思えば、テストは(成功/失敗)のどちらかの状態が必ず発生する必要があり、 assertionのないテストは必ず成功する=原則に反するから、なのではないか？と考えます。*1

テスト名が長くなる

状況ごとにテストクラスを分割する方法が良いそうです。 Javaでは@Nestedアノテーションでクラスを入れこにすることにより、テストクラスを分類できます。テストコードを書いて見ました。 https://github.com/yuyabu/KanjiToInteger/blob/master/KanjiToInteger/src/TestKanjiToInteger.java