为什么我的RSpec测试失败，但我的应用程序正在运行？

我刚刚完成了Ruby on Rails教程的第10章 ，添加了编辑/更新，索引和销毁用户的function。 一切似乎都在我的应用程序中正常工作，但是当我运行RSpec时，我的许多测试都失败了。

我设置的users_controller_spec与书的设置完全一样，我的应用程序代码也是一样的。 一个问题可能是我使用Rails 3.1.1而不是他在书中使用的Rails 3.0？ 它对于以前的测试来说并不是一个问题，只是偶尔会出现几行不同的代码。 在我开始第10.2.1节后 ，问题开始出现。

以下是我看到的错误列表，如果您需要更多信息，请告诉我。 谢谢！

1) UsersController GET 'index' for signed-in users should be successful Failure/Error: response.should be_success expected success? to return true, got false # ./spec/controllers/users_controller_spec.rb:31:in `block (4 levels) in ' 2) UsersController GET 'index' for signed-in users should have the right title Failure/Error: response.should have_selector("title", :content => "All users") expected following output to contain a  tag:  You are being redirected. # ./spec/controllers/users_controller_spec.rb:36:in `block (4 levels) in ' 3) UsersController GET 'index' for signed-in users should have an element for each user Failure/Error: response.should have_selector("li", :content => user.name) expected following output to contain a 
Richard Berger
 tag:  You are being redirected. # ./spec/controllers/users_controller_spec.rb:42:in `block (5 levels) in ' # ./spec/controllers/users_controller_spec.rb:41:in `each' # ./spec/controllers/users_controller_spec.rb:41:in `block (4 levels) in ' 4) UsersController GET 'index' for signed-in users should paginate users Failure/Error: response.should have_selector("div.pagination") expected following output to contain a 
 tag:  You are being redirected. # ./spec/controllers/users_controller_spec.rb:48:in `block (4 levels) in ' 5) UsersController GET 'edit' should be successful Failure/Error: response.should be_success expected success? to return true, got false # ./spec/controllers/users_controller_spec.rb:184:in `block (3 levels) in ' 6) UsersController GET 'edit' should have the right title Failure/Error: response.should have_selector("title", :content => "Edit user") expected following output to contain a  tag:  You are being redirected. # ./spec/controllers/users_controller_spec.rb:189:in `block (3 levels) in ' 7) UsersController GET 'edit' should have a link to change the Gravatar Failure/Error: response.should have_selector("a", :href => gravatar_url, :content => "change") expected following output to contain a change tag:  You are being redirected. # ./spec/controllers/users_controller_spec.rb:195:in `block (3 levels) in ' 8) UsersController PUT 'update' failure should render the 'edit' page Failure/Error: response.should render_template('edit') expecting  but rendering with  # ./spec/controllers/users_controller_spec.rb:214:in `block (4 levels) in ' 9) UsersController PUT 'update' failure should have the right title Failure/Error: response.should have_selector("title", :content => "Edit user") expected following output to contain a  tag:  You are being redirected. # ./spec/controllers/users_controller_spec.rb:219:in `block (4 levels) in ' 10) UsersController PUT 'update' success should change the user's attributes Failure/Error: @user.name.should == @attr[:name] expected: "New Name" got: "Richard Berger" (using ==) # ./spec/controllers/users_controller_spec.rb:232:in `block (4 levels) in ' 11) UsersController PUT 'update' success should redirect to the user show page Failure/Error: response.should redirect_to(user_path(@user)) Expected response to be a redirect to  but was a redirect to  # ./spec/controllers/users_controller_spec.rb:238:in `block (4 levels) in ' 12) UsersController PUT 'update' success should have a flash message Failure/Error: flash[:success].should =~ /updated/ expected: /updated/ got: nil (using =~) # ./spec/controllers/users_controller_spec.rb:243:in `block (4 levels) in ' 13) UsersController authentication of edit/update pages for signed-in users should require matching users for 'edit' Failure/Error: response.should redirect_to(root_path) Expected response to be a redirect to  but was a redirect to  # ./spec/controllers/users_controller_spec.rb:276:in `block (4 levels) in ' 14) UsersController authentication of edit/update pages for signed-in users should require matching users for 'update' Failure/Error: response.should redirect_to(root_path) Expected response to be a redirect to  but was a redirect to  # ./spec/controllers/users_controller_spec.rb:281:in `block (4 levels) in ' 15) UsersController DELETE 'destroy' as a non-admin user should protect the page Failure/Error: response.should redirect_to(root_path) Expected response to be a redirect to  but was a redirect to  # ./spec/controllers/users_controller_spec.rb:303:in `block (4 levels) in ' 16) UsersController DELETE 'destroy' as an admin user should destroy the user Failure/Error: lambda do count should have been changed by -1, but was changed by 0 # ./spec/controllers/users_controller_spec.rb:315:in `block (4 levels) in ' 17) UsersController DELETE 'destroy' as an admin user should redirect to the users page Failure/Error: response.should redirect_to(users_path) Expected response to be a redirect to  but was a redirect to  # ./spec/controllers/users_controller_spec.rb:322:in `block (4 levels) in ' 

更新：

我解决了2个错误，但大多数仍然存在。 一个问题似乎是当测试试图“获取：索引”时，由于重定向（到/ signin页面），它找不到正确的页面。 我没有建立任何重定向（无论如何），但也许有人知道这可能发生了什么以及如何解决？

更新＃2：

Per @ bkempner的请求，这里是users_controller_spec和users_controller（users_controller_spec优先）的相关代码。 很多代码，但很多错误…：

 describe "for signed-in users" do before(:each) do @user = test_sign_in(Factory(:user)) second = Factory(:user, :email => "[e-mail]") third = Factory(:user, :email => "[e-mail]") @users = [@user, second, third] 30.times do @users < Factory.next(:email)) end end it "should be successful" do get :index response.should be_success end it "should have the right title" do get :index response.should have_selector("title", :content => "All users") end it "should have an element for each user" do get :index @users[0..2].each do |user| response.should have_selector("li", :content => user.name) end end it "should paginate users" do get :index response.should have_selector("div.pagination") response.should have_selector("span.diabled", :content => "Previous") response.should have_selector("a", :href => "/users?page=2", :content => "2") response.should have_selector("a", :href => "/users?page=2", :content => "Next") end end describe "GET 'edit'" do before(:each) do @user = Factory(:user) test_sign_in(@user) end it "should be successful" do get :edit, :id => @user response.should be_success end it "should have the right title" do get :edit, :id => @user response.should have_selector("title", :content => "Edit user") end it "should have a link to change the Gravatar" do get :edit, :id => @user gravatar_url = "http://gravatar.com/emails" response.should have_selector("a", :href => gravatar_url, :content => "change") end end describe "PUT 'update'" do before(:each) do @user = Factory(:user) test_sign_in(@user) end describe "failure" do before(:each) do @attr = { :email => "", :name => "", :password => "", :password_confirmation => "" } end it "should render the 'edit' page" do put :update, :id => @user, :user => @attr response.should render_template('edit') end it "should have the right title" do put :update, :id => @user, :user => @attr response.should have_selector("title", :content => "Edit user") end end describe "success" do before(:each) do @attr = { :name => "New Name", :email => "[e-mail]", :password => "barbaz", :password_confirmation => "barbaz" } end it "should change the user's attributes" do put :update, :id => @user, :user => @attr @user.reload @user.name.should == @attr[:name] @user.email.should == @attr[:email] end it "should redirect to the user show page" do put :update, :id => @user, :user => @attr response.should redirect_to(user_path(@user)) end it "should have a flash message" do put :update, :id => @user, :user => @attr flash[:success].should =~ /updated/ end end end describe "authentication of edit/update pages" do describe "for signed-in users" do before(:each) do wrong_user = Factory(:user, :email => "[e-mail]") test_sign_in(wrong_user) end it "should require matching users for 'edit'" do get :edit, :id => @user response.should redirect_to(root_path) end it "should require matching users for 'update'" do get :update, :id => @user, :user => {} response.should redirect_to(root_path) end end end describe "DELETE 'destroy'" do before(:each) do @user = Factory(:user) end describe "as a non-admin user" do it "should protect the page" do test_sign_in(@user) delete :destroy, :id => @user response.should redirect_to(root_path) end end describe "as an admin user" do before(:each) do admin = Factory(:user, :email => "[e-mail]", :admin => true) test_sign_in(admin) end it "should destroy the user" do lambda do delete :destroy, :id => @user end.should change(User, :count).by(-1) end it "should redirect to the users page" do delete :destroy, :id => @user response.should redirect_to(users_path) end end end 

现在整个users_controller：

 class UsersController  [:index, :edit, :update, :destroy] before_filter :correct_user, :only => [:edit, :update] before_filter :admin_user, :only => :destroy def index @title = "All users" @users = User.paginate(:page => params[:page]) end def show @user = User.find(params[:id]) @title = @user.name end def new @user = User.new @title = "Sign up" end def create @user = User.new(params[:user]) if @user.save sign_in @user flash[:success] = "Welcome to the Sample App!" redirect_to @user else @title = "Sign up" render 'new' end end def edit @title = "Edit user" end def update @user = User.find(params[:id]) if @user.update_attributes(params[:user]) flash[:success] = "Profile updated" redirect_to @user else @title = "Edit user" render 'edit' end end def destroy User.find(params[:id]).destroy flash[:success] = "User destroyed." redirect_to users_path end private def authenticate deny_access unless signed_in? end def correct_user @user = User.find(params[:id]) redirect_to(root_path) unless current_user?(@user) end def admin_user redirect_to(root_path) unless current_user.admin? end end 

Answer 1

我有同样的问题，发现这里的答案。

基本上RSPEC需要为sign_in和sign_out设置@current_user和当前用户。 奇怪而烦人，但也许是人们可能会考虑在生产应用程序中使用Devise的原因！

Answer 2

提供控制器和测试代码会很有帮助。 但是如果没有它们，我的猜测就是你在前面的块中错过了这个失败的测试：

 @user = Factory(:user) test_sign_in(@user) 

重定向正在发生，因为教程中使用了任何授权逻辑，我认为它在10.11和10.12中有详细说明。

这是我在控制器测试中使用auth看到的常见问题，希望这也是它的全部内容。

编辑：

感谢您提供代码。

这是我注意到的一些事情：

1.）故障1-14,19是由于用户认证不起作用，特别是signed_in？。 您的会话控制器测试是否通过？ 在测试用户控制器之前，您一定要确保它们通过，因为它取决于该function。

2.）失败15-18是因为您可能没有创建迁移以添加清单10.35中的布尔属性 – 或者 – 您需要在测试数据库上运行迁移： rake db:test:prepare

对不起，我无法指出你所有的问题，但希望能给你一个正确的方向。

Answer 3

检查SessionsHelper中的deny_access是否在private块之前定义。

这本书有一个拼写错误，暗示你应该把deny_access放在模块的末尾。 不幸的是，这意味着该函数被定义为私有。

 module SessionsHelper . . . def deny_access redirect_to signin_path, :notice => "Please sign in to access this page." end end 

Railstutorial.org网站上的更正代码如下：

 module SessionsHelper . . . def deny_access redirect_to signin_path, :notice => "Please sign in to access this page." end private . . . end 

Answer 4

我遇到了同样的问题并使用Rails 3.1.3。 这是我的解决方案。

首先，确保两者：

 def current_user=(user) @current_user = user end def current_user @current_user ||= user_from_remember_token end 

在sessions_helper.rb中

另外，在sessions_helper.rb中更改：

 def sign_out cookies.delete(:remember_token) current_user.nil end 

至

 def sign_out cookies.delete(:remember_token) self.current_user.nil end 

还有一件事。

在spec_helper.rb中更改：

 def test_sign_in(user) controller.sign_in(user) end 

至

 def test_sign_in(user) controller.current_user = user end 

看看是否有帮助，我还没有完成这一章，但这让我了解了10.2.1。 有经验的人可以评论为什么会有效。 我读了另一个根据before_filter编写的解决方案，以使rspec通过。

Answer 5

在render_views顶级describe之后，您是否调用了render_views ？

Answer 6

今天我使用rails 3.1.3遇到了同样的问题。 但是，我去了github网站上的railsstutorial示例应用程序，并注意到sessions_helper.erb有一些更改。

1.）在sign_in（用户）方法中将’self’关键字添加到’current_user’：

 def sign_in(user) cookies.permanent.signed[:remember_token] = [user.id, user.salt] self.current_user = user end 

2.）这也适用于’sign_out’方法，将’self’关键字添加到’current_user’

 def sign_out cookies.delete(:remember_token) self.current_user = nil end 

我没有做任何其他更改，所有测试都通过。