Sonar に怒られました

Only one method invocation is expected when testing runtime exceptions java:S5778

どういう意味？

JUnitのテストメソッド内で例外が発生することを検証する場合の非推奨な実装だと怒られています。実装はこんな感じ

    @Test
    @DisplayName("API実行時、期待する Exception がスローされること")
    void getException() {
        // テスト実行・結果確認
        var exception =
                assertThrows(WalletException.class,
                        () -> testTarget.getException(
                                new HttpClientErrorException(
                                        HttpStatus.NOT_FOUND
                                        , "Not Found"
                                        , "{\"message\":\"Error message\"}".getBytes()
                                        , CHARSET)
                                ));
    }
}

非推奨のコード例では、getException() や new Exception() のように複数のメソッド呼び出しがテストコード内に含まれています。この場合、例外が発生するのはgetException()なのか、new Exception()なのか明示的にはわかりません。複数のメソッド呼び出しが含まれると、テストの明確性が低下し、意図しないメソッドが例外をスローする場合に誤ったテスト結果になる可能性があります。

解決策

コードが実行時例外を発生させることを検証する場合、テストされるコード内部で複数のメソッドコールを行わないようにし、どのメソッドコールが例外を発生させるのかを明示するのが良い方法です。

    @Test
    @DisplayName("API実行時、期待する Exception がスローされること")
    void getException() {
        // テスト準備
        var httpException = new HttpClientErrorException(
                HttpStatus.NOT_FOUND, "Not Found", "{\"message\":\"Error message\"}".getBytes(), CHARSET);
        // テスト実行／結果確認
        assertThrows(WalletException.class, () -> testTarget(httpException));
    }
}