From 6a7e82c44ce258a8fff2170d58e061b87990f2fb Mon Sep 17 00:00:00 2001 From: Joffrey JAFFEUX Date: Tue, 11 Feb 2020 17:21:03 +0100 Subject: [PATCH] DEV: Apply rubocop (#8926) --- app/controllers/webhooks_controller.rb | 2 +- spec/requests/webhooks_controller_spec.rb | 17 ++++++++--------- 2 files changed, 9 insertions(+), 10 deletions(-) diff --git a/app/controllers/webhooks_controller.rb b/app/controllers/webhooks_controller.rb index 936b0bfc0c3..320abe17feb 100644 --- a/app/controllers/webhooks_controller.rb +++ b/app/controllers/webhooks_controller.rb @@ -69,7 +69,7 @@ class WebhooksController < ActionController::Base message_id = params["MessageID"] to_address = params["Email"] - type = params["Type"] + type = params["Type"] case type when "HardBounce", "SpamNotification", "SpamComplaint" process_bounce(message_id, to_address, SiteSetting.hard_bounce_score) diff --git a/spec/requests/webhooks_controller_spec.rb b/spec/requests/webhooks_controller_spec.rb index 79cc1f976e9..7772e0eb186 100644 --- a/spec/requests/webhooks_controller_spec.rb +++ b/spec/requests/webhooks_controller_spec.rb @@ -144,10 +144,10 @@ describe WebhooksController do email_log = Fabricate(:email_log, user: user, message_id: message_id, to_address: email) post "/webhooks/postmark.json", params: { - "Type"=> "HardBounce", - "MessageID"=> message_id, - "Email"=> email - } + "Type" => "HardBounce", + "MessageID" => message_id, + "Email" => email + } expect(response.status).to eq(200) email_log.reload @@ -159,10 +159,10 @@ describe WebhooksController do email_log = Fabricate(:email_log, user: user, message_id: message_id, to_address: email) post "/webhooks/postmark.json", params: { - "Type"=> "SoftBounce", - "MessageID"=> message_id, - "Email"=> email - } + "Type" => "SoftBounce", + "MessageID" => message_id, + "Email" => email + } expect(response.status).to eq(200) email_log.reload @@ -171,7 +171,6 @@ describe WebhooksController do end end - context "sparkpost" do it "works" do user = Fabricate(:user, email: email)