3

想知道这是否是测试事件结果的正确方法。

我正在开发一个应用程序,当保存正在进行/完成时它会触发事件。

为了测试它,我想出了以下(虚构的场景)。我想知道这是否是你这样做的方式:

[Test]
public void Save_WhenCalled_IsSuccessfull()
{
    //Arrange
    var customerService= new CustomerService();

    customerService.OnSaved += (sender, args) =>
        {                                             
            Assert.IsTrue(args.HasSaved);
        };

    customerService.Save(new Customer {Id=1,Name="Jo"});
}

我不喜欢的是,如果您明白我的意思,我之前就在断言。

我希望断言在视觉上是最后的。顺便说一句,上面的工作很好,但不是很开心。

有什么建议么?

4

1 回答 1

5

看起来不错,但您应该将接收到的参数(或任何其他检查)存储在变量中,以保持排列/动作/断言顺序。这样,您还可以断言该事件已实际触发,您的示例未验证这一点。

[Test]
public void Save_WhenCalled_IsSuccessfull()
{
    //Arrange
    YourArgsType actualArgs;
    var customerService= new CustomerService();  
    customerService.OnSaved+= (sender, args) =>
                                  {                                           
                                      actualArgs = args;
                                  };

    //Act
    customerService.Save(new Customer{Id=1, Name="Jo"});

    //Assert
    Assert.IsTrue(actualArgs.HasSaved);
}

编辑:添加了亚历山大的建议。

于 2013-01-27T16:39:45.753 回答