Как написать тест, покрывающий 100% кода при двойной проверке условия?

Question

[Сергей Блохин]

Имеется некий код, в котором присутствует двойная проверка одного из условий.
Каким образом можно написать unit-тест так, чтобы он покрыл 100% кода?
Двойная проверка нужна в случае, если по какой-либо причине база данных отдаст результат, где price будет равен нулю.
Как вариант, написать заглушку (mock/stub) для метода работы с БД, который бы вернул товар с нулевой ценой?
Или есть иное (более правильное) решение?

def products
  # first check on zero price
  products = database.query 'SELECT * FROM product WHERE price > 0;'
  products.map do |product|

    # second check on zero price
    unless product[:price] > 0
      warn 'wrong price'
      next
    end

    # some operation under product
  end
end

Comments

[Сергей Блохин]

Егор Скороходов, я всецело верю, что база не отдаст результат, равный нулю.
Но, запрос может поменяться (ошибочно), и тогда дополнительная проверка точно не допустит нулевую цену.

Answer 1

[Павло Пономаренко]

Правильный подход в таком случае - разделить получение данных из базы и проверку на нулевую цену. Ну самый простой пример - создайте функцию ensure_that_every_goods_has_price(goods) и вызывайте её вместо того, чтобы включать цикл в тело.

Тогда вы сможете в тестах передать в неё набор товаров, среди которых будет нулевая цена и таким образом покрыть, а также переиспользовать её при необходимости. Я на руби не пишу, но псевдокод приблизительно такой:

def ensure_that_every_goods_has_price(products)
  products.map do |product|
    unless product[:price] > 0
      warn 'wrong price'
      next
    end
  end
end

def products
  products = database.query 'SELECT * FROM product WHERE price > 0;'
  ensure_that_every_goods_has_price(products)
end

## test:

products = database.query 'SELECT * FROM product WHERE price >= 0;'
should_warn ensure_that_every_goods_has_price(products)

Answer 2

[Alexej Simakov]

Лучше написать обработку ошибки запроса к базе. От этого действительно будет польза.
Например в указаной таблице нет желаемого поля, либо таблицу переименовали. Клиентский код перестанет работать.
В остальном можно смело доверять драйверу базы, что он работает правильно, если вы посылаете правильные запросы. Если просили с ценой больше нуля, то и получите с ценой больше нуля. Проверять это не нужно.

Comments

[Сергей Блохин]

Спасибо, вы подтвердили мои мысли.
Я не думаю, что драйвер БД может сломаться, но может измениться (ошибочно) запрос к БД.
Для исключения этой ошибки и была добавлена вторая проверка нулевой цены.