質問編集履歴

誤字

2020/07/29 22:21

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -378,234 +378,6 @@
-Ruby : ruby 2.7.1p83
+ruby 2.7.1
-Rails : Rails 6.0.3.2
+Rails 6.0.3.2
-### 実現したいこと
-・自分以外のユーザーのプロフィールを編集できないようにする
-・ユーザーのプロフィールページに置かれたリンクをクリックすることで、編集ページに移動
-### 発生している問題・エラーメッセージ
-上記の機能を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を定義しました。
-ただ、これが原因でエラーが出ているようです。
-```Ruby
-# 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)です。
-```Ruby
-<% provide(:title, @user.name) %>
-<h3>プロフィール</h3>
-<p>名前 : <%= @user.name %></p>
-<p>メールアドレス： <%= @user.email %></p>
-<%= link_to 'プロフィールの編集', edit_user_registration_path(@user) %>
-```
-### routes.rb
-```Ruby
-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
-```
-### 試したこと、確認したこと
-correct_userがなければ編集ページに移動できることは確認しており、その際URLにはユーザーのIDが記載されていました。なので、IDが見つからないからユーザーを検索できないというエラーの意味が
-質問に不備がある場合は遠慮なく伝えていただけると幸いです。どうぞよろしくお願いします。
-### 補足情報（FW/ツールのバージョンなど）
-Ruby : ruby 2.7.1p83
-Rails : Rails 6.0.3.2

記載漏れ

2020/07/29 22:21

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -210,21 +210,395 @@
+### 試したこと
+Rspecでregistrations_controllerのテストをしました。
+そのテストはパスするのですが、そもそも正しいテストを書けているか自信がありません。
+念のため、以下にテスト内容とテストデータ(FactoryBot)を記載します。
+```Ruby
+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
+```
+```Ruby
+FactoryBot.define do
+  factory :user, class: User do
+    name { 'foo' }
+    email { 'foo@test.com' }
+    password { 'foo123' }
+    password_confirmation { 'foo123' }
+    confirmed_at { Date.today }
+  end
+  factory :other_user, class: User do
+    name { 'bar' }
+    email { 'bar@test.com' }
+    password { 'bar123' }
+    password_confirmation { 'bar123' }
+    confirmed_at { Date.today }
+  end
+end
+```
+### 確認したこと
+correct_userがなければ編集ページに移動できることは確認しており、その際URLにはユーザーのIDが記載されていました。そのため、IDが見つからないからユーザーを検索できないというエラーの原因がわかりませんでした。
+1日悩んでわからなかったので、どなたか原因を教えていただけないでしょうか。
+質問に不備がある場合は遠慮なく伝えていただけると幸いです。
+どうぞよろしくお願いします。
+### 補足情報（FW/ツールのバージョンなど）
+Ruby : ruby 2.7.1p83
+Rails : Rails 6.0.3.2
+### 実現したいこと
+・自分以外のユーザーのプロフィールを編集できないようにする
+・ユーザーのプロフィールページに置かれたリンクをクリックすることで、編集ページに移動
+### 発生している問題・エラーメッセージ
+上記の機能を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を定義しました。
+ただ、これが原因でエラーが出ているようです。
+```Ruby
+# 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)です。
+```Ruby
+<% provide(:title, @user.name) %>
+<h3>プロフィール</h3>
+<p>名前 : <%= @user.name %></p>
+<p>メールアドレス： <%= @user.email %></p>
+<%= link_to 'プロフィールの編集', edit_user_registration_path(@user) %>
+```
+### routes.rb
+```Ruby
+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
+```
 ### 試したこと、確認したこと
-correct_userがなければ編集ページに移動できることは確認しており、その際URLにはユーザーのIDが記載されていました。そのため、IDが見つからないからユーザーを検索できないというエラーの原因がわかりませんでした。
+correct_userがなければ編集ページに移動できることは確認しており、その際URLにはユーザーのIDが記載されていました。なので、IDが見つからないからユーザーを検索できないというエラーの意味が
-1日悩んでわからなかったので、どなたか原因を教えていただけないでしょうか。
-質問に不備がある場合は遠慮なく伝えていただけると幸いです。
+質問に不備がある場合は遠慮なく伝えていただけると幸いです。どうぞよろしくお願いします。
-どうぞよろしくお願いします。

説明不足

2020/07/29 22:20

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

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

説明不足

2020/07/29 22:11

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

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

誤字

2020/07/29 22:09

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -208,11 +208,17 @@
-correct_userがなければ編集ページに移動できることは確認しており、その際URLにはユーザーのIDが記載されていました。なので、IDが見つからないからユーザーを検索できないというエラーの意味が
+correct_userがなければ編集ページに移動できることは確認しており、その際URLにはユーザーのIDが記載されていました。そのため、IDが見つからないからユーザーを検索できないというエラーの原因がわかりませんでした。
+1日悩んでわからなかったので、どなたか原因を教えていただけないでしょうか。
-質問に不備がある場合は遠慮なく伝えていただけると幸いです。どうぞよろしくお願いします。
+質問に不備がある場合は遠慮なく伝えていただけると幸いです。
+どうぞよろしくお願いします。