Only respond with an emoji (eyes) when it makes sense

Author: arfon

app/responders/check_references_responder.rb

@@ -13,7 +13,7 @@ def process_message(message)
     if target_repo_value.empty?
       respond("I couldn't find the URL for the target repository")
     else
-      react_to_comment
+      react_to_comment("eyes")
       DOIWorker.perform_async(serializable(locals), target_repo_value, branch_name_value)
     end
   end

app/responders/external_service_responder.rb

@@ -13,7 +13,6 @@ def define_listening
 
   def process_message(message)
     respond(params[:message]) if params[:message]
-    react_to_comment
     ExternalServiceWorker.perform_async(serializable(params), serializable(locals))
   end
 

app/responders/github_action_responder.rb

@@ -30,7 +30,7 @@ def process_message(message)
     parameters = {}.merge(inputs, mapped_parameters)
 
     if trigger_workflow(workflow_repo, workflow_name, parameters, ref)
-      react_to_comment
+      react_to_comment("eyes")
       respond(params[:message]) if params[:message]
       process_labeling
     end

app/responders/repo_checks_responder.rb

@@ -13,7 +13,7 @@ def process_message(message)
     if target_repo_value.empty?
       respond("I couldn't find the URL for the target repository")
     else
-      react_to_comment
+      react_to_comment("eyes")
       RepoChecksWorker.perform_async(serializable(locals), target_repo_value, branch_name_value, serializable(params[:checks]))
     end
   end

spec/responders/check_references_responder_spec.rb

@@ -40,7 +40,7 @@
 
     it "should react to the comment when dispatching worker" do
       @responder.context.issue_body += "<!--target-repository-->PAPERURL<!--end-target-repository-->"
-      expect(@responder).to receive(:react_to_comment)
+      expect(@responder).to receive(:react_to_comment).with("eyes")
       @responder.process_message("@botsci check references")
     end
 

spec/responders/external_service_responder_spec.rb

@@ -53,8 +53,8 @@
       expect { @responder.process_message("") }.to change(ExternalServiceWorker.jobs, :size).by(1)
     end
 
-    it "should react to the comment" do
-      expect(@responder).to receive(:react_to_comment)
+    it "should not react to the comment" do
+      expect(@responder).to_not receive(:react_to_comment)
       @responder.process_message("")
     end
 

spec/responders/github_action_responder_spec.rb

@@ -79,7 +79,7 @@
     end
 
     it "should react to the comment when workflow is triggered" do
-      expect(@responder).to receive(:react_to_comment)
+      expect(@responder).to receive(:react_to_comment).with("eyes")
       @responder.process_message("")
     end
 

spec/responders/repo_checks_responder_spec.rb

@@ -43,7 +43,7 @@
     end
 
     it "should react to the comment when dispatching worker" do
-      expect(@responder).to receive(:react_to_comment)
+      expect(@responder).to receive(:react_to_comment).with("eyes")
       @responder.process_message("@botsci check repository")
     end