11

Предположим у меня есть следующий класс

public class Residue
{
    public int Id {get;set;}
    public int WarehouseId {get;set;}
    public int MaterialAssetId {get;set;}

    public virtual ICollection<ResidueHistory> Histories {get;set;}

    public Residue()
    {
        this.Histories = new List<ResidueHistory>();
    }
}

Мне необходимо добавить метод который будет пересчитывать остатки с даты переданной в качестве параметра.

В тело класса добавляю новый метод

public void RecalculateResidueSince(DateTime since)
{
    throw new NotImplementedException();
}

Первый же тест который я решил реализовать это выход из метода если свойство Histories пустое;

Добавил в решение новый проект UnitTest:

[TestClass]
public class TestOfResidueMethods
{    
    [TestMethod]
    public void Test_RecalculateResidueForEmptyHistory()
    {        
        var residue = new Residue();
        residue.RecalculateResidueSince(DateTime.Now);
    }    
}

и тут у меня ступор а что делать дальше в тестовом методе, если бы это была функция то тогда я бы сверил то что получилось в результате с тем что ожидаю.

Подскажите как тестировать и надо ли писать такие тесты для таких методов?

UPD:

после написания этого метода я видоизменил тестируемый метод следующим образом:

public void RecalculateResidueSince(DateTime since)
{
    if(!this.Histories.Any())
        return;
}

т.е. если фактически остатков нет то соответственно ничего не делать, но не соображу как проверить это в тесте

9
  • 2
    Если у вас есть ничего не возвращающий функционал, то он либо не делает никаких изменений вообще, либо существует ради side effects. Соответственно вам и нужно проконтролировать появление ожидаемых side effects.
    – etki
    2 сен 2016 в 8:23
  • @Etki обновил вопрос
    – Bald
    2 сен 2016 в 8:26
  • @Bald: Комментарий @Etki всё ещё является ответом на вопрос. Вы не должны тестировать внутреннюю логику функции, только её результат.`
    – VladD
    2 сен 2016 в 8:31
  • @VladD в текущей реализации метод не делает изменений, в дальнейшем я смогу проверять результат на основании коллекции Histories, на текущем же этапе у меня есть два возможных варианта: в случае если Histories не содержит данные то тогда либо надо просто выйти из метода ничего не делая иначе выкинуть исключение. исключение мне кажется здесь не нужно так как остатков может не быть
    – Bald
    2 сен 2016 в 8:35
  • 1
    @Bald: Вы должны составить формальную спецификацию метода (что должно быть на входе, и что на выходе), и проверять именно её. Проверка того, как именно нужный эффект достигается, лишена смысла. Если Histories — внутренняя подробность имплементации, то и проверять эффекты, оказанные на неё, не надо. А если часть официального фасада класса, то наоборот надо.
    – VladD
    2 сен 2016 в 8:52

2 ответа 2

15

В тестах тестируются не возвращаемые значения и не реализация методов, а контракт. Контракт -- это некоторые обещания метода о том, что он сделает. Т.е. своеобразный набор пар "вход-выход". Если метод при этом принимает параметры, то количество этих обещаний (пар) может увеличиться и будет зависеть от комбинаций значений параметров.

При этом "выход" может быть разный:

  • в виде возвращаемого значения функции
  • в виде изменения состояния текущего объекта или других объектов, а также вызова методов в других объектах (зависимостях)

С первым типом контракта все понятно. Получил значение, проверил его.

Во втором типе контракт нужно проверять изменения состояния текущего объекта, а также, если у вас есть зависимости и в тестах вы их мокаете, обращения к этим зависимостям.


Контракт вашего метода заключается в следующем (из того, что вы описали в вопросе):

  • он пересчитывает остатки, начиная с переданной ему даты, при этом более ранние остатки не изменяются
  • если остатков нет, он ничего не делает

Соответственно ваши тесткейсы должны быть следующие:

  1. Пустая коллекция остатков. Вызвать метод и проверить, что состояние остатков не изменилось. Как более жесткий вариант: проверить, что состояние всего объекта не изменилось/не было обращений к зависмостям. (Но это уже на грани и попахивает паранойей, хотя в каких-то случаях может быть и оправдано.)
  2. Непустая коллекция остатков за разные даты. Вызвать метод за дату, которая больше максимальной даты в остатках. Проверить, что состояние остатков/всего объекта не изменилось.
  3. Непустая коллекция остатков. Вызвать метод за дату, которая больше минимальной, но меньше максимальной даты в остатках. Проверить, что остатки с меньшей датой не изменились и что остатки с большей датой изменились корректно.
7
  • т.е. в данном случае после вызова метода мне достаточно проверить что коллекция Histories не изменилась?!
    – Bald
    2 сен 2016 в 8:39
  • @Bald да. Как более жесткий вариант -- проверить, что состояние всего объекта не изменилось и не были вызваны зависимости (если они у вас есть и вы их мокаете).
    – andreycha
    2 сен 2016 в 8:41
  • 3
    @Bald, если у тебя входные данные 2 и 2 и на выходе должно получиться 4, то тебе не важно будет внутри метода 2+2 или 2*2
    – Grundy
    2 сен 2016 в 9:17
  • 1
    @Bald нет, не то же самое. Один и тот же результат может быть достигнут несколькими реализациями. Реализация -- это внутренность класса. Контракт -- это то, что торчит наружу. Мы не должны при тестировании лезть во внутренности, должны пользоваться тем, что доступно снаружи.
    – andreycha
    2 сен 2016 в 9:19
  • 4
    после комментария @Grundy я понял о чем речь: под проверкой реализации я подразумевал что выходные данные соответствуют тому что ожидается, а не то как это делается
    – Bald
    2 сен 2016 в 9:23
3

Тут можно проверять две вещи:

  1. ожидаемое состояние (своего класса или параметра (мокабельно), который внутрь передается)
  2. выбрасываемые исключения (только негативный сценарий)
3
  • обновил вопрос, добавил начальную реализацию метода. в данном случае я не хочу бросать исключение так как отсутствие остатков в данном случае ожидаемо и я хочу просто выйти из метода, и если не сложно то как можно проверить в методе выбрасываемое методом исключение
    – Bald
    2 сен 2016 в 8:28
  • @Bald Читайте доку по вашему тест-сьюту. Это может быть в виде аттрибута, вроде [ExpectedException(typeof(ArgumentException))]
    – free_ze
    2 сен 2016 в 8:42
  • огромное спасибо за подсказку что возвращаемое исключение может быть определено атрибутом
    – Bald
    2 сен 2016 в 8:49

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.