PropertyChangedイベントテスト：これは良い方法ですか？

Question

MVVMパターンを使用してWPFアプリケーションを開発しています。

public bool EditModeEnabled 
{ 
    get { return _EditModeEnabled; } 
    set 
    { 
     _ModeEditModeEnabled = value; 
     OnPropertyChanged("EditModeEnabled"); 
     OnPropertyChanged("CommentTextBoxVisibility"); 
    } 
} 

OnPropertyChangedがちょうどPropertyChangedイベントを発生させる基底クラスの仮想メソッドです：私はこのようなコードでのViewModelを持っています。 私が上げPropertyChangedイベントをテストしたいとそこに私の試験方法：

public void EditModeEnabledTest() 
{ 
    var imageViewModel = TestHelper.GetTestImageViewModel(); 
    var firedEvents = new List<string>(); 
    imageViewModel.PropertyChanged += ((sender, e) => firedEvents.Add(e.PropertyName)); 
    imageViewModel.Mode = true; 
    Assert.AreEqual(firedEvents.Count, 2); 
    Assert.IsTrue(firedEvents.Contains("EditModeEnabled")); 
    Assert.IsTrue(firedEvents.Contains("CommentTextBoxVisibility")); 
    ... 
} 

はそれがProprtyChangedイベントをテストする良い方法ですか？

Answer 1

私はそれを正確に行うためにlittle Fluent APIを使用します。それはあなたがこのようにテストを書くことができます：

var imageViewModel = TestHelper.GetTestImageViewModel(); 
imageViewModel.ShouldNotifyOn(s => s.EditModeEnabled) 
    When(s => s.Mode = true); 

簡潔であることに加え、それはタイプセーフなので、私はこのアプローチを好む - 何の文字列値は、あなたのAPIと同期しておくんし。

複数のプロパティに対してイベントが発生していることをテストするには、これを行う別のテストを作成します。これは多くのテストを行いますが、それぞれが非常に小さくなり、Assertion Rouletteを回避します。

Answer 2

私はあなたが示した例でPropertyChangedイベントを単体テストすることをお勧めします。プロパティー名の文字列を間違って記述したため、更新が失われている可能性があります。

Person person = new Person(); 
AssertHelper.PropertyChangedEvent(person, x => x.Name,() => person.Name = "Luke");