Как сказать «any_instance» «should_receive» любое количество раз в RSpec - PullRequest
65 голосов
/ 21 марта 2012

У меня есть контроллер импорта в рельсах, который импортирует несколько файлов CSV с несколькими записями в мою базу данных. Я хотел бы проверить в RSpec, действительно ли записи сохраняются с использованием RSpec:

<Model>.any_instance.should_receive(:save).at_least(:once)

Однако я получаю сообщение об ошибке:

The message 'save' was received by <model instance> but has already been received by <another model instance>

Придуманный пример контроллера:

rows = CSV.parse(uploaded_file.tempfile, col_sep: "|")

  ActiveRecord::Base.transaction do
    rows.each do |row| 
    mutation = Mutation.new
    row.each_with_index do |value, index| 
      Mutation.send("#{attribute_order[index]}=", value)
    end
  mutation.save          
end

Можно ли это проверить с помощью RSpec или есть какое-то решение?

Ответы [ 6 ]

45 голосов
/ 28 июня 2014

Для этого есть новый синтаксис:

expect_any_instance_of(Model).to receive(:save).at_least(:once)
44 голосов
/ 03 апреля 2012

Вот лучший ответ, позволяющий избежать необходимости переопределять метод: new:

save_count = 0
<Model>.any_instance.stub(:save) do |arg|
    # The evaluation context is the rspec group instance,
    # arg are the arguments to the function. I can't see a
    # way to get the actual <Model> instance :(
    save_count+=1
end
.... run the test here ...
save_count.should > 0

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

Обновление - новая версия rspec требует следующий синтаксис:

save_count = 0
allow_any_instance_of(Model).to receive(:save) do |arg|
    # The evaluation context is the rspec group instance,
    # arg are the arguments to the function. I can't see a
    # way to get the actual <Model> instance :(
    save_count+=1
end
.... run the test here ...
save_count.should > 0
15 голосов
/ 27 марта 2012

Мне наконец удалось сделать тест, который работает для меня:

  mutation = FactoryGirl.build(:mutation)
  Mutation.stub(:new).and_return(mutation)
  mutation.should_receive(:save).at_least(:once)

Метод-заглушка возвращает один единственный экземпляр, который получает метод сохранения несколько раз. Поскольку это единственный экземпляр, я могу отбросить метод any_instance и нормально использовать метод at_least.

10 голосов
/ 23 февраля 2013

Заглушка вот так

User.stub(:save) # Could be any class method in any class
User.any_instance.stub(:save) { |*args| User.save(*args) }

Тогда ожидайте вот так:

# User.any_instance.should_receive(:save).at_least(:once)
User.should_receive(:save).at_least(:once)

Это упрощение этой сущности , чтобы использовать any_instance, так каквам не нужно прокси к оригинальному методу.Обратитесь к этой сути для других целей.

8 голосов
/ 14 октября 2015

Это пример Роба, использующего RSpec 3.3, который больше не поддерживает Foo.any_instance. Я нашел это полезным, когда в цикле создания объектов

# code (simplified version)
array_of_hashes.each { |hash| Model.new(hash).write! }

# spec
it "calls write! for each instance of Model" do 
  call_count = 0
  allow_any_instance_of(Model).to receive(:write!) { call_count += 1 }

  response.process # run the test
  expect(call_count).to eq(2)
end
2 голосов
/ 13 августа 2015

Мой случай был немного другим, но я остановился на этом вопросе и решил оставить здесь свой ответ. В моем случае я хотел заглушить любой экземпляр данного класса. Я получил ту же ошибку, когда использовал expect_any_instance_of(Model).to. Когда я изменил его на allow_any_instance_of(Model).to, моя проблема была решена.

Ознакомьтесь с документацией для получения дополнительной информации: https://github.com/rspec/rspec-mocks#settings-mocks-or-stubs-on-any-instance-of-a-class

...