Блог

Я использую модуль декоратора, который включается в экземпляр модели (с помощью метода «extends»). Так, например :

 module Decorator
  def foo
  end
end

class Model < ActiveRecord::Base
end

class ModelsController < ApplicationController
  def bar
    @model = Model.find(params[:id])
    @model.extend(Decorator)
    @model.foo
  end
end
  

Затем я хотел бы в тестах выполнить следующее (используя Mocha) :

 test "bar" do
  Model.any_instance.expects(:foo).returns("bar")
  get :bar
end 
  

Возможно ли это каким-то образом, или вы имеете в виду какой-либо другой способ получить эту функциональность???

Просто предположение: я предполагаю, что ваш метод Decorator foo возвращает «bar», который не показан в отправленном вами коде. Если я не предполагаю этого, то ожидания в любом случае не будут выполнены, потому что метод возвращает nil, а не «bar».

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

Я пришел к такому выводу, пытаясь следовать стеку вызываемых методов во время ожидания. ожидает, что вызовет заглушки в Mocha::Central, который вызывает заглушки в Mocha::ClassMethod, который вызывает *hide_original_method * в Mocha::AnyInstanceMethod . Там *hide_original_method * не находит никакого метода для скрытия и ничего не делает. Затем моделируйте.метод foo не является псевдонимом для заглушенного метода mocha, который должен быть вызван для реализации вашего ожидания mocha, но фактической модели.вызывается метод foo, который вы динамически подключаете к экземпляру модели внутри вашего контроллера.

Мой ответ заключается в том, что это невозможно сделать.

Это работает (подтверждено в тестовом приложении с помощью render:text)

Я обычно включаю декораторы (вместо того, чтобы расширять их во время выполнения), и я избегаю any_instance, поскольку это считается плохой практикой (вместо этого я имитирую find).

 module Decorators
  module Test
    def foo
      "foo"
    end
  end
end

class MoufesController < ApplicationController

  def bar
    @moufa = Moufa.first
    @moufa.extend(Decorators::Test)
    render :text => @moufa.foo
  end
end

require 'test_helper'

class MoufesControllerTest < ActionController::TestCase
  # Replace this with your real tests.
  test "bar" do
    m = Moufa.first
    Moufa.expects(:find).returns(m)
    m.expects(:foo).returns("foobar")

    get :bar, {:id => 32}
    assert_equal @response.body, "foobar"
  end
end
  

Хорошо, теперь я понимаю. Вы хотите отключить вызов внешней службы. Интересно, что mocha не работает с extend таким образом. Помимо того, что упомянуто выше, похоже, это связано с тем, что заглушенные методы определены в одноэлементном классе, а не в модуле, так что не вмешивайтесь.

Почему бы не что-то подобное?

 test "bar" do
  Decorator = Module.new{ def foo; 'foo'; end }
  get :bar
end
  

Если вы предпочитаете не получать предупреждения о том, что декоратор уже определен — что является намеком на то, что в любом случае происходит какая-то связь — вы можете ввести его:

 class ModelsController < ApplicationController
  class << self
    attr_writer :decorator_class
    def decorator_class; @decorator_class ||= Decorator; end
  end

  def bar
    @model = Model.find(params[:id])
    @model.extend(self.class.decorator_class)
    @model.foo
  end
end
  

что делает тест похожим:

 test "bar" do
  dummy = Module.new{ def foo; 'foo'; end }
  ModelsController.decorator_class = dummy
  get :bar
end
  

Конечно, если у вас более сложная ситуация с несколькими декораторами или декораторами, определяющими несколько методов, это может не сработать для вас.

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

Одно незначительное изменение, если вы хотите проверить возвращаемое значение:bar —

 test "bar" do
  Model.any_instance.expects(:foo).returns("bar")
  assert_equal "bar", get(:bar)
end
  

Но если вы просто проверяете, что экземпляр модели имеет методы декоратора, вам действительно нужно это проверять? Похоже, в этом случае вы тестируете Object#extend .

Если вы хотите протестировать поведение @model.foo, вам не нужно делать это в интеграционном тесте — в этом преимущество декоратора, затем вы можете протестировать его изолированно, например

 x = Object.new.extend(Decorator)
#.... assert something about x.foo ...
  

По моему опыту, издевательство в интеграционных тестах обычно пахнет кодом.