SPEC: check enqueued jobs size instead of using mocks

This commit is contained in:
Vinoth Kannan 2019-02-19 13:14:06 +05:30
parent 76696b22fe
commit bf2059baf5
1 changed files with 8 additions and 7 deletions

View File

@ -125,13 +125,20 @@ describe WebHook do
end end
describe 'when there are no active hooks' do describe 'when there are no active hooks' do
it 'should not generate payload and enqueue anything' do it 'should not generate payload and enqueue anything for topic events' do
topic_web_hook.destroy! topic_web_hook.destroy!
post = PostCreator.create(user, raw: 'post', title: 'topic', skip_validations: true) post = PostCreator.create(user, raw: 'post', title: 'topic', skip_validations: true)
expect(Jobs::EmitWebHookEvent.jobs.length).to eq(0) expect(Jobs::EmitWebHookEvent.jobs.length).to eq(0)
WebHook.expects(:generate_payload).times(0) WebHook.expects(:generate_payload).times(0)
PostDestroyer.new(admin, post).destroy PostDestroyer.new(admin, post).destroy
expect(Jobs::EmitWebHookEvent.jobs.length).to eq(0)
end
it 'should not enqueue anything for tag events' do
tag = Fabricate(:tag)
tag.destroy!
expect(Jobs::EmitWebHookEvent.jobs.length).to eq(0)
end end
end end
@ -387,12 +394,6 @@ describe WebHook do
expect(payload["id"]).to eq(tag.id) expect(payload["id"]).to eq(tag.id)
end end
it 'should not generate payload if webhooks not exist' do
WebHook.expects(:generate_payload).times(0)
tag = Fabricate(:tag)
tag.destroy!
end
it 'should enqueue the right hooks for flag events' do it 'should enqueue the right hooks for flag events' do
post = Fabricate(:post) post = Fabricate(:post)
admin = Fabricate(:admin) admin = Fabricate(:admin)