Ruby on Rails6 [devise] 自分以外のユーザーのプロフィールを編集できないようにしたい

Question

実現したいこと

・deviseでユーザー認証機能を導入したい
・その過程で、自分以外のユーザーのプロフィールを編集できないようにしたい
・ユーザーのプロフィールページに置かれたリンクをクリックすることで、プロフィールの編集ページに移動させたい

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

上記の機能をprotectedメソッドとして定義してみたのですが、プロフィールの編集のリンクを押したところ、編集ページ(registrations#edit)に入れませんでした。

ブラウザには下記のエラーが表示されています。

ActiveRecord::RecordNotFound in Users::RegistrationsController#edit

Couldn't find User without an ID

 # edit, updateアクションを現在のユーザーのみに限定する
    def correct_user
      @user = User.find(params[:id])
      redirect_to(root_url) unless @user == current_user
    end

該当のソースコード

Railstutorialを参考にprotectedメソッドにcorrect_userを定義しました。
ただ、これが原因でエラーが出ているようです。

# frozen_string_literal: true

class Users::RegistrationsController < Devise::RegistrationsController
  before_action :authenticate_user!, only: [:edit, :update, :destroy]
  before_action :correct_user, only: [:edit, :update]

  def new
    super
  end

  def create
    super
  end

  def edit
    super
  end

  def update
    super
  end

  def destroy
    super
  end

  protected

  # edit, updateアクションを現在のユーザーのみに限定する
    def correct_user
      @user = User.find(params[:id])
      redirect_to root_path unless @user == current_user
    end

end

該当のview

こちらはdeviseと関係なく作成したユーザーのプロフィールページ(users#show)です。
「プロフィールの編集」リンクを押して、プロフィール編集画面に移動する際に上記のエラーがでました。
このページは正しく表示されています。

<% provide(:title, @user.name) %>

<h3>プロフィール</h3>
<p>名前 : <%= @user.name %></p>
<p>メールアドレス： <%= @user.email %></p>

<%= link_to 'プロフィールの編集', edit_user_registration_path(@user) %>

routes.rb

Rails.application.routes.draw do
  root 'static_pages#home'
  get  '/help', to:'static_pages#help'
  get  '/about', to:'static_pages#about'
  get  '/contact', to:'static_pages#contact'
  devise_for :users, :controllers => {
    :confirmations => 'users/confirmations',
    :registrations => 'users/registrations',
    :sessions => 'users/sessions'
  }
  devise_scope :user do
    get      "sign_in", to:"users/sessions#new"
    delete   "sign_out", to:"users/sessions#destroy"
  end
   resources :users, only: [:index, :show]
end

試したこと

Rspecでregistrations_controllerのテストをしました。
そのテストはパスするのですが、そもそも正しいテストを書けているか自信がありません。

念のため、以下にテスト内容とテストデータ(FactoryBot)を記載します。

require 'rails_helper'

describe Users::RegistrationsController, type: :controller do
  let(:user) { create(:user) }
  let(:other_user) { create(:other_user) }

  # ユーザー情報編集ページの表示をテスト
  describe 'GET #edit' do
    # ログインしている場合
    context 'with logged in' do
      # edit(ユーザー情報編集)ページの検証
      it 'responds successfully' do
        @request.env["devise.mapping"] = Devise.mappings[:user]
        sign_in user
        get :edit, params: { id: user.id }
        expect(response).to have_http_status "200"
        expect(response).to render_template :edit
      end
    end
    # ログインしていない場合
    context 'without logged in' do
      # edit(ユーザー情報編集)ページの代わりにsign_in(ログイン)ページを返すか検証
      it 'redirect to new_session_path' do
        @request.env["devise.mapping"] = Devise.mappings[:user]
        get :edit, params: { id: user.id }
        expect(response).to have_http_status "302"
        expect(response).to redirect_to new_user_session_path
      end
    end
    # 他のユーザーでログインしている場合
    context 'with logged in as other user' do
      # ルートにリダイレクトされるか検証
      it 'redirect to root_path' do
        @request.env["devise.mapping"] = Devise.mappings[:user]
        sign_in other_user
        get :edit, params: { id: user.id }
        expect(response).to have_http_status "302"
        expect(response).to redirect_to root_path
      end
    end
  end

end

FactoryBot.define do

  factory :user, class: User do
    name { 'foo' }
    email { '[e-mail]' }
    password { 'foo123' }
    password_confirmation { 'foo123' }
    confirmed_at { Date.today }
  end

  factory :other_user, class: User do
    name { 'bar' }
    email { '[e-mail]' }
    password { 'bar123' }
    password_confirmation { 'bar123' }
    confirmed_at { Date.today }
  end

end

確認したこと

correct_userがなければ編集ページに移動できることは確認しており、その際URLにはユーザーのIDが記載されていました。そのため、IDが見つからないからユーザーを検索できないというエラーの原因がわかりませんでした。

1日悩んでわからなかったので、どなたか原因を教えていただけないでしょうか。
質問に不備がある場合は遠慮なく伝えていただけると幸いです。

どうぞよろしくお願いします。

補足情報（FW/ツールのバージョンなど）

ruby 2.7.1
Rails 6.0.3.2

Comments

[maisumakun]

deviseが生成したルーティングを書いていただけますか？

[退会済みユーザー]

質問の意図が把握できず申し訳ないのですが、ターミナルで「rails routes」と打って出てくるルーティングのことでしょうか？
もしそれだとしたら、下記のとおりです。(整形ができず見辛くてすみません）

new_user_session GET /users/sign_in(.:format) users/sessions#new
user_session POST /users/sign_in(.:format) users/sessions#create
destroy_user_session DELETE /users/sign_out(.:format) users/sessions#destroy
new_user_password GET /users/password/new(.:format) devise/passwords#new
edit_user_password GET /users/password/edit(.:format) devise/passwords#edit
user_password PATCH /users/password(.:format) devise/passwords#update
PUT /users/password(.:format) devise/passwords#update
POST /users/password(.:format) devise/passwords#create
cancel_user_registration GET /users/cancel(.:format) users/registrations#cancel
new_user_registration GET /users/sign_up(.:format) users/registrations#new
edit_user_registration GET /users/edit(.:format) users/registrations#edit
user_registration PATCH /users(.:format) users/registrations#update
PUT /users(.:format) users/registrations#update
DELETE /users(.:format) users/registrations#destroy
POST /users(.:format) users/registrations#create
new_user_confirmation GET /users/confirmation/new(.:format) users/confirmations#new
user_confirmation GET /users/confirmation(.:format) users/confirmations#show
POST /users/confirmation(.:format) users/confirmations#create
sign_in GET /sign_in(.:format) users/sessions#new
sign_out DELETE /sign_out(.:format) users/sessions#destroy
users GET /users(.:format) users#index
user GET /users/:id(.:format) users#show

Answer 1

edit_user_registration GET /users/edit(.:format) users/registrations#edit

とあるように、もともとこのコントローラーはidの引数を取りません。つまり、何もしなくても現在ログインしているユーザー専用です。

Comments

[退会済みユーザー]

ご回答ありがとうございます！謎が解けてスッキリしました！
つまり、correct_userを定義する必要はなく、自動的にログインユーザーのプロフィールしか編集できないように設計されているということなんですね。
この問題に関係がありそうな記事がヒットしなかたことにも納得しました笑

ご丁寧に教えていただきましてありがとうございました！