4

我正在尝试编写一些 RSpec 测试来测试我的应用程序,但我偶然发现了几个我找不到任何解决方案的问题。1)我正在尝试测试更新操作。这是我的代码:

        it "email is a new one" do
            put :update, id: @user, user: FactoryGirl.attributes_for(:user, :email=>"a@b.c")
            @user.reload
            @user.email.should == "a@b.c"
            puts @user.email
        end

这是 UsersController 更新操作:

  def update
    @user = User.find(params[:id])
    respond_to do |format|
      if @user.update_attributes(params[:user])
        format.html { redirect_to edit_user_path(@user), :notice => "Your settings were successfully updated."}
        format.json { head :no_content }
      else
        format.html { render action: "edit" }
        format.json { render json: @user.errors, status: :unprocessable_entity }
      end
    end
  end

这是错误:

 Failure/Error: @user.email.should == "a@b.c"
       expected: "a@b.c"
            got: "user16@example.com" (using ==)

很明显,测试并没有改变用户的电子邮件。我从这里获取了更新操作教程:http: //everydayrails.com/2012/04/07/testing-series-rspec-controllers.html。我在哪里可以找到解决方案?

4

2 回答 2

5

可能@user.update_attributes(params[:user])由于验证原因而失败?

此外,您可以确保您的测试和控制器方法正在与同一个 ruby​​ 对象进行交互。过去这对我来说是一个陷阱。我这样做的find方法是在类上存根方法。

it "email is a new one" do
  User.stubs(:find).returns(@user)
  put :update, id: @user, user: FactoryGirl.attributes_for(:user, :email=>"a@b.c")
  @user.reload
  @user.email.should == "a@b.c"
  puts @user.email
end

这可确保您在测试期间谈论的不仅是相同的记录,而且是相同的对象。


最后,我认为您的测试对您有很大帮助。您基本上是在 testing update_attributes,这是一个核心功能,并且已经过全面测试。我将专注于测试控制器的行为。像这样的东西:

let(:user) { FactoryGirl.create(:user) }

describe "PUT #update" do

  before(:each) {
    User.stubs(:find).returns(user)
  }

  it "should redirect to the user path on succesful save" do
    user.should_receive(:update_attributes).and_return true
    put :update, user, {}
    response.should redirect_to(edit_user_path(user))
  end

  it "should render the edit screen again with errors if the model doesn't save" do
    user.should_receive(:update_attributes).and_return false
    put :update, user, {}
    response.should render_template("edit")
  end
end
于 2013-06-30T13:15:06.957 回答
3

我认为 for 的论点put是不正确的。

put, get, delete,post接受三个参数。第一个是路径,第二个是参数,第三个是选项。

在您的代码中,您将两个参数作为两个参数,这是不正确的。

put :update, id: @user, user: FactoryGirl.attributes_for(:user, :email=>"a@b.c")

因此,将其更改为

put :update, {id: @user, user: FactoryGirl.attributes_for(:user, :email=>"a@b.c")}

可是等等!您的代码将通过上述更改工作,但您当前的代码中存在安全漏洞。确保您将在控制器代码中添加权限检查。例如

return unauthorized unless @user == current_user || current_user.role == "admin"
于 2013-06-30T13:43:12.407 回答