diff --git a/lib/gitlab/email/handler/create_note_handler.rb b/lib/gitlab/email/handler/create_note_handler.rb index 6964665122346651fb8ffc2c49d068c7eb2cbc5d..06dae31cc27e7ce139c6a7476918df83c34141a8 100644 --- a/lib/gitlab/email/handler/create_note_handler.rb +++ b/lib/gitlab/email/handler/create_note_handler.rb @@ -6,8 +6,7 @@ module Gitlab module Handler class CreateNoteHandler < BaseHandler def can_handle? - # We want to raise SentNotificationNotFoundError for missing key - !!(mail_key.nil? || mail_key =~ /\A\w+\z/) + mail_key =~ /\A\w+\z/ end def execute diff --git a/spec/lib/gitlab/email/handler/create_note_handler_spec.rb b/spec/lib/gitlab/email/handler/create_note_handler_spec.rb index 9b7fb6a1a4b2a845517bfecc3f183f927a1031ac..a2119b0dadfb1619434cc6ffe1c60d33593c3a53 100644 --- a/spec/lib/gitlab/email/handler/create_note_handler_spec.rb +++ b/spec/lib/gitlab/email/handler/create_note_handler_spec.rb @@ -20,8 +20,8 @@ describe Gitlab::Email::Handler::CreateNoteHandler, lib: true do context "when the recipient address doesn't include a mail key" do let(:email_raw) { fixture_file('emails/valid_reply.eml').gsub(mail_key, "") } - it "raises a SentNotificationNotFoundError" do - expect { receiver.execute }.to raise_error(Gitlab::Email::SentNotificationNotFoundError) + it "raises a UnknownIncomingEmail" do + expect { receiver.execute }.to raise_error(Gitlab::Email::UnknownIncomingEmail) end end