Проверка вызова внешней библиотеки в MiniTest

Question

Я работаю над приложением для блога с использованием Ruby on Rails. У меня есть модель под названием «Эссе» с декоратором Дрейпера. Я также использую MiniTest :: Spec для тестирования этого приложения. В каждом эссе есть тело, которое будет храниться как Markdown. В EssayDecorator у меня есть метод, называемый телом, который отображает Markdown в html с помощью RedCarpet.

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

describe '#body' do 
    it 'returns html from the markdown' do 
    essay = FactoryGirl.create(:essay) 
    @decorated_essay = essay.decorate 
    markdown = Minitest::Mock.new 

    @decorated_essay.stub :markdown, markdown do 
     markdown.expect :render, "<p>Test</p>", [essay.body] 
     @decorated_essay.send(:body) 
     markdown.verify 
    end 
    end 
end 

И внутри декоратора У меня есть два метода:

def body 
    markdown.render(model.body).html_safe 
    end 

    def markdown 
    Redcarpet::Markdown.new(Redcarpet::Render::HTML, :autolink => true, :space_after_headers => true) 
    end 

Этот тест проходит, но кажется странным для меня. Я не хочу проверять, что RedCarpet выполняет свою работу, я просто хочу проверить, что я вызываю метод render.

Есть ли лучшая практика для издевательства такого рода вещей в MiniTest? Я довольно новичок в использовании Mocks и очень новичок в использовании MiniTest.

Спасибо, заранее.

Answer 1

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

Я не хочу проверять, что RedCarpet выполняет свою работу, я просто хочу проверить, что вызываю метод визуализации.

Это реализация. Почему вы управляете телом через уценку? Потому что вы хотите создавать гиперссылки из URL-адресов? Я бы создал для этого тест. Вы хотите, чтобы ссылки имели атрибут no-follow? Я бы создал для этого тест. Создайте тесты для того, почему код что-то делает, а не как он это делает.

Возможно также, что в приложении отсутствует абстракция. Что-то, что отвечает за форматирование простого текста в HTML. То, что использует Redcarpet или RDiscount или любую другую библиотеку, считается важным. Возможно, EssayDecorator не несет ответственности за то, что текст был отформатирован правильно, но он может нести ответственность за указание правильному объекту применить форматирование.