Make responder resilient to multiple spaces around username

Author: xuanxu

app/responders/reviewers_list_responder.rb

@@ -6,7 +6,7 @@ class ReviewersListResponder < 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/reviewers_list_responder_spec.rb

@@ -15,13 +15,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