Make responder resilient to multiple spaces around username

Author: xuanxu

app/responders/openjournals/reviewers_list_with_external_call_responder.rb

@@ -8,7 +8,7 @@ class ReviewersListWithExternalCallResponder < Responder
 
     def define_listening
       @event_action = "issue_comment.created"
-      @event_regex = /\A@#{bot_name} (add|remove) (\S+) (to reviewers|from reviewers|as reviewer)\.?\s*$/i
+      @event_regex = /\A@#{bot_name} (add|remove) +(\S+) +(to reviewers|from reviewers|as reviewer)\.?\s*$/i
     end
 
     def process_message(message)

spec/responders/openjournals/reviewers_list_with_external_call_responder_spec.rb

@@ -16,13 +16,18 @@
 
     it "should define regex" do
       expect(@responder.event_regex).to match("@botsci add @arfon to reviewers")
+      expect(@responder.event_regex).to match("@botsci add   @arfon to reviewers")
+      expect(@responder.event_regex).to match("@botsci add @arfon  to reviewers")
+      expect(@responder.event_regex).to match("@botsci add   @arfon    to reviewers")
       expect(@responder.event_regex).to match("@botsci add @arfon as reviewer")
       expect(@responder.event_regex).to match("@botsci add me as reviewer")
       expect(@responder.event_regex).to match("@botsci remove me from reviewers")
       expect(@responder.event_regex).to match("@botsci remove @arfon from reviewers  ")
       expect(@responder.event_regex).to match("@botsci remove @arfon from reviewers  \r\n")
       expect(@responder.event_regex).to match("@botsci remove @arfon from reviewers  \r\n more ")
       expect(@responder.event_regex).to_not match("@botsci add to reviewers")
+      expect(@responder.event_regex).to_not match("@botsci add@arfon to reviewers")
+      expect(@responder.event_regex).to_not match("@botsci add @arfonto reviewers")
       expect(@responder.event_regex).to_not match("@botsci remove   from reviewers")
     end
   end