RSpec 期待したモデルが作成されない

Question

#実現したい事
RSpecでrecordというメソッドに引数を渡した場合の分岐処理に関するテストを書いています。

条件1
user.record(1) または user.record(-1)とした場合はRecordモデルだけが生成される
条件2
user.record(-1) かつ userにfollowerがいた場合はRecordモデルとNotificationモデルが生成される
というメソッドのテストを書いているのですがNotificationモデルが生成されません。

わかる方いましたらぜひ知恵を貸していただきたいです。
よろしくお願いしますm(__)m

#エラーメッセージ

Failure/Error:
       expect{user.record(-1)}.to change{ Record.count }.by(1)
       .and change{ Notification.count }.by(1)
     
       expected `Notification.count` to have changed by 1, but was changed by 0

user.rb

class User < ApplicationRecord

 has_many :habits, dependent: :destroy
 has_many :records, dependent: :destroy
 has_many :posts, dependent: :destroy

 has_many :active_relationships, class_name:  "Relationship",
                                foreign_key: "follower_id",
                                  dependent:   :destroy
 has_many :passive_relationships, class_name:  "Relationship",
                                 foreign_key: "followed_id",
                                   dependent:   :destroy
 has_many :following, through: :active_relationships, source: :followed
 has_many :followers, through: :passive_relationships, source: :follower
 
 has_many :active_notifications, class_name: 'Notification',
                                 foreign_key: 'visitor_id',
                                 dependent: :destroy
 has_many :passive_notifications, class_name: 'Notification',
                                  foreign_key: 'visited_id',
                                  dependent: :destroy
  
 devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
         
 attachment :image
 
 validates :name, presence: true
 validates :profile, length: {maximum: 80}

  def follow(other_user)
    following << other_user
  end

  def unfollow(other_user)
    active_relationships.find_by(followed_id: other_user.id).destroy
  end

  def following?(other_user)
    following.include?(other_user)
  end
  
  def self.guest
      find_or_create_by!(name: "ゲスト", email: '[e-mail]') do |user|
      user.password = SecureRandom.urlsafe_base64
    end
  end
  
 def record(before_level)
     after_level = self.level
     date = Date.current
     self.records.create(level: after_level,date: date)
     if before_level > after_level && self.followers
     self.followers.each do |f|
     Notification.create(visitor_id: self.id,visited_id: f.id, action: "lose")
 end
end

end

end

user_spec.rb

describe '#record' do
  context 'userのlevelが上がった場合' do
  it 'recordモデルが生成される' do
      expect{user.record(1)}.to change{ Record.count }.by(1)
  end
  end
  context 'userのlevelが下がった場合でfollowerがいない場合' do
  it 'recordモデルが生成される' do
      expect{user.record(-1)}.to change{ Record.count }.by(1)
  end
  end
  context 'userのlevelが下がった場合でfollowersがいる場合' do
  it 'recordモデルとnotificationモデルが生成される' do
      user.passive_relationships.create(follower_id: other_user.id)
      expect{user.record(-1)}.to change{ Record.count }.by(1)
      .and change{ Notification.count }.by(1)
  end
 end
end

userをother_userがフォローしているというデータは出来てます。

    51:   end
    52:   context 'userのlevelが下がった場合でfollowersがいる場合' do
    53:   it 'recordモデルとnotificationモデルが生成される' do
    54:       user.passive_relationships.create(follower_id: other_user.id)
 => 55:       binding.pry
    56:       expect{user.record(-1)}.to change{ Record.count }.by(1)
    57:       .and change{ Notification.count }.by(1)
    58:   end
    59:   end
    60:   

[1] pry(#<RSpec::ExampleGroups::User::Record::UserLevelFollowers>)> user.passive_relationships
=> [#<Relationship:0x00005637f5574508
  id: 1,
  follower_id: 2,
  followed_id: 1,
  created_at: Fri, 04 Dec 2020 16:15:50 JST +09:00,
  updated_at: Fri, 04 Dec 2020 16:15:50 JST +09:00>]
[2] pry(#<RSpec::ExampleGroups::User::Record::UserLevelFollowers>)> 

#バージョン

Rails 6.0.3.4
ruby 2.6.3p62
RSpec 3.10

Answer 1

if before_level > after_level && self.followers という条件の
before_level > after_level が満たされていないだけでは？
methodでのbefore と after が逆みたいに思えるのですが。

Comments

[hanayama]

after_level > before_levelとしたらテストが通りました！
userのlevelが下がった時に通知を送るようにしたかったのですがご指摘頂いた事で、渡す引数がおかしい事に気付き、
user.level -= 1
expect{user.record(0)}.to change{ Record.count }.by(1)
.and change{ Notification.count }.by(1)
とすると無事テストが通りました。
説明不足の中ご指摘いただき本当にありがとうございました！！m(__)m