「管理者が承認していない投稿は見れない」というテストを書きたいです。

Question

前提・実現したいこと

回答者の皆様

現在で駐輪場の検索アプリを作っており、 管理者が承認していない投稿は投稿者以外閲覧ができないという機能を実装しました。
実装後にRspecでテストを書いたのですが、テストが通らず困っております。
解決策等ご教示頂ければと思い、こちらで質問させて頂きます。
お忙しい中誠に恐れ入りますが、ご回答いただけると幸いです。

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

1) Parkings GET /show 投稿者と閲覧者が違うユーザーの場合 投稿が承認されていない場合 ホーム画面にリダイレクトされること
     Failure/Error: expect(response).to have_http_status(302)
       expected the response to have status code 302 but it was 200
     # ./spec/requests/parkings_request_spec.rb:51:in `block (5 levels) in <top (required)>'

  2) Parkings GET /show ログインしていない場合 投稿が承認されていない場合 ホーム画面にリダイレクトされること
     Failure/Error: expect(response).to have_http_status(302)
       expected the response to have status code 302 but it was 200

該当のソースコード

spec/requests/parkings_request_spec.rb

require 'rails_helper'

RSpec.describe "Parkings", type: :request do

  # parkingに関してはdestroyアクションをテストする前にデータが保存されていないと
  # エラーが出たため、let!とした。
  let(:user) { create(:user) }
  let!(:parking) { create(:parking, user_id: user.id) }
  let(:other_user) { create(:user) }

  describe "GET /show" do
    context 'ユーザー本人の投稿の場合、かつ投稿が承認されていない場合' do
      before do
        user.confirm
        sign_in user
      end
      it '正常にレスポンスを返すこと' do
        parking.approval = nil
        get parking_url(parking)
        expect(response).to have_http_status(200)
      end
    end
    context '投稿者と閲覧者が違うユーザーの場合' do
      before do
        other_user.confirm
        sign_in other_user
      end
      context '投稿が承認されている場合' do
        it '正常にレスポンスを返すこと' do
          get parking_url(parking)
          expect(response).to have_http_status(200)
        end
      end
      context '投稿が承認されていない場合' do
        it 'ホーム画面にリダイレクトされること' do
          parking.approval = nil
          get parking_url(parking)
          expect(response).to have_http_status(302)
          expect(response.body).to include '投稿が未承認のため、閲覧できません。'
        end
      end
    end
    context 'ログインしていない場合' do
      context '投稿が承認されている場合' do
        it '正常にレスポンスを返すこと' do
          get parking_url(parking)
          expect(response).to have_http_status(200)
        end
      end
      context '投稿が承認されていない場合' do
        it 'ホーム画面にリダイレクトされること' do
          parking.approval = nil
          get parking_url(parking)
          expect(response).to have_http_status(302)
          expect(response.body).to include '投稿が未承認のため、閲覧できません。'
        end
      end
    end
  end
end

app/controllers/parkings_controller.rb

class ParkingsController < ApplicationController
  before_action :authenticate_user!, only: [:new, :create, :edit, :destroy, :update, :search, :favorites, :current_spot_search]
  before_action :permit_update_delete, only: [:edit, :destroy, :update]
  before_action :permit_show, only: [:show]

  def index
    @parkings = Parking.page(params[:page]).per(10)
  end

  def show
   @parking = Parking.find(params[:id])
   # ログインしており、投稿者でない場合かつ投稿が承認されていない場合 
　　if user_signed_in? && @parking.user_id != current_user.id && @parking.approval != 'approval'
      flash[:notice] = '投稿が未承認のため、閲覧できません。'
      redirect_to root_path
　　# ログインしていなくて、かつ投稿が承認されていない場合
    elsif !user_signed_in? && @parking.approval != 'approval'
      flash[:notice] = '投稿が未承認のため、閲覧できません。'
      redirect_to root_path
  end
end

試したこと

• parkings_controller.rbを以下のように修正してみた。

→エラーメッセージに変化はなかった。

class ParkingsController < ApplicationController
  before_action :authenticate_user!, only: [:new, :create, :edit, :destroy, :update, :search, :favorites, :current_spot_search]
  before_action :permit_update_delete, only: [:edit, :destroy, :update]
  before_action :permit_show, only: [:show]

  def index
    # @parkings = Parking.where(approval: 1).page(params[:page]).per(10)
    @parkings = Parking.page(params[:page]).per(10)
  end

  def show
    @parking = Parking.find(params[:id])
  end

  private

  def permit_show
    @parking = Parking.find(params[:id])
    if user_signed_in? && @parking.user_id != current_user.id && @parking.approval != 'approval'
      flash[:notice] = '投稿が未承認のため、閲覧できません。'
      redirect_to root_path
    elsif !user_signed_in? && @parking.approval != 'approval'
      flash[:notice] = '投稿が未承認のため、閲覧できません。'
      redirect_to root_path
    end
  end
end

• ブラウザで動作確認を行った。

→ブラウザでは問題なく動作した。

• システムスペックの変更

→下記のように変更したら、エラーメッセージが変わった。
spec/requests/parkings_request_spec.rb

require 'rails_helper'

RSpec.describe "Parkings", type: :request do

  # parkingに関してはdestroyアクションをテストする前にデータが保存されていないと
  # エラーが出たため、let!とした。
  let(:user) { create(:user) }
  let!(:parking) { create(:parking, user_id: user.id) }
  let(:other_user) { create(:user) }

  describe "GET /show" do
    context 'ユーザー本人の投稿の場合、かつ投稿が承認されていない場合' do
      before do
        user.confirm
        sign_in user
      end
      it '正常にレスポンスを返すこと' do
        parking.approval = nil
        get parking_url(parking)
        expect(response).to have_http_status(200)
      end
    end
    context '投稿者と閲覧者が違うユーザーの場合' do
      before do
        other_user.confirm
        sign_in other_user
      end
      context '投稿が承認されている場合' do
        it '正常にレスポンスを返すこと' do
          get parking_url(parking)
          expect(response).to have_http_status(200)
        end
      end
      context '投稿が承認されていない場合' do
        it 'ホーム画面にリダイレクトされること' do
          parking.approval = nil
          get parking_url(parking)
          expect(response.body).to include '投稿が未承認のため、閲覧できません。'
        end
      end
    end
    context 'ログインしていない場合' do
      context '投稿が承認されている場合' do
        it '正常にレスポンスを返すこと' do
          get parking_url(parking)
          expect(response).to have_http_status(200)
        end
      end
      context '投稿が承認されていない場合' do
        it 'ホーム画面にリダイレクトされること' do
          parking.approval = nil
          get parking_url(parking)
          expect(response.body).to include '投稿が未承認のため、閲覧できません。'
        end
      end
    end
  end
end

変更後のエラーメッセージ

2) Parkings GET /show ログインしていない場合 投稿が承認されていない場合 ホーム画面にリダイレクトされること
     Failure/Error: get parking_url(parking)
     
       expected "<!DOCTYPE html>\n<html class=\"w-full\">\n  <head>\n    <title>Myapp</title>\n    <meta name=\"viewp... sm:text-xl\">Copyright©. All Rights Reserved.</p>\n</footer> \n  </body>\n</html>\n" to include "投稿が未承認のため、閲覧できません。"
       
# 以下、投稿詳細ページのhtmlがエラーメッセージに表示されていた。

どうやらリダイレクトされずに投稿詳細ページが読み込まれてしまっているらしい、、、
→つまり、原因はシステムスペックに存在する？？

おそらくシステムスペックの書き方に原因があるということまでは絞り込めたのですが、そこからどうコードを変えていくかというところで詰まってしまいました。
「Failure/Error: expect(response).to have_http_status(302)」で検索をしたのですが、解決に繋がる記事を見つけることができませんでした。

何か不足している情報等ございましたらご連絡いただけると幸いです。
お忙しい中誠に恐れ入りますが、ご回答の程宜しくお願い致します。

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

• Ruby 2.6.6
• Rails 6.1.0
• Rspec-rails 4.0.2

Answer 1

parking.approval = nil これは読み込まれた　parking　の状態を変更していますが、DBのデータの状態は変化ありません、
parking.update(approval: nil) とすれば通るでしょう。

let (un_approved_parking) てのを用意したほうが分かりやすいかも

Comments

[KouKuma]

winterboum様

お世話になっております。
DB内のデータの状態を変化させる必要があったとのことで、ご指摘いただいたようにテストを書き直したら無事に通りました！

前回に引き続き、今回もご回答いただき誠にありがとうございました！