JUnit Testアノテーションで例外メッセージをアサートするにはどうすればよいですか?


315

@Test注釈付きのJUnitテストをいくつか作成しました。テストメソッドがチェックされた例外をスローし、例外とともにメッセージをアサートしたい場合、JUnit @Testアノテーションでそれを行う方法はありますか?私の知る限り、JUnit 4.7はこの機能を提供していませんが、将来のバージョンではこの機能を提供しますか?.NETでは、メッセージと例外クラスをアサートできることを知っています。Javaの世界で同様の機能を探しています。

これが私が欲しいものです:

@Test (expected = RuntimeException.class, message = "Employee ID is null")
public void shouldThrowRuntimeExceptionWhenEmployeeIDisNull() {}

1
もう少し考えてみましたが...メッセージをアサートすることは良い考えですか?あなたの質問で私はjunitソースコードを少し掘り下げました、そして彼らはこの機能を簡単に追加できたようです。彼らがそうしなかったという事実、それが良い習慣とは考えられないかもしれないと私に思わせます。プロジェクトでメッセージを表明することがなぜ重要なのですか?
c_maker 2010年

10
15行のコードを含むメソッドが2つの異なる場所から同じ例外をスローするとします。テストケースでは、例外クラスだけでなく、その中のメッセージもアサートする必要があります。理想的な世界では、異常な動作には独自の例外があるはずです。そうであれば、私の質問は決して発生しませんが、本番アプリケーションには、異常な動作ごとに独自のカスタム例外がありません。
Cshah 2010年

補足として、@expectedExceptionMessagePHPUnit には注釈があります。
バンサー

回答:


535

次のように、@RuleアノテーションをExpectedExceptionで使用できます。

@Rule
public ExpectedException expectedEx = ExpectedException.none();

@Test
public void shouldThrowRuntimeExceptionWhenEmployeeIDisNull() throws Exception {
    expectedEx.expect(RuntimeException.class);
    expectedEx.expectMessage("Employee ID is null");

    // do something that should throw the exception...
    System.out.println("=======Starting Exception process=======");
    throw new NullPointerException("Employee ID is null");
}

ExpectedExceptionドキュメントの例は(現在のところ)間違っていることに注意してください。パブリックコンストラクタがないため、を使用する必要がありますExpectedException.none()


1
注:expectMessageが空の文字列として指定された場合、メッセージの比較は実行されませんでした
redDevil

1
私にとって便利です。ありがとう。testmethodはthrows RuntimeException、例外をスローするコードを追加した後に必要です。キャッチしないでください...
Bumbolt

5
メソッドの小さなサブセットを目的としてフィールドを作成することは悪い習慣であるため、個人的にはこれを使用したくありません。応答に対する批判ではなく、JUnitの設計に対する批判です。OPの仮説的な解決策は、もし存在すれば、はるかに優れたものになります。
Sridhar Sarnobat

2
@redDevil:expectedMessageは、エラーメッセージがこの関数で指定された文字列(エラーメッセージのサブストリングのような)を「含む」かどうかをチェックします
tuan.dinh

3
文字列パラメータを指定したexpectMessageは、例外メッセージの完全一致について、String.containsチェックを実行します。hamcrestマッチャーを使用しますfailure.expectMessage(CoreMatchers.equalTo(...))
Sivabalan

42

私は@Rule答えが好きです。ただし、何らかの理由でルールを使用したくない場合。3番目のオプションがあります。

@Test (expected = RuntimeException.class)
public void myTestMethod()
{
   try
   {
      //Run exception throwing operation here
   }
   catch(RuntimeException re)
   {
      String message = "Employee ID is null";
      assertEquals(message, re.getMessage());
      throw re;
    }
    fail("Employee Id Null exception did not throw!");
  }

32

使用する必要があります@Test(expected=SomeException.class)か?例外の実際のメッセージをアサートする必要がある場合、これは私たちが行うことです。

@Test
public void myTestMethod()
{
  try
  {
    final Integer employeeId = null;
    new Employee(employeeId);
    fail("Should have thrown SomeException but did not!");
  }
  catch( final SomeException e )
  {
    final String msg = "Employee ID is null";
    assertEquals(msg, e.getMessage());
  }
}

6
キャッチブロックを記述し、その中でアサートを使用することを知っていますが、コードを読みやすくするために、アノテーションを使用したいと思います。
Cshah 2010年

また、それを「正しい」方法で行うときのような素晴らしいメッセージは表示されません。
NeplatnyUdaj 2013年

15
try / catchバージョンの問題は、今のJUnitが提供する@Test(expected=...)ExpectedException、私は何度も誰かに見ているということであるに電話を入れて忘れfail()の終わりでtryブロック。コードレビューに引っ掛からない場合、テストは偽陽性で常に合格する可能性があります。
ウィリアム価格

これが、私がこの宣言的なものすべてを好きではない理由です。必要なものにアクセスするのが難しくなります。
Sridhar Sarnobat

30

JUnit 4.13では次のことができます。

import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertThrows;

...

@Test
void exceptionTesting() {
  IllegalArgumentException exception = assertThrows(
    IllegalArgumentException.class, 
    () -> { throw new IllegalArgumentException("a message"); }
  );

  assertEquals("a message", exception.getMessage());
}

これはJUnit 5でも機能しますが、インポートは異なります。

import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertThrows;

...

このソリューションのように。JUnit 5に移動する必要があります
。– WesternGun

ガァァァァァァァ。4.13(2019年秋)の時点でまだベータ版ですか? mvnrepository.com/artifact/junit/junit
granadaCoder

v4.13はベータ版ではなくなりました(2020年1月にリリース)
Simon

11

実際、最適な使用法はtry / catchです。どうして?例外を予期する場所を制御できるからです。

この例を考えてみましょう:

@Test (expected = RuntimeException.class)
public void someTest() {
   // test preparation
   // actual test
}

ある日コードが変更され、テストの準備がRuntimeExceptionをスローするとどうなりますか?その場合、実際のテストはテストされず、例外がスローされなくてもテストはパスします。

そのため、アノテーションに依存するよりも、try / catchを使用する方がはるかに優れています。


悲しいことに、これも私の答えです。
Sridhar Sarnobat 2016

2
コードの変更に関する懸念は、小さな順列固有のテストケースを用意することで緩和されます。場合によっては避けられないことがあり、catch / tryメソッドに依存する必要がありますが、それが頻繁に発生する場合は、テストケース関数の記述方法を変更する必要がある可能性があります。
luis.espinal

これはテストやコードに問題があります。一般的なRuntimeException、特定の例外、または少なくとも特定のメッセージを予期していません。
DennisK 2016年

使った RuntimeExceptionは例として、この例外を他の例外に置き換えます。
Krzysztof Cislo 16

8

レイストームは良い答えを出しました。私もルールの大ファンではありません。読みやすさと使いやすさを支援するために次のユーティリティクラスを作成することを除いて、私は同様のことをします。これは、そもそも注釈の大きなプラスの1つです。

このユーティリティクラスを追加します。

import org.junit.Assert;

public abstract class ExpectedRuntimeExceptionAsserter {

    private String expectedExceptionMessage;

    public ExpectedRuntimeExceptionAsserter(String expectedExceptionMessage) {
        this.expectedExceptionMessage = expectedExceptionMessage;
    }

    public final void run(){
        try{
            expectException();
            Assert.fail(String.format("Expected a RuntimeException '%s'", expectedExceptionMessage));
        } catch (RuntimeException e){
            Assert.assertEquals("RuntimeException caught, but unexpected message", expectedExceptionMessage, e.getMessage());
        }
    }

    protected abstract void expectException();

}

次に、単体テストで必要なのは次のコードだけです。

@Test
public void verifyAnonymousUserCantAccessPrivilegedResourceTest(){
    new ExpectedRuntimeExceptionAsserter("anonymous user can't access privileged resource"){
        @Override
        protected void expectException() {
            throw new RuntimeException("anonymous user can't access privileged resource");
        }
    }.run(); //passes test; expected exception is caught, and this @Test returns normally as "Passed"
}

2

@Ruleを使用する場合、例外セットはTestクラスのすべてのテストメソッドに適用されます。


2
Jesse Merriman応答を使用すると、expectedEx.expect()およびexpectedEx.expectMessage()を呼び出すテストメソッドでのみ例外がチェックされます。他のメソッドは、expectedEx = ExpectedException.none()の定義を使用します。つまり、予期される例外はありません。
Egl

2

Junitで例外をアサートする方法は好きではありませんでした。アノテーションで「expected」を使用する場合、「then」がテスト定義の上部に配置されているため、私の観点からは、「given、when then then」パターンに違反しているようです。

また、 "@ Rule"を使用する場合、多くの定型コードを処理する必要があります。したがって、テスト用に新しいライブラリをインストールできる場合は、AssertJを確認することをお勧めします(そのライブラリにはSpringBootが付属しています)。

次に、「指定された/いつ/その後」の原則に違反していないテストを行い、AssertJを使用して確認します。

1-例外は私たちが期待しているものです。2-予想されるメッセージもあります

このようになります:

 @Test
void should_throwIllegalUse_when_idNotGiven() {

    //when
    final Throwable raisedException = catchThrowable(() -> getUserDAO.byId(null));

    //then
    assertThat(raisedException).isInstanceOf(IllegalArgumentException.class)
            .hasMessageContaining("Id to fetch is mandatory");
}

1

私はuser64141の答えが好きですが、もっと一般化できることがわかりました。これが私の見解です。

public abstract class ExpectedThrowableAsserter implements Runnable {

    private final Class<? extends Throwable> throwableClass;
    private final String expectedExceptionMessage;

    protected ExpectedThrowableAsserter(Class<? extends Throwable> throwableClass, String expectedExceptionMessage) {
        this.throwableClass = throwableClass;
        this.expectedExceptionMessage = expectedExceptionMessage;
    }

    public final void run() {
        try {
            expectException();
        } catch (Throwable e) {
            assertTrue(String.format("Caught unexpected %s", e.getClass().getSimpleName()), throwableClass.isInstance(e));
            assertEquals(String.format("%s caught, but unexpected message", throwableClass.getSimpleName()), expectedExceptionMessage, e.getMessage());
            return;
        }
        fail(String.format("Expected %s, but no exception was thrown.", throwableClass.getSimpleName()));
    }

    protected abstract void expectException();

}

「失敗」ステートメントをtryブロック内に残すと、関連するアサーション例外がキャッチされることに注意してください。catchステートメント内でreturnを使用すると、これを防ぐことができます。


0

キャッチ例外ライブラリをインポートして使用します。それはよりもはるかにきれいだExpectedExceptionルールまたはAtry-catch。。

ドキュメントの例:

import static com.googlecode.catchexception.CatchException.*;
import static com.googlecode.catchexception.apis.CatchExceptionHamcrestMatchers.*;

// given: an empty list
List myList = new ArrayList();

// when: we try to get the first element of the list
catchException(myList).get(1);

// then: we expect an IndexOutOfBoundsException with message "Index: 1, Size: 0"
assertThat(caughtException(),
  allOf(
    instanceOf(IndexOutOfBoundsException.class),
    hasMessage("Index: 1, Size: 0"),
    hasNoCause()
  )
);

-2
@Test (expectedExceptions = ValidationException.class, expectedExceptionsMessageRegExp = "This is not allowed")
public void testInvalidValidation() throws Exception{
     //test code
}

誰かがこの答えが-1である理由を理解するのを手伝ってくれませんか
aasha

質問は求めてJunitいますが、TestNG
Huazhe Yin
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.