Make checklist responder case insensitive for the GH handle

Author: xuanxu

app/responders/reviewer_checklist_comment_responder.rb

@@ -22,7 +22,7 @@ def process_message(message)
   end
 
   def sender_in_reviewers_list?
-    reviewers.include?("@#{context.sender}")
+    reviewers.include?("@#{context.sender.downcase}")
   end
 
   def update_checklists_links
@@ -38,7 +38,7 @@ def update_checklists_links
   end
 
   def reviewers
-    @reviewers ||= read_value_from_body("reviewers-list").split(",").map(&:strip)
+    @reviewers ||= read_value_from_body("reviewers-list").split(",").map(&:strip).map(&:downcase)
   end
 
   def checklists_mapping

spec/responders/reviewer_checklist_comment_responder_spec.rb

@@ -51,6 +51,18 @@
         @responder.process_message(@msg)
       end
 
+      it "should be case insensitive for the reviewer's username" do
+        @responder.context[:sender] = "ReVIEwer1"
+
+        expected_locals = { issue_id: 5, issue_author: "opener", bot_name: "botsci", repo: "openjournals/buffy", sender: "ReVIEwer1" }
+        expected_checklist = "Checklist for @reviewer1 \n[] A"
+
+        expect(@responder).to receive(:render_external_template).with("checklist.md", expected_locals).and_return(expected_checklist)
+        expect(@responder).to receive(:update_comment).with(111222, expected_checklist)
+        expect(@responder).to_not receive(:respond)
+        @responder.process_message(@msg)
+      end
+
       it "should not add user checklist if sender is not a reviewer" do
         @responder.context[:sender] = "nonreviewer"