[Test Runner] Updates error catching, skips disallowed scripted metrics tests

Author: picandocodigo

elasticsearch-api/api-spec-testing/rspec_matchers.rb

@@ -106,7 +106,6 @@
 RSpec::Matchers.define :match_gt_field do |expected_pairs, test|
   match do |response|
     expected_pairs.all? do |expected_key, expected_value|
-
       split_key = TestFile::Test.split_and_parse_key(expected_key).collect do |k|
         test.get_cached_value(k)
       end
@@ -128,7 +127,6 @@
 RSpec::Matchers.define :match_lte_field do |expected_pairs, test|
   match do |response|
     expected_pairs.all? do |expected_key, expected_value|
-
       split_key = TestFile::Test.split_and_parse_key(expected_key).collect do |k|
         test.get_cached_value(k)
       end
@@ -150,7 +148,6 @@
 RSpec::Matchers.define :match_lt_field do |expected_pairs, test|
   match do |response|
     expected_pairs.all? do |expected_key, expected_value|
-
       split_key = TestFile::Test.split_and_parse_key(expected_key).collect do |k|
         test.get_cached_value(k)
       end
@@ -302,7 +299,8 @@ def compare_string_response(expected_string, response)
     # Remove surrounding '/' in string representing Regex
     expected_error = expected_error.chomp("/")
     expected_error = expected_error[1..-1] if expected_error =~ /^\//
-    message = actual_error.message.tr("\\","")
+
+    message = actual_error.message.tr('\\', '')
 
     case expected_error
     when 'request_timeout'
@@ -317,12 +315,12 @@ def compare_string_response(expected_string, response)
       message =~ /\[400\]/
     when 'param'
       message =~ /\[400\]/ ||
-          actual_error.is_a?(ArgumentError)
+        actual_error.is_a?(ArgumentError)
     when 'unauthorized'
       actual_error.is_a?(Elastic::Transport::Transport::Errors::Unauthorized)
     when 'forbidden'
       actual_error.is_a?(Elastic::Transport::Transport::Errors::Forbidden)
-    when /error parsing field/
+    when /error parsing field/, /illegal_argument_exception/
       message =~ /\[400\]/ ||
         actual_error.is_a?(Elastic::Transport::Transport::Errors::BadRequest)
     when /NullPointerException/

elasticsearch-api/spec/rest_api/skipped_tests_free.yml

@@ -107,3 +107,12 @@
 -
   :file: 'search.vectors/180_update_dense_vector_type.yml'
   :description: 'Index, update and merge'
+-
+  :file: 'search/330_disallow_scripted_metrics.yml'
+  :description: 'all scripts allowed by default'
+-
+  :file: 'search/330_disallow_scripted_metrics.yml'
+  :description: 'disallow all for empty allow lists'
+-
+  :file: 'search/330_disallow_scripted_metrics.yml'
+  :description: 'allowed stored scripts do not affect allowed inline scripts'