根據私有方法測試公共方法的方法

Question

我試圖將測試添加到遺留代碼，並且當我開始添加代碼時，我感覺到某些錯誤。

在下面的代碼，public方法RegisterChange呼籲兩個私有方法：

1. 獲取對象存儲
2. 存儲對象

public class ChangeService { 

    IRepository repository; 

    public ChangeService(IRepository repository){ 
     this.repository = repository; 
    } 

    public bool RegisterChange(int entityId){ 
     var entity = GetParsedEntity(entityId);  
     SaveEntity(entity); 
     return true; 
    } 

    private Entity GetParsedEntity(int id) { 
     var entity = repository.GetEntityById(id); 
     return new Entity{ Name = entity.Name }; 
    } 

    private void SaveEntity(Entity entity) { 
     repository.Save(Entity); 
    } 
} 

public class ChangeServiceFact(){ 

    [Fact] 
    public void When_valid_entity__Should_save_entity(){ 

     var mock = new Mock<IRepository>(); 
     var service = new ChangeService(mock.object); 

     var result = service.RegisterChange(0); 

     Assert.True(result); 
    } 
} 

所以，當林嘲笑庫，我不得不去檢查私有方法的代碼，以知道哪些操作來模擬。

我用這種方法看到的問題是，因爲代碼不僅測試測試主體（公共方法）而且測試私有方法，因此不清楚應該通過查看測試結果測試主題（公共方法）。

在稍後有人決定修改一個私有方法（如從GetParsedEntity拋出異常）時，測試將繼續正確傳遞，但由於此更改，客戶端代碼可能會失敗。

在這種特殊情況下，我使用C＃，XUnit和Moq，但我認爲更通用的測試問題。

Answer 1

我用這種方法看到的問題是，因爲代碼不僅測試測試主體（公共方法），而且測試私有方法，不清楚哪個應該是測試結果看着考試科目（公開的方法）。

的測試對象你提到有不知道它的全部合同沒有明顯的效果。這裏的全部合同是什麼？提到的公共方法和構造函數，它依賴於依賴。這是重要的依賴關係，與此依賴關係的交互是應該測試的。私有方法是（一如既往）實現細節 - 與單元測試無關。

話雖如此，讓我們回到合同。測試主題的實際合同是什麼（ChangeService方法）？要基於某個id從存儲庫中檢索對象，請創建不同的對象並將其保存在同一個存儲庫中。這是你的考驗。

[Fact] 
public void ChangeService_StoresNewEntityInRepository_BasedOnProvidedId() 
{ 
    const string ExpectedName = "some name"; 
    var otherEntity = new OtherEntity { Name = ExpectedName }; 
    var mock = new Mock<IRepository>(); 
    var service = new ChangeService(mock.object); 
    mock.Setup(m => m.GetEntityById(0)).Return(otherEntity); 

    service.RegisterChange(0); 

    mock.Verify(m => m.SaveEntity(It.Is<Entity>(e => e.Name == ExpectedName)); 
}