Assert.assertTrue（true）を使用しています。 Assert.fail（）をアサートします。 JUnitテストの良い練習ですか？

Question

Assert.assertTrue（true）を使うのが良いか悪いのですか。 JUnitテストのAssert.fail（）とその理由

@Test（expected = Exception.class）というルールもありますが、テスト済みのメソッドで発生させることができる複数の例外を持つことができます。これはオプションである可能性があります。ゼロ、一つ以上の例外を発生させることができあなたの試験方法（例では、1つだけ、それを簡単にするために）、あなたは、コールがそうは例外は発生しませんしなければならないOKであった場合をテストしたい：

ユースケースはこれですcatchブロックに入ると、Assert.fail（）が呼び出されます。例外を発生させる必要がある場合はテストケースが必要なので、catchブロックをコールします。Assert.assertTrue（true）;言うまでもなく、「これは私が欲しいものです」そして、テストから、あなたが望むように実際に動作する場合、テストが失敗したときとテストを実行するときの1つのテスト方法で見ることができます。
質問です：この良いか悪い習慣のですか？どうして？

public void fooMethod(boolean paramenter) { 
    if(false) { 
    throw new Exception(); 
    } 
} 

@Test 
public void testFooMethod() { 

    try { 
    myTestedClassMock.fooMethod(true); //should not raise exception 
    } catch(Exception e) { 
    Assert.fail(); 
    } 

    try { 
    myTestedClassMock.fooMethod(false); //should raise exception 
    } catch (Exception e) { 
    Assert.assertTrue(true); 
    } 

} 

Answer 1

assertTrueは異なる目的のために設計されているので、いつでもAssert.assertTrue(true)を残すことができます。複数のブール項目を含む一般的な条件をチェックすることがあります。定数trueをアサート

Assert.assertTrue(t == null ||| minSize < 0 || t.size() > minSize); 

は常に跡形もなく成功したので、あなたは完全にあなたのコードからこの行を削除することができます。予想される例外はがスローされていないときの状況をキャッチすることはすぐに予想される例外がスローされていないことを示すメッセージと、例外をスローしなければならないテストコードの行の後Assert.fail();を置くことによって行われます。

テストされた1つのメソッドに対して膨大なテストメソッドが必要ない場合は、これがオプションになる可能性があります。

典型的には、「膨大な数の試験方法は、」より良いオプションがあります。別の方法として、あなたではなく、個々の例外をキャッチするよりも、あなたのテストにExceptionまたはRuntimeExceptionをキャッチすることができます。これはプロダクションコードでは疑わしい習慣ですが、例外がスローされないようにする必要がある場合は、テストコードでは問題ありません。

しかし、Assert.fail();を持つことは珍しくショートカットではない、それはあまりにも、通常は許容されます。障害の原因を説明するメッセージを追加することをお勧めします。その結果、コードを管理しているユーザーは、何が起きているのかをテストで確認する必要がありません。例外をキャッチすると、そのメッセージはテストの失敗メッセージの候補になります。

Answer 2

しかしあなたがそれを行うようにしないfail()使用することをお勧めします。これは、例外が発生した場合にテストを失敗させたいからです。これは、この例外をキャッチすることなく行うことができます。表示されている場合、テストは失敗します。代わりにassertTrue(true)を使用しての第2のケースで

は、あなたのメソッド呼び出しの後fail()を追加します。

myTestedClassMock.fooMethod(true); //should not raise exception 

try { 
    myTestedClassMock.fooMethod(false); //should raise exception 
    Assert.fail("Exception expected!"); 
} catch (Exception e) { 
    // exception handling 
} 

これはあなたの意図がより明確になり、あなたは例外がスローされない場合、テストが失敗したことがわかります。

Answer 3

第2のケースがありますか？ 私はむしろ、次のチェックだろう。

try { 
    myTestedClassMock.fooMethod(false); 
    Assert.fail("should raise exception"); 
    } catch (Exception e) { 
    //Check if this is the exception you expected 
    } 

例外がスローされます場合は、最初のケースについては、それはとにかくテストを失敗する何を、そうAssert.fail()とtry-catchブロック

Answer 4

一部をする必要はありませんあなたのソースコードに関する良い習慣。

最初：各テストメソッド内に1つのアサートしかありません。だから、その方法が失敗すると、あなたはすぐにどこを見るかを知っています。また、あなたの問題に役立ちます

：

@Test 
public void testFooMethodThatShouldNotThrow() throws WhatEverException { 
    bar.foo(true) 

のfoo（）は、上記のコードでスローのであれば、ユニットテストは失敗します。スロー句を追加しても問題ありません。

例外がスローされると予想されるとき。

@Test(expected=WhateverException.class) 
public void testFooMethodThatShouldThrow() { 
    bar.foo(false); 
} 

スローされた例外の特定のプロパティをチェックする場合は、次のようにします。あなたはそうするでしょう。

@Test 
public void bla { 
    try { 
    bar.foo(); 
    fail("should have thrown"); 
    catch (WhateverException e) { 
    assertThat(e.getMessage(), is("bla blub flubber")); 
    } 

たとえば、

最後に、正確に1つのアサートが必要です。assertThat。他のすべてについては忘れてください。それを使うことを学ぶ。