jeudi 16 juillet 2015

Password_resets Test Won't Past - Error: undefined method `errors' for nil:NilClass

Hello Im trying to pass the Password Reset Test you see below. I think the reason it wont pass is due to this line of code directly below. Because i just changed it so my _error_messages partial would render properly.

<%= render 'shared/error_messages', form_object: :password_reset%>

Here is the error.

  1) Error:
PasswordResetsTest#test_password_resets:
ActionView::Template::Error: undefined method `errors' for nil:NilClass
    app/views/shared/_error_messages.html.erb:1:in `_app_views_shared__error_messages_html_erb__4284961570170766644_70157133091540'
    app/views/password_resets/edit.html.erb:8:in `_app_views_password_resets_edit_html_erb___3012273417726284162_70157193336420'
    test/integration/password_resets_test.rb:36:in `block in <class:PasswordResetsTest>'

Here is my test. Line 36 which relates to error is get edit_password_reset_path(user.reset_token, email: user.email)

 test "password resets" do
    get new_password_reset_path
    assert_template 'password_resets/new'
    # Invalid email
    post password_resets_path, password_reset: { email: "" }
    assert_not flash.empty?
    assert_template 'password_resets/new'
    # Valid email
    post password_resets_path, password_reset: { email: @user.email }
    assert_not_equal @user.reset_digest, @user.reload.reset_digest
    assert_equal 1, ActionMailer::Base.deliveries.size
    assert_not flash.empty?
    assert_redirected_to root_url
    # Password reset form
    user = assigns(:user)
    # Wrong email
    get edit_password_reset_path(user.reset_token, email: "")
    assert_redirected_to root_url
    # Inactive user
    user.toggle!(:activated)
    get edit_password_reset_path(user.reset_token, email: user.email)
    assert_redirected_to root_url
    user.toggle!(:activated)
    # Right email, wrong token
    get edit_password_reset_path('wrong token', email: user.email)
    assert_redirected_to root_url
    # Right email, right token
    get edit_password_reset_path(user.reset_token, email: user.email)
    assert_template 'password_resets/edit'
    assert_select "input[name=email][type=hidden][value=?]", user.email
    # Invalid password & confirmation
    patch password_reset_path(user.reset_token),
          email: user.email,
          user: { password:              "foobaz",
                  password_confirmation: "barquux" }
    assert_select 'div#error_explanation'
    # Empty password
    patch password_reset_path(user.reset_token),
          email: user.email,
          user: { password:              "",
                  password_confirmation: "" }
    assert_not flash.empty?
    assert_template 'password_resets/edit'
    # Valid password & confirmation
    patch password_reset_path(user.reset_token),
          email: user.email,
          user: { password:              "foobaz",
                  password_confirmation: "foobaz" }
    assert is_logged_in?
    assert_not flash.empty?
    assert_redirected_to user
  end

Here is views/password_resets/edit.html.erb

Reset password

<div class="container">
  <div class=“row”>
    <div class="col-md-6 col-md-offset-3">
      <div class="panel panel-primary">
        <div class="panel-body">
          <%= render 'shared/error_messages', form_object: :password_reset%>
          <%= simple_form_for(@user, url: password_reset_path(params[:id])) do |f| %>
            <%= render 'shared/error_messages' %>
            <%= hidden_field_tag :email, @user.email %>
            <%= f.input :password, class: 'form-control' %>
            <%= f.input :password_confirmation, class: 'form-control' %>
            <%= f.button :submit, "Update Password", class: "btn btn-primary" %>
          <% end %>
        </div>
      </div>    
    </div>
  </div>
</div>

Here is my Password_resets_controller

class PasswordResetsController < ApplicationController
  before_action :get_user,   only: [:edit, :update]
  before_action :valid_user, only: [:edit, :update]
  before_action :check_expiration, only: [:edit, :update]

  def create
    @user = User.find_by(email: params[:password_reset][:email].downcase)
    if @user
      @user.create_reset_digest
      @user.send_password_reset_email
      flash[:info] = "Email sent with password reset instructions"
      redirect_to root_url
    else
      flash.now[:danger] = "Email address not found"
      render 'new'
    end
  end

  def edit
  end

  def update
    if params[:user][:password].empty?
      flash.now[:danger] = "Password can't be empty"
      render 'edit'
    elsif @user.update_attributes(user_params)
      log_in @user
      flash[:success] = "Password has been reset."
      redirect_to @user
    else
      render 'edit'
    end
  end

  private

    def user_params
      params.require(:user).permit(:password, :password_confirmation)
    end

    # Before filters

    def get_user
      @user = User.find_by(email: params[:email])
    end

    # Confirms a valid user.
    def valid_user
      unless (@user && @user.activated? &&
              @user.authenticated?(:reset, params[:id]))
        redirect_to root_url
      end
    end

    # Checks expiration of reset token.
    def check_expiration
      if @user.password_reset_expired?
        flash[:danger] = "Password reset has expired."
        redirect_to new_password_reset_url
      end
    end

end

Aucun commentaire:

Enregistrer un commentaire