例外テストの書き方について

Question

前提・実現したいこと

例外テストの記述方法について教えてください。

様々なインプットに対して、対応した例外を投げるテストを作成したいと考えています。

インプットと期待例外のパターンの組み合わせ次第で、書き方としてはいくつか方法があると思いますが、
試みて失敗したものを記載しますので、解決案を教えて頂ければ助かります。

テストメソッドにはinvokeを使用しており、できるだけ@ParameterizedTestで、記述したいと考えています。

よろしくお願いします。

発生している問題・エラーメッセージ

エラーメッセージ

該当のソースコード

1. 異なるインプットに対して、全て同じ例外を返す場合
> テストメソッド:@ParameterizedTest宣言し、dataProvider部に'テストデータ'を記述する
（メソッド引数：テストデータを渡す）

→[問題] @Test(expected = xxException.class)のように、@ParameterizedTest宣言できない。

@ParameterizedTest(xxx)
testMethod(testData){
　　　　〜
       xxx.invoke();
}


2. 異なるインプットに対して、対応した例外を返す場合（例外パターンは複数）
> テストメソッド:@Rule、@ParameterizedTest宣言し、dataProvider部に'テストデータ'と期待exceptionを記述する
（メソッド引数：'テストデータ'と'期待exception.class'を渡す）

→[問題] Ruleが機能していない。expected exception

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

@ParameterizedTest(xxx)
testMethod(testData, zzz.class){
    thrown.expect(zzz.class);
    〜
    xxx.invoke();
}

試したこと

try-catchの記述方法は、動作確認できております。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

これがテストとして書きやすいかどうかはわからないですが、とりあえずこんな感じではできそうです。

Calculatorクラス

public class Calculator {
  private int exception_div(Integer first, Integer second) {
    return first / second;
  }
}

CalculatorTestクラス

class CalculatorTests {

  static Stream<Arguments> multiArgumentsProvider() {
    return Stream.of(Arguments.of(null, 2, 50, NullPointerException.class),
        Arguments.of(100, 0, -1, ArithmeticException.class),
        // 3番目のケースでは例外が起きないのでテスト失敗します
        Arguments.of(100, 2, 50, ArithmeticException.class));
  }


  @ParameterizedTest(name = "{0} / {1} = {2}")
  @MethodSource("multiArgumentsProvider")
  void div(Integer first, Integer second, int expectedResult, Class<Exception> expectedException)　 throws NoSuchMethodException, SecurityException {

    Calculator calculator = new Calculator();
    Method method = Calculator.class.getDeclaredMethod("exception_div", Integer.class, Integer.class);
    method.setAccessible(true);

    InvocationTargetException exception = assertThrows(InvocationTargetException.class, () -> {
      method.invoke(calculator, first, second);
    });
    assertEquals(exception.getTargetException().getClass().toString(), expectedException.toString());
  }

}

Answer 2

tenpuさんの希望されているのはこんな感じでしょうか?
JUnit5での記述です。

import java.util.stream.Stream;

import org.junit.jupiter.api.Assertions;
import org.junit.jupiter.params.ParameterizedTest;
import org.junit.jupiter.params.provider.Arguments;
import org.junit.jupiter.params.provider.MethodSource;

class CalculatorTests {

  static Stream<Arguments> multiArgumentsProvider() {
    return Stream.of(
        Arguments.of(null, 2, 50, NullPointerException.class),
        Arguments.of(100, 0, -1, ArithmeticException.class),
        // 3番目のケースでは例外が起きないのでテスト失敗します
        Arguments.of(100, 2, 50, ArithmeticException.class)
    );
  }
  
  @ParameterizedTest(name = "{0} / {1} = {2}")
  @MethodSource("multiArgumentsProvider")
  void div(Integer first, Integer second, int expectedResult, Class<Exception> expectedException) {
    Assertions.assertThrows(expectedException, () -> {
        System.out.println(first / second);
      });
  }
}

Comments

[tenpu]

ありがとうございます。
assertThrowsは良さそうな感じがしますが、 () -> {)部がreflectionのinvoke()先でエラーになった場合は、テスト対象メソッドで発生したexceptionではないラップされたexceptionが返ってくることになり、assert期待値のexceptionではなくなると思います。

これに関しては、どのように対応すれば良いでしょうか？

[euledge]

テストを行うのは 例外を発生するメソッドの挙動ですよね。 運用レベルで抽象化するためにリフレクションするのはいいと思うのですが、例外を発生するメソッドをテストするときには実体でよいのではないかと思います。
リフレクション出来ているかどうかはまた別のテストですればよいのかと思いますがどうでしょう？
ひとつのテストで全部やろうとしないほうがシンプルに捉えられると思います。

[tenpu]

はい。
テストメソッドがprivateなので、reflectionを使用しているのですが、他にやり方はありますでしょうか？