フレンドリーフォワーディングの実装

Question

フレンドリーフォワーディングの実装について

現在railsでポートフォリオを作成中なのですが、次のようなエラーが発生してしまいます。
おそらくフレンドリーフォワーディングが実装できていないのが原因だと思うのですが、いろいろ調べても必要なファイルやコードなどは揃っているように感じます。
何か解決策がありましたらご教示お願いします。

発生している問題・エラーメッセージ

Failures:

  1) プロフィール編集 認可されたユーザーの場合 レスポンスが正常に表示されること(+フレンドリーフォワーディング)
     Failure/Error: expect(response).to redirect_to edit_user_url(user)
     
       Expected response to be a redirect to <http://www.example.com/users/17183/edit> but was a redirect to <http://www.example.com/users/17183>.
       Expected "http://www.example.com/users/17183/edit" to be === "http://www.example.com/users/17183".
     # ./spec/requests/users_edit_spec.rb:11:in `block (3 levels) in <top (required)>'
     # -e:1:in `<main>'

  2) プロフィール編集 ログインしていないユーザーの場合 ログイン画面にリダイレクトすること
     Failure/Error: expect(response).to redirect_to login_path
     
       Expected response to be a redirect to <http://www.example.com/login> but was a redirect to <http://www.example.com/>.
       Expected "http://www.example.com/login" to be === "http://www.example.com/".
     # ./spec/requests/users_edit_spec.rb:26:in `block (3 levels) in <top (required)>'
     # -e:1:in `<main>'

該当のソースコード

[users_edit_spec.rb]
require "rails_helper"

RSpec.describe "プロフィール編集", type: :request do
  let!(:user) { create(:user) }
  let!(:other_user) { create(:user) }

  context "認可されたユーザーの場合" do
    it "レスポンスが正常に表示されること(+フレンドリーフォワーディング)" do
      get edit_user_path(user)
      login_for_request(user)
      expect(response).to redirect_to edit_user_url(user)
      patch user_path(user), params: { user: { name: "Example User",
                                               email: "[e-mail]",
                                               introduction: "初めまして",
                                               sex: "男性" } }
      follow_redirect!
      expect(response).to render_template('users/show')
    end
  end

  context "ログインしていないユーザーの場合" do
    it "ログイン画面にリダイレクトすること" do
      # 編集
      get edit_user_path(user)
      expect(response).to have_http_status "302"
      expect(response).to redirect_to login_path
      # 更新
      patch user_path(user), params: { user: { name: user.name,
                                               email: user.email } }
      expect(response).to have_http_status "302"
      expect(response).to redirect_to login_path
    end
  end

  context "別アカウントのユーザーの場合" do
    it "ホーム画面にリダイレクトすること" do
      # 編集
      login_for_request(other_user)
      get edit_user_path(user)
      expect(response).to have_http_status "302"
      expect(response).to redirect_to root_path
      # 更新
      patch user_path(user), params: { user: { name: user.name,
                                               email: user.email } }
      expect(response).to have_http_status "302"
      expect(response).to redirect_to root_path
    end
  end
end

[users_controller.rb]
class UsersController < ApplicationController
  before_action :correct_user,   only: [:edit, :update]
  before_action :logged_in_user, only: [
    :index, :show, :edit, :update, :destroy, :following, :followers,
  ]

  def new
    @user = User.new
  end

  def show
    @user = User.find(params[:id])
    @cafes = @user.cafes.paginate(page: params[:page], per_page: 5)
  end

  def index
    @users = User.paginate(page: params[:page])
  end

  def create
    @user = User.new(user_params)
    if @user.save
      log_in @user
      flash[:success] = "カフェアプリへようこそ！"
      redirect_to @user
    else
      render 'new'
    end
  end

  def edit
    @user = User.find(params[:id])
  end

  def update
    @user = User.find(params[:id])
    if @user.update_attributes(user_params_update)
      flash[:success] = "プロフィールが更新されました！"
      redirect_to @user
    else
      render 'edit'
    end
  end

  def destroy
    @user = User.find(params[:id])
    # 管理者ユーザーの場合
    if current_user.admin?
      @user.destroy
      flash[:success] = "ユーザーの削除に成功しました"
      redirect_to users_url
    # 管理者ユーザーではないが、自分のアカウントの場合
    elsif current_user?(@user)
      @user.destroy
      flash[:success] = "自分のアカウントを削除しました"
      redirect_to root_url
    else
      flash[:danger] = "他人のアカウントは削除できません"
      redirect_to root_url
    end
  end

  def following
    @title = "フォロー中"
    @user  = User.find(params[:id])
    @users = @user.following.paginate(page: params[:page])
    render 'show_follow'
  end

  def followers
    @title = "フォロワー"
    @user  = User.find(params[:id])
    @users = @user.followers.paginate(page: params[:page])
    render 'show_follow'
  end

  private

    # ユーザー新規作成時に許可する属性
    def user_params
      params.require(:user).permit(:name, :email, :password,
                                   :password_confirmation)
    end

    # プロフィール編集時に許可する属性
    def user_params_update
      params.require(:user).permit(:name, :email, :introduction, :sex)
    end

    # ログイン済みユーザーかどうか確認
    def logged_in_user
      unless logged_in?
        store_location
        flash[:danger] = "ログインしてください"
        redirect_to login_url
      end
    end

    # 正しいユーザーかどうか確認
    def correct_user
      @user = User.find(params[:id])
      if !current_user?(@user)
        flash[:danger] = "このページへはアクセスできません"
        redirect_to(root_url)
      end
    end
end

[sessions_helper.rb]
module SessionsHelper
  def log_in(user)
    session[:user_id] = user.id
  end

  # ユーザーのセッションを永続的にする
  def remember(user)
    user.remember
    cookies.permanent.signed[:user_id] = user.id
    cookies.permanent[:remember_token] = user.remember_token
  end

  # 渡されたユーザーがログイン済みユーザーであればtrueを返す
  def current_user?(user)
    user == current_user
  end

  # 記憶トークンcookieに対応するユーザーを返す
  def current_user
    if (user_id = session[:user_id])
      @current_user ||= User.find_by(id: user_id)
    elsif (user_id = cookies.signed[:user_id])
      user = User.find_by(id: user_id)
      if user && user.authenticated?(cookies[:remember_token])
        log_in user
        @current_user = user
      end
    end
  end

  def logged_in?
    !current_user.nil?
  end

  # 永続的セッションを破棄する
  def forget(user)
    user.forget
    cookies.delete(:user_id)
    cookies.delete(:remember_token)
  end

  def log_out
    forget(current_user)
    session.delete(:user_id)
    @current_user = nil
  end

  # 記憶したURL (もしくはデフォルト値) にリダイレクト
  def redirect_back_or(default)
    redirect_to(session[:forwarding_url] || default)
    session.delete(:forwarding_url)
  end

  # アクセスしようとしたURLを覚えておく
  def store_location
    session[:forwarding_url] = request.original_url if request.get?
  end
end

[sessions_controller.rb]
class SessionsController < ApplicationController
  def new
  end

  def create
    user = User.find_by(email: params[:session][:email].downcase)
    if user && user.authenticate(params[:session][:password])
      log_in user
      params[:session][:remember_me] == '1' ? remember(user) : forget(user)
      redirect_back_or user
    else
      flash.now[:danger] = 'メールアドレスとパスワードの組み合わせが誤っています'
      render 'new'
    end
  end

  def destroy
    log_out if logged_in?
    redirect_to root_url
  end
end

[application_controller.rb]
class ApplicationController < ActionController::Base
  before_action :set_search
  protect_from_forgery with: :exception
  include SessionsHelper

  # フィードから検索条件に該当する投稿を検索
  def set_search
    if logged_in?
      @search_word = params[:q][:name_or_ingredients_name_cont] if params[:q]
      @q = current_user.feed.paginate(page: params[:page], per_page: 5).ransack(params[:q])
      @cafes = @q.result(distinct: true)
    end
  end

  private

    # ログイン済みユーザーかどうか確認
    def logged_in_user
      unless logged_in?
        store_location
        flash[:danger] = "ログインしてください"
        redirect_to login_url
      end
    end
end

試したこと

ネットでフレンドリーフォワーディングの実装に必要なことを色々調べ、確認しました。

Answer 1

自己解決しました！

結論から言うと、users_controller.rbにある「before_action」の順番が逆でした。
「logged_action_user」でログインしてないユーザーだったらログインページ(/login)に飛ぶところを、「correct_user」が先に来てしまっていたためログインページに飛ぶ前にrootページ(/)に飛んでしまっていました。