letとlet!の挙動の違いがわかりません(requestspecでエラーが発生する)

Question

##現状
現在、Railsでアプリケーションを作成しています。
RSpecのrequestsスペックにてコントローラーのテストを行っているのですが、destroyアクションのテストで躓いているため、わかる方いましたら教えてください。

挙動としては以下を想定しています。

• 管理者(admin: true) → 削除が行える
• 管理者以外(admin: false) → 削除は行われずトップページにリダイレクトする

以下のコードでテストを行ったのですが、削除が思ったように行われません。(なぜか削除したい時に削除されず、削除して欲しくない時に削除されるという結果が返ってきています。)
トップページへのリダイレクトに関するテストは通ります。

describe '#destroy' do
    let(:admin_user) { FactoryBot.create(:user, admin: true) }
    let(:user) { FactoryBot.create(:user, admin: false) }
    let(:laundry) { FactoryBot.create(:laundry) }

    context '管理者の場合' do
      it '店舗情報を削除できること' do
        sign_in admin_user
        expect {
          delete laundry_path(laundry)
        }.to change { Laundry.count }.by(-1)
      end
    end

    context '認可されていないユーザーの場合' do
      it '店舗情報を削除できないこと' do
        sign_in user
        expect {
          delete laundry_path(laundry)
        }.to_not change { Laundry.count }
      end

      it 'トップページにリダイレクトすること' do
        sign_in user
        delete laundry_path(laundry)
        expect(response).to redirect_to root_path
      end
    end

エラー文↓

1) Laundries #destroy 管理者の場合 店舗情報を削除できること
     Failure/Error:
         expect {
           delete laundry_path(laundry)
       }.to change { Laundry.count }.by(-1)
     
       expected `Laundry.count` to have changed by -1, but was changed by 0
     # ./spec/requests/laundries_spec.rb:226:in `block (4 levels) in <top (required)>'

  2) Laundries #destroy 認可されていないユーザーの場合 店舗情報を削除できないこと
     Failure/Error:
       expect {
         delete laundry_path(laundry)
       }.to_not change { Laundry.count }
     
       expected `Laundry.count` not to have changed, but did change from 0 to 1
     # ./spec/requests/laundries_spec.rb:240:in `block (4 levels) in <top (required)>'

  before_action :check_user, except: [:index, :show, :search]

  def destroy
    laundry = Laundry.find(params[:id])
    if laundry.destroy
      redirect_to root_path, notice: '削除が完了しました'
    else
      redirect_to root_path, alert: '削除が失敗しました'
    end
  end

private

  def check_user
    redirect_to root_path unless user_signed_in? && current_user.admin?
  end

##質問内容
let(:laundry) { FactoryBot.create(:laundry) }をlet!(:laundry) { FactoryBot.create(:laundry) }にするとテストが成功するのですが、なぜlet!にすることで成功するのかがわかりません。

そもそも自分が書いたテストコード自体が間違っているのでしょうか？
わかる方いましたら教えていただきたいです。

よろしくお願いいたします。

##開発環境

• rails (6.0.3.3)
• ruby (2.6.5)
• rspec-rails (4.0.1)

Answer 1

letは遅延評価、let!は事前評価です。

let(:laundry){ FactoryBot.create(:laundry) }
it do
  expect {
    delete laundry_path(laundry)
  }.to change { Laundry.count }
end

上記のコードは

1. Laundry.countが実行される
2. expect { ... }が実行される
3. Laundry.countが実行される

という流れです。

1 の時点でLaundry.countは0
2 でlaundryが遅延評価され作成されるが、削除される
3 でLaundry.countは0

let! にすると最初のLaundry.count が 1 なので、1 が(削除されて) 0 になります。

let(:laundry){ FactoryBot.create(:laundry) }
it do
  expect {
    p "expect"
    delete laundry_path(laundry)
  }.to change { p "change"; Laundry.count }
end

としてテストを走らせると
"change" "expect" "change" と表示されるはずです。

Comments

[退会済みユーザー]

1の「Laundry.countが実行される」の部分の考えが抜けていました。

ありがとうございます。