SPEC: check enqueued jobs size instead of using mocks

SPEC: check enqueued jobs size instead of using mocks

diff --git a/spec/models/web_hook_spec.rb b/spec/models/web_hook_spec.rb
index f424cb0..89beb70 100644
--- a/spec/models/web_hook_spec.rb
+++ b/spec/models/web_hook_spec.rb
@@ -125,13 +125,20 @@ describe WebHook do
     end
 
     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!
         post = PostCreator.create(user, raw: 'post', title: 'topic', skip_validations: true)
         expect(Jobs::EmitWebHookEvent.jobs.length).to eq(0)
 
         WebHook.expects(:generate_payload).times(0)
         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
 
@@ -387,12 +394,6 @@ describe WebHook do
       expect(payload["id"]).to eq(tag.id)
     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
       post = Fabricate(:post)
       admin = Fabricate(:admin)

GitHub sha: bf2059ba

1 Like