Question

我有一个页面dashboard.html.erb，可能会从几个不同的控制器/操作重定向到。当它收到重定向时，它还会收到几个url参数。

控制器代码：

class PlansController
  def some_action
    redirect_to dashboard_path(show: "default", cool_array: ["test", "test"])
  end
end

class StaticpagesController
  def dashboard
  end
end

查看代码：

<% if cool_array %>
  <% cool_array.each do |a| %>
    <li><%= a %></li>
  <% end %>
<% end %>

<script>
var show = getParamFromURL("show")
// some sort of function that returns the value of the show param, in this case, "default"
if (show == "default") {
  $("#default").addClass("show");
} else {
  $("#other").addClass("hidden");
}
</script>

由于这些是重定向，因此使用render_views进行控制器测试将无效。因此，我曾考虑构建我的测试，以便控制器规范测试正确的参数被传递，并且视图规范测试如果存在某些参数，则存在正确的css类，或者其他。但在我的观点规范中，我无法模拟参数的传递。

这似乎是我发现的唯一其他问题：RSpec View testing: How to modify params?

但是没有一个解决方案适合我;我不确定在这种情况下我需要一个帮手......这似乎有点人为......

我尝试过（来自上面链接的SO问题）：

before do
  controller.request.path_parameters[:cool_array] = ["test","test"]
end
# => ActionView::Template::Error:
#  undefined local variable or method `cool_array' for #<#<Class:0x007fe2d9815100>:0x007fe2d90a62e8>
# thrown by line in view file with <% if cool_array %>

before do
  view.stub(:params).and_return({cool_array: ["test", "test"]})
end
# => ActionView::Template::Error:
#  undefined local variable or method `cool_array' for #<#<Class:0x007fe2d9815100>:0x007fe2d90a62e8>
# thrown by line in view file with <% if cool_array %>

helper.params = {:cool_array => ["test", "test"]]}
# honestly not even sure this is right, because it can only go into a describe block (not a before, not an it) and then it throws:
# NoMethodError:
#   undefined method `params=' for #<#<Class:0x007fd7cc8f4930>:0x007fd7cc8a5060>

他们都失败了：

Answer 1

因为这些是重定向，所以使用render_views进行控制器测试不行。

不，你做错了。

如果StaticpagesController#dashboard接受url参数，你应该测试它如何响应StaticpagesController的控制器规范中的所述参数。

RSpec.describe StaticpagesController, type: :controller do 
  describe 'GET #dashboard' do
    render_views
    it "does something with the x parameter" do
       get :dashboard, x: 'hello!'
       expect(response.body).to match 'hello!'
    end
  end
end

同样，您应该测试PlansController＃some_action在响应中包含正确的参数。

我认为你的观点规格问题告诉你，你的观点中有代码味道。在MVC中，您的控制器负责获取输入并将其传递给视图和模型。将视图与直接处理参数隔离意味着处理它的参数不会在整个应用程序中重复。

在控制器中使用locals将变量传递到您的视图，避免在视图中使用params。

它有助于更好的设计和更简单的测试：

describe "rendering locals in a partial" do
  it "displays the widget" do
    widget = stub_model(Widget, :name => "slicer")
    render :partial => "widgets/widget.html.erb", :locals => {:widget => widget}
    rendered.should contain("slicer")
  end
end

使用url参数进行Rspec视图测试

1 个答案: