はユニットテストのアプローチが悪いですか？

Question

私はクラスをテストしていますが、これは他のクラスからかなりの依存関係があります。これを解決するために私はモックを使います。これまでのところうまくいきました。

[Test] 
public void DataField_Gets_Properly_Created() 
{ 
    // Arrange all those mocks 

    _dataField = new DataField(dependency1, dependency2, dependency3); 

    Assert.NotNull(dataField.Id); 
    // other assertions ... 
} 

[Test] 
public void DataField_Gets_Properly_Saved() 
{ 
    var entityList = new List<IEntity>(); 
    var dfId = Guid.NewGuid(); 

    _dataField.SetValue(true, entityList, dfId); 

    Assert.True(_datenFeld.ValueBoolean); 
    // other assertions 
} 

ご覧のとおり、この専用の_dataField変数を使用します。ご存じのように、私は実際に、最初のテストでデータフィールドを作成できるかどうかをテストします。最初の方法で作成した2番目のテストで同じデータフィールドを使用するのは悪い方法ですか？私はこれが非常にクリーンなコードではないと思います。

私が持っていた2番目のアイデアは、このデータフィールドを[Setup]に作成することでした。これは非常にきれいではありません。なぜなら、それが作成されたとしても、後でセットアップやテストで作成する必要があるからです。よくわかりません。

その後、実際のクラスをテストするためにデータフィールドのインスタンスが必要なさらに多くのテストが行​​われる可能性があります。DataField

Answer 1

ユニットテストは、他のユニットテストに依存すべきではありません。それぞれを個別に実行してパスする必要があり、任意の順序で実行できるはずです。

この場合、各テストで再初期化するローカル変数を使用することをお勧めしますが、すべてのモックは[Setup]に配置してください。初期化はヘルパーメソッドCreateDataFieldForTest()などでカプセル化できます。

Answer 2

あなたのユニットテストは、他のユニットテストに依存すべきではありません。

[Setup]の機能は、より前に呼び出され、[Test]がクラス内で機能します。したがって、_dataFieldの作成を[Setup]に移動すると、それは継続的に再作成されます。それはあなたが確信していることですか？

[TestFixtureSetUp]と表示されている機能は、[TestFixure]の先頭で一度呼び出されます。だから、その適切に「初期化」ではない場合はここで最初のテストは失敗しますすべての[Test]

[SetUp] 
public void TestSetup() 
{ 
    _dataField = new DataField(dependency1, dependency2, dependency3); 
} 

[Test] 
public void DataField_Gets_Properly_Created() 
{ 
    Assert.NotNull(dataField.Id); 
    // other assertions ... 
} 

[Test] 
public void Most_Test_Using_dataField() 
{ 
    ... 
} 

前に呼び出すことが文句を言いません。この場合、問題があることを知り、他のテストがうまくいくことを保証することはできません。