From 904ba266cf10da6479ae715283bc7e2b9f73cc08 Mon Sep 17 00:00:00 2001 From: Vinoth Kannan Date: Sun, 31 Mar 2019 16:28:40 +0530 Subject: [PATCH] SPEC: Add test case in emit_web_hook_event_spec for commit 4c6bfb9 --- spec/jobs/emit_web_hook_event_spec.rb | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/spec/jobs/emit_web_hook_event_spec.rb b/spec/jobs/emit_web_hook_event_spec.rb index 06aadf83de7..099234b3b42 100644 --- a/spec/jobs/emit_web_hook_event_spec.rb +++ b/spec/jobs/emit_web_hook_event_spec.rb @@ -1,4 +1,5 @@ require 'rails_helper' +require 'excon' describe Jobs::EmitWebHookEvent do let(:post_hook) { Fabricate(:web_hook) } @@ -24,6 +25,16 @@ describe Jobs::EmitWebHookEvent do end.to raise_error(Discourse::InvalidParameters) end + it "should not destroy webhook event in case of error" do + Excon::Connection.any_instance.stubs(:post).raises("error") + subject.execute( + web_hook_id: post_hook.id, + payload: { id: post.id }.to_json, + event_type: WebHookEventType::POST + ) + expect(WebHookEvent.last.web_hook_id).to eq(post_hook.id) + end + context 'when the web hook is failed' do before do SiteSetting.retry_web_hook_events = true