Единичный тест не будет «покрывать» простой метод получения. (C#)

Question

У меня есть простой метод «Получить». Пример:

public class Foo : IFoo 
{ 
    public Dictionary<string,string> GetSomething(string xyz) 
    { 
     var result = new Dictionary<string,string> 
     ... Go to DB and return some key value pairs 
     return result; 
    } 
} 

Я написал простой тест, который выполняется и успешно проходит, но я не получаю покрытие кода по методу.

[TestMethod()] 
    public void GetSomething() 
    { 
     var target = new StubIFoo(); 

     var expected = new Dictionary<string, string> 
     { 
      {"blahKey","blahValue"} 
     }; 

     var results = target.GetSomethingString = s => 
     { 
      var result = new Dictionary<string, string> {{"a", "b"}}; 
      return result; 
     }; 

     var actual = results("a"); 

     CollectionAssert.AreEqual(expected,actual); 
    } 

Я также попытался предназначаться самого класса, который обеспечивает покрытие, но не возвращает никаких результатов (например: «вар цель = новый StubFoo();»)

Опять же, он успешно выполняет и проходит, но я не получаю никакого покрытия. Любые указатели будут оценены. Благодарю.

Answer 1

В вашем тесте вы не вызываете метод GetSomething, но вместо этого устанавливаете значение для свойства GetSomethingString.

// Here is the problem. This: 
var results = target.GetSomethingString = s => 
{ 
    var result = new Dictionary<string, string> {{"a", "b"}}; 
    return result; 
}; 
// Is equal to this: 
var lambda = s => 
{ 
    var result = new Dictionary<string, string> {{"a", "b"}}; 
    return result; 
}; 
var results2 = target.GetSomethingString = lambda; 

Answer 2

Foo.GetSomething() имеет 0 охват кода, потому что вы никогда не называете его в своем тесте. Вместо этого вы вызываете StubIFoo.GetSomething().

Измените var target = new StubIFoo(); на var target = new Foo();, удалите код, инициализирующий StubIFoo, и вы получите некоторое покрытие.

Заготовки для предотвращения использования (и тестирования) настоящего класса. Но вы не должны использовать заглушку класса, который вы тестируете!

Answer 3

Вот общая идея, что вы должны сделать

public class Foo : IFoo 
{ 
    IDbAccess db; 

    // Pass the interface, do pass parameters to create it inside constructor 
    public Foo(IDbAccess db) 
    { 
     this.db = db; 
    } 

    public Dictionary<string,string> GetSomething(string xyz) 
    { 
     var result = new Dictionary<string,string> 

     // ... Go to DB and return some key value pairs 
     result.Add(db.MethodWhatever(xyz)); 

     return result; 
    } 
} 

[TestMethod()] 
public void GetSomething() 
{ 
    var dbMock = new DatabaseMock(); // This implements IDbAccess 
    var target = new Foo(dbMock); 

    var expected = new Dictionary<string, string> 
    { 
     {"blahKey","blahValue"} 
    }; 

    // get something 
    var results = target.GetSomething("xyzstring"); 

    // verify results 
    var actual = results.whatever; 

    CollectionAssert.AreEqual(expected,actual); 
}