Change `have_enqueued_sidekiq_job` usage to always make argument expectations explicit (#29974)
parent
013671f29f
commit
8bece467f8
|
@ -24,7 +24,7 @@ RSpec.describe CreateFeaturedTagService do
|
||||||
expect { subject.call(account, tag) }
|
expect { subject.call(account, tag) }
|
||||||
.to change(FeaturedTag, :count).by(1)
|
.to change(FeaturedTag, :count).by(1)
|
||||||
expect(ActivityPub::AccountRawDistributionWorker)
|
expect(ActivityPub::AccountRawDistributionWorker)
|
||||||
.to_not have_enqueued_sidekiq_job
|
.to_not have_enqueued_sidekiq_job(any_args)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -24,7 +24,7 @@ RSpec.describe UnmuteService do
|
||||||
it 'removes the account mute and does not create a merge' do
|
it 'removes the account mute and does not create a merge' do
|
||||||
expect { subject.call(account, target_account) }
|
expect { subject.call(account, target_account) }
|
||||||
.to remove_account_mute
|
.to remove_account_mute
|
||||||
expect(MergeWorker).to_not have_enqueued_sidekiq_job
|
expect(MergeWorker).to_not have_enqueued_sidekiq_job(any_args)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -39,7 +39,7 @@ RSpec.describe UnmuteService do
|
||||||
it 'does nothing and returns' do
|
it 'does nothing and returns' do
|
||||||
expect { subject.call(account, target_account) }
|
expect { subject.call(account, target_account) }
|
||||||
.to_not(change { account.reload.muting?(target_account) })
|
.to_not(change { account.reload.muting?(target_account) })
|
||||||
expect(MergeWorker).to_not have_enqueued_sidekiq_job
|
expect(MergeWorker).to_not have_enqueued_sidekiq_job(any_args)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
Loading…
Reference in New Issue