Почему мои тесты RSpec не проходят, а мое приложение работает? - PullRequest
Новая
       31

Почему мои тесты RSpec не проходят, а мое приложение работает?

9 голосов
/ 31 октября 2011

Я только что закончил главу 10 учебника по Ruby on Rails , добавив возможность редактировать / обновлять, индексировать и уничтожать пользователей.Кажется, что все работает правильно в моем приложении, но многие мои тесты не выполняются, когда я запускаю 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 <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)>'

ОБНОВЛЕНИЕ:

Я исправил 2 ошибки, но большинство все еще не устранены.Одна проблема заключается в том, что, когда тест пытается «получить: индекс», он не находит правильную страницу из-за перенаправления (на страницу / signin).Я не создавал никаких перенаправлений (намеренно в любом случае), но, возможно, у кого-то есть представление о том, как это могло произойти и как это можно исправить?

ОБНОВЛЕНИЕ № 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 => "another@example.com")
    third  = Factory(:user, :email => "another@example.net")

    @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 => "user@example.org", :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 => "user@example.net")
      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 => "admin@example.com", :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

Ответы [ 6 ]

1 голос
/ 24 ноября 2011

У меня была такая же проблема, и я нашел ответ на вопрос здесь .

в основном для RSPEC необходимо установить @current_user и текущего пользователя для sign_in и sign_out. Странно и раздражающе, но, возможно, причина, по которой можно подумать об использовании Devise в производственном приложении!

0 голосов
/ 28 ноября 2011

Я столкнулся с этой же проблемой сегодня, используя рельсы 3.1.3. Тем не менее, я зашел на сайт github для примера приложения Railstutorial и заметил, что в файле session_helper.erb произошли некоторые изменения.

1.) Добавьте ключевое слово «self» к «current_user» в методе sign_in (пользователь): 

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

Я не внес никаких других изменений, и все тесты пройдены.

0 голосов
/ 28 ноября 2011

У вас есть render_views вызов после верхнего уровня describe в users_controller_spec?

0 голосов
/ 23 ноября 2011

У меня была такая же проблема и при использовании Rails 3.1.3. Это мое решение.

Сначала убедитесь, что оба: 

def current_user=(user)
  @current_user = user
end

def current_user
  @current_user ||= user_from_remember_token
end

в сессиях_helper.rb

Кроме того, в session_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.

0 голосов
/ 20 ноября 2011

Убедитесь, что deny_access в SessionsHelper определено перед блоком 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
0 голосов
/ 01 ноября 2011

Предоставление вашего контроллера и тестового кода было бы полезно.Но без них я думаю, что вы пропустили это в блоке before для ваших неудачных тестов: 

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

Перенаправление происходит из-за какой-либо логики авторизации, используемой в учебнике, я думаю, что этоподробно описано в 10.11 и 10.12.

Это обычная ошибка, которую я вижу в тестах контроллеров с аутентификацией, надеюсь, это все, что вам нужно.

РЕДАКТИРОВАТЬ:

Спасибо за предоставление вашего кода.

Вот несколько замечаний, которые я заметил:

1.) Сбои 1–14, 19 связаны с тем, что аутентификация пользователя не работает, в частности подписанный_in ?.Тесты вашего сеансового контроллера проходят?Вы обязательно должны убедиться, что они пройдены перед тестированием пользовательского контроллера, поскольку он зависит от этой функциональности.

2.) Сбои 15-18 связаны с тем, что вы, вероятно, не создали миграцию для добавления логического атрибута в листинге 10.35.или вам нужно запустить миграцию в своей тестовой базе данных: rake db:test:prepare

Извините, я не смог определить все ваши проблемы, но, надеюсь, даст вам шаг в правильном направлении.

...