Rails 5 Rspec получават с ActionController::Params

Бихте могли да направите:

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, ще ви даде малка или никаква стойност.

Забиването може да ви даде бърз набор от тестове, който е изключително крехък поради тясното свързване и който позволява на много грешки да се промъкнат през пукнатините.

Справих се с това, като създадох в 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')

Не е много по-различно от това, което вече правите, но прави неудобната необходимост от това допълнително повикване малко по-семантично значима.

@max имаше добри предложения как да се избегне това напълно и съм съгласен, че се отказаха от хеш, за да обезсърчат използването им с хешове взаимозаменяемо.

Ако все пак искате да ги използвате като прост хак за по-сложни ситуации (например ако expect използвате a_hash_including), можете да опитате да използвате нещо подобно:

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