Rspec的控制器测试失败时零

Question

的Rails 3.2.6

我已经设置了联系控制器和形式这篇文章的建议： http://matharvard.ca/posts/2011/aug/22/contact-form-in-rails-3/

我Message模型类：

class Message 
    include ActiveModel::Validations 
    include ActiveModel::Conversion 
    extend ActiveModel::Naming 

    attr_accessor :name, :email, :message_body 

    # validations are here... 

    def initialize(attributes = {}) 
     attributes.each do |name, value| 
     send("#{name}=", value) 
     end 
    end 

    def persisted? 
     false 
    end 
end 

The Contact Controller：

def create 
    @message = Message.new(params[:message]) 

    if @message.valid? 
     ContactMailer.new_message(@message).deliver 
     redirect_to(root_path, :notice => "Message was successfully sent.") 
    else 
     render :new 
    end 
end 

我的RSpec的控制器测试：

describe "POST create" do 
    context "with valid information" do 
     let (:message) { FactoryGirl.build(:message) } 
     it "creates a new message" do 
     post :create, params: message 
     assigns(:message).should eq(message) 
    end 
    end 
end 

我已经通过let验证了message变量不是零，所以FactoryGirl正在做的工作。

测试结果：

ContactController POST create with valid information creates a new message 
Failure/Error: post :create, params: message 
NoMethodError: 
    undefined method `each' for nil:NilClass 
# ./app/models/message.rb:14:in `initialize' 
# ./app/controllers/contact_controller.rb:8:in `new' 
# ./app/controllers/contact_controller.rb:8:in `create' 
# ./spec/controllers/contact_controller_spec.rb:22:in `block (4 levels) in <top (required)>' 

我知道它的失败是因为Message的initialize方法。但我现在确定它为什么初始化为零。当我在开发环境中测试表单时，它似乎工作正常。谢谢你的帮助。

Answer 1

我不得不将一个散列作为值传递给post的散列选项。像这样：

message_attributes = FactoryGirl.attributes_for(:message) 
post :create, message: message_attributes 

为了测试两个消息比较，我不得不修改Message模型像这样：

attr_accessor :attributes, :name, :email, :message_body 
def initialize(attributes = {}) 
    if not attributes.nil? 
     attributes.each do |name, value| 
      send("#{name}=", value) 
     end 
    end 
    @attributes = attributes 
end 

所以最后一个经过测试是这样的：

it "creates a new message" do 
    message_attributes = FactoryGirl.attributes_for(:message) 
    post :create, message: message_attributes 
    assigns(:message).attributes.symbolize_keys.should eq(message_attributes) 
end 

我不知道这是否是最佳的解决方案，但它确实创造通过的测试和验证控制器正确地创建消息。

Answer 2

我想你应该将消息传递与它明确的名称，就像这样：

it "creates a new message" do 
    post :create, :message => message 
    assigns(:message).should eq(message) 
end 

或短期风格：

post :create, message: message 

，那么你将有PARAMS哈希键'message'

params[:message] = ... # here's your message 

否则你不会有一个关键的params和hense params[:message]返回零