如何将哈希方法 rspec 传递给控制器

Question

使用 Rails 5.1.4、Ruby 2.4.1、rspec

场景：

在文章中销毁只允许角色为“a,m”的用户 current_ma_user

然后：

检查 current_ma_user.role = "a,m" 或 current_ma_user 自己的文章 (@article.user)

所以我创建 current_ma_user 作为哈希以及用户。 然后调用role来检查什么是user[role]

问题：

1. 如何向哈希添加新方法。

2. 如何将该 hash.method 从 rspec controller_spec 传递给控制器​​。

失败：

  1) ArticlesController DELETE #destroy destroys the requested article
     Failure/Error: delete :destroy, params: {id: article.to_param}, session: valid_session, :current_ma_user.role => "a,m"

     NoMethodError:
       undefined method `role' for :current_ma_user:Symbol
     # ./spec/controllers/articles_controller_spec.rb:172:in `block (4 levels) in <top (required)>'
     # ./spec/controllers/articles_controller_spec.rb:171:in `block (3 levels) in <top (required)>'

这是gist

articles_controller_spec.rb：

require 'rails_helper'
RSpec.describe ArticlesController, type: :controller do

    class Hash #patch to temp pass problem 1
      def role
        "a,m"  #Hard Code, need to call user["role"] need code
      end
    end

    user = {}
    user["uid"] = "admin"
    user["provider"] = "Facebook"
    user["email"] = "1.0@kul.asia"
    user["role"] = "a,m"

    current_ma_user = user

  describe "DELETE #destroy" do

    it "destroys the requested article" do
      article = Article.create! valid_attributes
      expect {
        delete :destroy, params: {id: article.to_param}, session: valid_session
      }.to change(Article, :count).by(-1)
    end

    it "redirects to the articles list" do
      article = Article.create! valid_attributes
      delete :destroy, params: {id: article.to_param}, session: valid_session
      expect(response).to redirect_to(articles_url)
    end
  end
end

控制器：

class ArticlesController < ApplicationController   before_action :load_article, only: [:show, :destroy]

  def destroy
    if current_ma_user.role.upcase.split(',').include?("A") || current_ma_user == @article.user
    #if current_ma_user == @article.user
      @article.destroy
    end
      redirect_to :action=>'index'   end

  private

  def load_article
    @article = Article.find(params[:id])   end

end

更新了行号：

更新了调试以在 .spec 和控制器中显示 current_ma_user 的值

Answer 1

这是您的错误的来源（在您的控制器中）：

if current_ma_user.role.upcase.split(',').include?("A") || current_ma_user == @article.user

建议的解决方案

• current_ma_user 在控制器中定义在哪里？（如果没有赋值，则需要在调用 current_ma_user 变量的角色方法之前赋值。

试试看效果如何。

• 做这样的事情：

current_ma_user = User.find( params[:user_id])

• 现在您似乎想将某些内容传递到 params 哈希中。请记住将您决定传递给参数的任何内容列入白名单。无论是用户 ID 或角色 ID 等，还是角色字符串。

• 在编写测试时，将适当的值传递给 params 哈希。如果您在测试中传入user_id，则必须确保在测试中创建了用户。

delete :destroy, {:id => article.id.to_s, :user_id => @current_ma_user.id }, session: valid_session

• 也可能在您的规范文件中，在您的测试中，将 current_ma_user 放在一个前置过滤器中，并使其成为一个实例变量，以便您的所有测试都可以访问它：

  before(:each) do @current_ma_user = user.create( <--- create the user with the appropriate attributes here --->) end

警告：未经测试

我只是在堆栈溢出编辑器中输入了它。