【RSpec】destroyアクションのテストが通らない

Question

前提・実現したいこと

今、Railsで開発したアプリのテストをRSpecで行っています。

そのアプリには通知機能があり、通知を削除するテストが通らないので、
解決策を教えていただけますと幸いです。

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

NotificationsController
  DELETE #destroy
    ユーザーがログインしている場合
      通知を削除できること (FAILED - 1)

Failures:

  1) NotificationsController DELETE #destroy ユーザーがログインしている場合 通知を削除できること
     Failure/Error: expect{ subject }.to change(Notification, :count).by(-1)
       expected `Notification.count` to have changed by -1, but was changed by 0
     # ./spec/controllers/notifications_controller_spec.rb:52:in `block (4 levels) in <top (required)>'

Finished in 0.24758 seconds (files took 1.48 seconds to load)
1 example, 1 failure

Failed examples:

rspec ./spec/controllers/notifications_controller_spec.rb:50 # NotificationsController DELETE #destroy ユーザーがログインしている場合 通知を削除できること

該当のソースコード

▼notifications_controller_spec.rb

require 'rails_helper'

describe NotificationsController do
  let(:user) { create(:user) }
  let(:notification) { create(:notification) }

  describe 'DELETE #destroy' do
    subject {
      delete :destroy,
      params: {id: notification.id}
    }

    context "ユーザーがログインしている場合" do  
      before do
        login user
      end

      it "通知を削除できること" do
        expect{ subject }.to change(Notification, :count).by(-1)
      end
    end
  end

end

▼spec/factories/notifications.rb

FactoryBot.define do
  factory :notification do
    association :visitor, factory: :user
    association :visited, factory: :user
    checked       { true }
    action        { "message" }
  end
end

▼app/models/notification.rb

class Notification < ApplicationRecord
  default_scope->{order(created_at: :desc)}

  belongs_to :group, optional: true
  belongs_to :visitor, class_name: 'User', foreign_key: 'visitor_id'
  belongs_to :visited, class_name: 'User', foreign_key: 'visited_id'
  ACTION_VALUES = ["message", "follow"]
  validates :action,  presence: true, inclusion: {in: ACTION_VALUES}
  validates :checked, presence: true, inclusion: {in: [true, false]}
end

▼notifications_controller.rb

class NotificationsController < ApplicationController
  before_action :authenticate_user!
  
  def destroy
    notification = Notification.find(params[:id])
    notification.destroy
    redirect_to notifications_path
  end

end

試したこと

test.logのファイルを確認したのですが、間違いがわかりませんでした。こちらがログの画像です。

Answer 1

notifications_controller_spec.rbで以下のように修正したら解決しました。
変更前：let(:notification) { create(:notification) }
変更後：let!(:notification) { create(:notification) }(letの後に!を追加）

Answer 2

subject
expect{ subject }.to change(Notification, :count).by(-1)
としていますが，これだと2回削除リクエストを出してることになると思うので，
expect{ subject }.to change(Notification, :count).by(-1)
だけで，動くのではないでしょうか？

Comments

[kumamin]

ご回答ありがとうございます。
確かにダブってしまっていますね。

ただ、1回目のsubjectを削除してみたのですが、エラー文は変わりませんでした。

[kmdkuk]

なるほど，，，
なんでだろう．
ログ出力も特に変わらなかった感じですかね？

[kumamin]

返信が遅れてしまい、申し訳ございません。
ログも変わっていないと思います。
一応質問に載せたログの画像を更新しておきます。