对 Viewmodel 进行单元测试

Question

我对 TDD 有点陌生。我已经开始在视图模型上创建我需要的属性作为普通的自动属性。

public string Firstname { get; set; }

然后我创建一个测试

[TestMethod]
[Tag("Property")]
public void FirstNameTest()
{
    ViewModel = new CustomerViewModel();
    ViewModel.PropertyChanged += (s, e) =>
                                     {
                                         Assert.AreEqual("Firstname", e.PropertyName);
                                         Assert.AreEqual("Test", ViewModel.Firstname);
                                     };
    ViewModel.Firstname = "Test";
}

然后我将扩展实际实现以使测试通过，如下所示：

public string Firstname
{
    get { return _contact.FirstName; }
    set
    {
        if (_contact.FirstName == value)
            return;

        _contact.FirstName = value;

        RaisePropertyChanged(() => Firstname);
    }
}

我遇到的问题是 Aut 属性的测试仍然通过。对我有什么建议可以改进我的流程吗？

Answer 1

你可以这样做：

    [TestMethod]
    [Tag("Property")]
    public void FirstNameTest()
    {
        bool didFire = false;
        ViewModel = new CustomerViewModel();
        ViewModel.PropertyChanged += (s, e) =>
                                         {
                                             didFire = true;
                                             Assert.AreEqual("Firstname", e.PropertyName);
                                             Assert.AreEqual("Test", ViewModel.Firstname);
                                         };
        ViewModel.Firstname = "Test";
        Assert.IsTrue(didFire);
    }

Answer 2

您可以尝试将测试编写为异步的。考虑这种测试方法：

[TestMethod]
[Asynchronous]
public void TestMethod1()
{
    TestViewModel testViewModel = new TestViewModel();

    bool firstNameChanged = false;

    testViewModel.PropertyChanged +=
        (s, e) =>
            {
                if (e.PropertyName == "FirstName")
                {
                    firstNameChanged = true;
                }
            };

    EnqueueCallback(() => testViewModel.FirstName = "first name");
    EnqueueConditional(() => firstNameChanged == true);
    EnqueueTestComplete();
}

注意方法顶部的异步属性。这里有两个重要的方法：EnqueueCallback 和 EnqueueTestComplete。 EnqueueCallback 会将 lambda 表达式添加到队列中，并且测试方法将等待直到执行当前回调。在本例中，我们订阅 ViewModel 上的 PropertyChanged 事件，并在 FirstName 属性通知更改时将本地布尔变量设置为 true。然后我们将两个回调入队：一个用于设置 FirstName 属性，另一个用于断言本地布尔变量已更改值。最后，我们需要添加对 EnqueueTestComplete() 的调用，以便框架知道测试已经结束。

注意：为了获得 EnqueueCallback 和 EnqueueTestComplete，您需要从测试类上的 SilverlightTest 继承。您还需要导入 Microsoft.Silverlight.Testing 以获取 Asynchronous 属性。它应该看起来像这样：

using Microsoft.Silverlight.Testing;
using Microsoft.VisualStudio.TestTools.UnitTesting;

namespace Foo.Example.Test
{
    [TestClass]
    public class Tests : SilverlightTest
    {

        // ... tests go here
    }
}

Answer 3

您需要进行另一个测试来实际断言您的 PropertyChanged 甚至会触发。

当您进行该测试时，您的 auto 属性应该会失败，因为该事件永远不会触发。

Here's an example of how to do that in Moq

Answer 4

除非它正在测试的行为已经实现，否则测试应该失败。

为了在我上次尝试中测试属性更改通知，我创建了a helper class，它帮助我编写了这样的测试（它在 NUnit 中）

[Test]
public void NotifiesChangeIn_TogglePauseTooltip()
{
   var listener = new PropertyChangeListener(_mainViewModel);

   _mainViewModel.TogglePauseCommand.Execute(null);

   Assert.That(listener.HasReceivedChangeNotificationFor("TogglePauseTooltip"));
}

Answer 5

这是我过去的做法（我使用过 NUnit，所以可能有点不同）：

[Test]
public void ShouldNotifyListenersWhenFirstNameChanges()
{
    var propertiesChanged = new List<string>();

    ViewModel = new CustomerViewModel();
    ViewModel.PropertyChanged += (s, e) => propertiesChanged.Add(e.PropertyName);

    ViewModel.Firstname = "Test";

    Assert.Contains("Firstname", propertiesChanged);      
    Assert.AreEqual("Test", ViewModel.Firstname);  
}

如果不是Firstname，它还有一个很好的副作用是能够调试和计算出发生了什么变化。当您从其他字段计算多个字段时非常方便。您还可以查看代码中行为的其他方面：

[Test]
public void ShouldNotNotifyListenersWhenPropertiesAreNotChanged()
{
    var propertiesChanged = new List<string>();

    ViewModel = new CustomerViewModel();
    ViewModel.Firstname = "Test";

    ViewModel.PropertyChanged += (s, e) => propertiesChanged.Add(e.PropertyName);

    ViewModel.Firstname = "Test";

    Assert.AreEqual(0, propertiesChanged.Count); 
}

Answer 6

也许这段代码有更多的背景信息没有被披露，但我所看到的似乎不必要地复杂。何必纠结于RaisePropertyChanged 事件呢？设置好后检查属性即可。

[TestMethod]
[Tag("Property")]
public void FirstNameTest()
{
    var expected = "John";
    var sut = new CustomerViewModel();

    sut.Firstname = expected;

    Assert.AreEqual(expected, sut.Firstname);
}

这也将测试变成了真正的单元测试。