升级到 Rails 4，现在出现“参数数量错误（1 代表 2+）”错误

Question

我最近将我的应用程序从 Rails 3 升级到 Rails 4，并且正在尝试运行规范测试。我认为过去可以正常工作的代码（在我在这里之前）突然抛出一个错误。

错误：

1) Admin::ReviewsController while logged in #index should get index
     Failure/Error: stub_search("product")
     ArgumentError:
       wrong number of arguments (1 for 2+)
     # ./spec/support/searchkick_stub.rb:5:in `stub_search'
     # ./spec/controllers/admin/reviews_controller_spec.rb:8:in `block (3 levels) in <top (required)>'

这里是 searchkick_stub.rb：

module SearchkickStub
  def stub_search(model)
    klass = model.to_s.camelize.constantize
    klass.any_instance.stub(:reindex) { true }
    klass.stub(:search) do |term, options|
      options ||= {}
      @search_term ||= term
      @search_params ||= options.dup

      response = {
    'hits' => {
          'total' => 0,
          'hits' => [],
    }
      }
      result_opts = {
    # per Searchkick::Query.new
    page: [options[:page].to_i, 1].max,
    per_page: (options[:limit] || options[:per_page] || 100000).to_i,
    padding: [options[:padding].to_i, 0].max,
    load: options[:load].nil? ? true : options[:load],
    # per Searchkick::Query.execute
    includes: options[:include] || options[:includes],
    json: !options[:json].nil?,
      }
      Searchkick::Results.new(klass, response, result_opts)
    end

    # Code that reindexes Products will reindex their Inventories too.
    stub_search(:inventory) if model == :product
  end
end

stub_search 的签名显然是针对单个参数的，而不是像错误声明那样的两个或更多。

这是我们在 reviews_controller_spec.rb 中使用 stub_search 的地方

describe ReviewsController do
  include SearchkickStub

  before do
    stub_search(:product)
    ...
  end
end

Answer 1

想通了。根据https://github.com/rspec/rspec-rails/issues/941，问题在于：spec_helper.rb 中的require 'minitest/autorun'。添加此行以删除以下警告：

Warning: you should require 'minitest/autorun' instead.
Warning: or add 'gem "minitest"' before 'require "minitest/autorun"'

但事实证明，您只需要 Gemfile 中的gem "minitest"（即使它已经安装，作为其他东西的依赖项，并且出现在 Gemfile.lock 中）。

Answer 2

我认为问题更多来自 ruby​​ 升级。关于如何处理块参数，解释器可能已经发生了变化。搜索方法在您的代码中采用 2 个参数：term 和 options。但它只用一个参数调用："product"。

options 在该块的第一行使用options ||= {} 设置为默认值，因此在 1.9.3 中不传递选项可能不是问题，但通过更严格的参数检查，它在 2.1.5 中会中断。

---

一个简单的解决方法是在块参数中设置一个默认参数，例如。

klass.stub(:search) do |term, options|

到

klass.stub(:search) do |term, options={}|

您也可以在执行此操作后安全地删除 options ||= {} 行。