Rails 5 Rspec 通过 ActionController::Params 接收

Question

我刚刚升级到 Rails 5。在我的规格中，我有以下内容

expect(model).to receive(:update).with(foo: 'bar')

但是，由于 params 不再扩展 Hash 而现在是 ActionController::Parameters，因此规范失败了，因为 with() 期待一个哈希值，但它实际上是 ActionController::Parameters

有没有更好的方法在 Rspec 中做同样的事情，比如不同的方法with_hash？

我可以使用

解决这个问题

expect(model).to receive(:update).with(hash_including(foo: 'bar'))

但这只是检查参数是否包含该哈希，而不是检查完全匹配。

Answer 1

你可以这样做：

params = ActionController::Parameters.new(foo: 'bar')
expect(model).to receive(:update).with(params)

但是它仍然有味道 - 您应该测试应用程序的行为 - 而不是它的工作方式。

expect {
  patch model_path(model), params: { foo: 'bar' }
  model.reload
}.to change(model, :foo).to('bar')

这就是我测试控制器集成的方式：

require 'rails_helper'
RSpec.describe "Things", type: :request do
  describe "PATCH /things/:id" do

    let!(:thing) { create(:thing) }
    let(:action) do
      patch things_path(thing), params: { thing: attributes }
    end

    context "with invalid params" do
      let(:attributes) { { name: '' } }
      it "does not alter the thing" do
         expect do 
           action 
           thing.reload
         end.to_not change(thing, :name)
         expect(response).to have_status :bad_entity
      end
    end

    context "with valid params" do
      let(:attributes) { { name: 'Foo' } }
       it "updates the thing" do
         expect do 
           action 
           thing.reload
         end.to change(thing, :name).to('Foo')
         expect(response).to be_successful
      end
    end
  end
end

在规范中接触数据库是否会很糟糕？

没有。当您测试控制器之类的东西时，最准确的测试方法是驱动整个堆栈。如果我们在这种情况下删除了@thing.update，我们可能会错过例如数据库驱动程序抛出错误，因为我们使用了错误的 SQL 语法。

例如，如果您要在模型上测试范围，那么剔除 DB 的规范几乎不会给您带来任何价值。

存根可能会为您提供一个快速测试套件，该套件由于紧密耦合而非常脆弱，并且会让大量错误从裂缝中溜走。

Answer 2

我通过在 spec/rails_helper.rb 中创建来处理这个

def strong_params(wimpy_params)
  ActionController::Parameters.new(wimpy_params).permit!
end

然后在特定的测试中，你可以说：

expect(model).to receive(:update).with(strong_params foo: 'bar')

这与您已经在做的并没有太大区别，但它使额外调用的尴尬必要性在语义上更有意义。

Answer 3

@max 对如何完全避免这种情况提出了很好的建议，我同意他们从散列切换到不鼓励将它们与散列互换使用。

但是，如果您仍然想使用它们，作为更复杂情况的简单 hack（例如，如果您 expect 使用 a_hash_including），您可以尝试使用类似这样的东西：

  .with( an_object_satisfying { |o| 
           o.slice(some_params) == ActionController::Parameters.new(some_params)
         })