私のRSpecテストは失敗しますが、私のアプリは動いていますか？

私はちょうどchapter 10 of the Ruby on Rails Tutorialを終了し、ユーザーの編集/更新、インデックス作成、および削除を追加しました。私のアプリではすべてが正常に動作しているように見えますが、RSpecを実行すると多くのテストが失敗しています。私のRSpecテストは失敗しますが、私のアプリは動いていますか？

私はusers_controller_specを本の通りに設定しました。私のアプリケーションコードも同じです。 1つの問題は、彼が本で使っているRails 3.0とは対照的に、Rails 3.1.1を使用している可能性がありますか？以前のテストでは問題になっていませんでした。たまには、いくつかの異なるコード行があります。問題は私がsection 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 <top (required)>' 

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 <title>All users</title> tag: 
    <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd"> 
    <html><body>You are being <a href="http://test.host/signin">redirected</a>.</body></html> 
# ./spec/controllers/users_controller_spec.rb:36:in `block (4 levels) in <top (required)>' 

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 <li>Richard Berger</li> tag: 
    <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd"> 
    <html><body>You are being <a href="http://test.host/signin">redirected</a>.</body></html> 
# ./spec/controllers/users_controller_spec.rb:42:in `block (5 levels) in <top (required)>' 
# ./spec/controllers/users_controller_spec.rb:41:in `each' 
# ./spec/controllers/users_controller_spec.rb:41:in `block (4 levels) in <top (required)>' 

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 <div.pagination/> tag: 
    <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd"> 
    <html><body>You are being <a href="http://test.host/signin">redirected</a>.</body></html> 
# ./spec/controllers/users_controller_spec.rb:48:in `block (4 levels) in <top (required)>' 

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 <top (required)>' 

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 <title>Edit user</title> tag: 
    <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd"> 
    <html><body>You are being <a href="http://test.host/signin">redirected</a>.</body></html> 
# ./spec/controllers/users_controller_spec.rb:189:in `block (3 levels) in <top (required)>' 

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 <a href='http://gravatar.com/emails'>change</a> tag: 
    <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd"> 
    <html><body>You are being <a href="http://test.host/signin">redirected</a>.</body></html> 
# ./spec/controllers/users_controller_spec.rb:195:in `block (3 levels) in <top (required)>' 

8) UsersController PUT 'update' failure should render the 'edit' page 
Failure/Error: response.should render_template('edit') 
    expecting <"edit"> but rendering with <""> 
# ./spec/controllers/users_controller_spec.rb:214:in `block (4 levels) in <top (required)>' 

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 <title>Edit user</title> tag: 
    <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd"> 
    <html><body>You are being <a href="http://test.host/signin">redirected</a>.</body></html> 
# ./spec/controllers/users_controller_spec.rb:219:in `block (4 levels) in <top (required)>' 

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 <top (required)>' 

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 <http://test.host/users/1> but was a redirect to <http://test.host/signin> 
# ./spec/controllers/users_controller_spec.rb:238:in `block (4 levels) in <top (required)>' 

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 <top (required)>' 

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 <http://test.host/> but was a redirect to <http://test.host/signin> 
# ./spec/controllers/users_controller_spec.rb:276:in `block (4 levels) in <top (required)>' 

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 <http://test.host/> but was a redirect to <http://test.host/signin> 
# ./spec/controllers/users_controller_spec.rb:281:in `block (4 levels) in <top (required)>' 

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 <http://test.host/> but was a redirect to <http://test.host/signin> 
# ./spec/controllers/users_controller_spec.rb:303:in `block (4 levels) in <top (required)>' 

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 <top (required)>' 

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 <http://test.host/users> but was a redirect to <http://test.host/signin> 
# ./spec/controllers/users_controller_spec.rb:322:in `block (4 levels) in <top (required)>'

UPDATE：

私は、エラーの2を解決しますが、ほとんどはまだ持続しています。 1つの問題は、テストで「get：index」を試みているときに、/ signinページにリダイレクトされたために正しいページが見つからないということです。どのようなリダイレクトも実施していませんでしたが、誰かがこれがどのように起こった可能性があり、どのように修正できるかという考えがありましたか？

UPDATE＃2：@ 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 => "[email protected]") 
    third = Factory(:user, :email => "[email protected]") 

    @users = [@user, second, third] 
    30.times do 
     @users << Factory(:user, :email => 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 => "[email protected]", :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 => "[email protected]") 
     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 => "[email protected]", :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 < ApplicationController 
before_filter :authenticate, :only => [: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

出典

2011-10-30 RBergs

これを把握しましたか？あなたは20日前に編集したばかりです。私は3.1に移植して、同じ問題を抱え始めました。 –

私は同じ問題があり、答えがhereと掲載されています。

基本的にRSPECは、@current_userと現在のユーザーの両方をsign_inとsign_outに設定する必要があります。奇妙で迷惑なのですが、プロダクションアプリでDeviseを使うと考えられるかもしれません！

出典

2011-11-24 08:06:51

これは私の問題を解決したようですが、私はまだ完全になぜそれほど確かではありません。助けてくれてありがとう！ – RBergs

あなたのコントローラとテストコードの提供が参考になります。しかし、それらなしで私の推測では、あなたの失敗のテストのために前のブロックでこれを欠けていることを次のようになります。リダイレクトが原因チュートリアルで使用されているものは何でも承認ロジックで何が起こっている

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

を、私はそれがで詳細だと思います10.11および10.12。

これはauthでコントローラテストに表示されるよくある問題です。うまくいけば、それはあなたにとっても同様です。

編集：

コードをご提供いただきありがとうございます。

ここに私は気づいたいくつかのことです：

1）障害1-14、19は、具体的signed_in、動作していないユーザ認証によるものでは？。あなたのセッションコントローラのテストは合格ですか？その機能に依存しているので、ユーザコントローラをテストする前にそれらが合格していることを間違いなく確認する必要があります。あなたはおそらく、あなたのテスト・データベース上で移行を実行する必要があります - または - リスト10.35にブール属性を追加するには、移行を作成していないので、

2）障害15-18は、次のとおりです。rake db:test:prepare

申し訳ありませんが、私すべての問題点を指摘することはできませんでしたが、うまくいけば、正しい方向へ一歩を踏み出すことができます。

出典

2011-10-31 22:02:38 bkempner

答えをありがとうが、残念ながら私はそれが問題だとは思わない。私のコントローラとテストコードをメインのポストに追加するつもりです。 – RBergs

ええ、問題を抱えていただきありがとうございます。すべての私のセッションコントローラのテストが合格しているので、それが問題だとは思わないでください。また、私は属性とデータベースを移行したことを確かめました。いくつかのマイナーな変更があったため、エラーのリストも更新されました。残念ながら何が間違っているのかまだ分かりません！ – RBergs

SessionsHelperのdeny_accessがブロックの前に定義されていることを確認してください。

この本には、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

出典

2011-11-20 14:54:26

提案に感謝しますが、私はすでに適切な場所にdeny_accessを持っています。 – RBergs

私は同じ問題を抱えていたと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.current_user = user 
end

へ

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

は参照してください、私は章を完了していないが、これは10.2.1を通して私を取得します。経験が豊富な人は、なぜこの作品にコメントすることができます。私は、before_filterに応じてrspecを渡すために書いた別の解決策を読んでいます。

出典

2011-11-22 22:04:19 Scott

users_controller_specのトップレベルdescribeの後にrender_viewsコールがありますか？

出典

2011-11-27 21:07:09

ypt、提案のおかげで – RBergs

レール3.1.3を使用して同じ問題が発生しました。しかし、私はrailstutorialサンプルアプリケーションのgithubサイトに行き、sessions_helper.erbに変更があったことに気付きました。、これはまた、「sign_out」メソッドのために行く

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

2）「自己を追加します。

1）sign_inの「CURRENT_USER」から「自己」キーワード（ユーザー）メソッドを追加します。 'キーワードを' current_user 'に変更

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

私は他の変更を行わず、すべてのテストに合格しました。

出典

2011-11-27 21:32:04

興味深いので、私は理由が分からないが、自分自身を追加することは、@ current_userを参照して2番目の行を追加しながら動作していない（ここで参照してください（http://stackoverflow.com/questions/6821692/cookies-do-not-persist-in-rspec-on-rails-3-1））は動作しました... – RBergs

私のRSpecテストは失敗しますが、私のアプリは動いていますか？

答えて

関連する問題