CV2-4126 fix missing line (#1871)

* CV2-4126 add more explicit flagging to avoid sending stray callback params into callback consumption function

* updates

* resolve broken function tests

* resolve broken test

* resolve one more issue

* CV2-4126 add test to make sure we trigger line

test/controllers/webhooks_controller_test.rb

@@ -248,10 +248,20 @@ def setup
     assert_equal expectation, response
   end
 
-  test "should process Alegre callback webhook" do
+  test "should process Alegre callback webhook with is_shortcircuited_search_result_callback" do
     CheckSentry.expects(:notify).once
     id = random_number
-    payload = { 'action' => 'audio', 'data' => {'item' => { 'callback_url' => '/presto/receive/add_item', 'id' => id.to_s }} }
+    payload = { 'action' => 'audio', 'data' => {'is_shortcircuited_search_result_callback' => true, 'item' => { 'callback_url' => '/presto/receive/add_item', 'id' => id.to_s }} }
+    Bot::Alegre.stubs(:process_alegre_callback).returns({})
+    post :index, params: { name: :alegre, token: CheckConfig.get('alegre_token') }.merge(payload)
+    assert_equal '200', response.code
+    assert_match /success/, response.body
+  end
+
+  test "should process Alegre callback webhook with is_search_result_callback" do
+    CheckSentry.expects(:notify).once
+    id = random_number
+    payload = { 'action' => 'audio', 'data' => {'is_search_result_callback' => true, 'item' => { 'callback_url' => '/presto/receive/add_item', 'id' => id.to_s }} }
     Bot::Alegre.stubs(:process_alegre_callback).returns({})
     post :index, params: { name: :alegre, token: CheckConfig.get('alegre_token') }.merge(payload)
     assert_equal '200', response.code