代码应该在单元测试还是集成测试中

Question

首先，我是测试新手，所以这可能是一个愚蠢的问题。我目前正在为我的课程创建单元测试。我有一个属性，它依赖于可为空的属性将向数据库发送一个新的 Comment 对象。通常通过单元测试我会确保一个对象被发送到我的模拟服务，并称之为好。但是，测试 Comment Object 上的值以确保它沿着正确的路径前进，而不是仅仅假设它是不是更有意义。这是我正在测试的代码示例：

if (DeliveryDate != null)
{
   AddPartHistory("Delivery Date Changed from " + ((DateTime)DeliveryDate).ToShortDateString() + " to " + ((DateTime)value).ToShortDateString());
}
else
{
   AddPartHistory("Delivered Date of " + ((DateTime)value).ToShortDateString() + " was added.");
}

AddPartHistory 函数将 Comment 对象（将文本保存在名为 Entry 的属性中）发送到数据库（或在测试期间发送到模拟服务），并将其存储在名为 NewPartHistory 的属性中。以下是我认为可能更像是集成测试的代码：

vm.DeliveryDate = DateTime.UtcNow;
Assert.AreEqual("Delivered Date of " + ((DateTime)vm.DeliveryDate).ToShortDateString() + " was added.", vm.NewPartHistory.Entry);

OldDeliveryDate = vm.DeliveryDate;
vm.DeliveryDate = DateTime.UtcNow;
Assert.AreEqual("Delivery Date Changed from " + ((DateTime)OldDeliveryDate).ToShortDateString() + " to " + ((DateTime)vm.DeliveryDate).ToShortDateString(), vm.NewPartHistory.Entry);

那么，回到这个问题，我应该把这段代码留在单元测试中，还是转移到集成测试中。

更新：

由于有很多关于我的AddPartHistory 方法的讨论，所以在这里。它只是为PartHistory 填写标准数据（始终相同），添加条目，然后用新数据更新Listview：

private void AddPartHistory(string historyText)
{
   NewPartHistory = new CdaService.PartHistory();
   NewPartHistory.EnteredBy = User.Current.UID;
   NewPartHistory.Entry = historyText;
   NewPartHistory.EntryDate = DateTime.UtcNow;
   NewPartHistory.PartId = ThisPart.Id;
   webService.Insert(NewPartHistory);
   GetPartHistory();
}

Answer 1

我会将其更改为将新的 Comment 对象传递给 AddPartHistory 方法 - 而不是传递值来构建对象。这样您就可以对包括注释对象的构造在内的逻辑进行单元测试。从 AddPartHistory 返回评论对象以使其更容易断言也会很有帮助。

像这样把它移出方法：

   NewPartHistory = new CdaService.PartHistory();
   NewPartHistory.EnteredBy = User.Current.UID;
   NewPartHistory.EntryDate = DateTime.UtcNow;
   NewPartHistory.PartId = ThisPart.Id;

   if()
   {  
      NewPartHistory.Entry = "Delivered Date of"......;
      return AddPartHistory(NewPartHistory );
   }
   else
   {
       NewPartHistory.Entry = "Delivery Date Changed from".....;
       return AddPartHistory(NewPartHistory );
   }

   //return comment object from AddPartHistory so that you can call this entire method and assert all properties