[GEM] Handle 413 responses for product check

Author: picandocodigo

elasticsearch/lib/elasticsearch.rb

@@ -86,11 +86,12 @@ def verify_elasticsearch
       begin
         response = elasticsearch_validation_request
       rescue Elastic::Transport::Transport::Errors::Unauthorized,
-             Elastic::Transport::Transport::Errors::Forbidden
+             Elastic::Transport::Transport::Errors::Forbidden,
+             Elastic::Transport::Transport::Errors::RequestEntityTooLarge
         @verified = true
         warn(SECURITY_PRIVILEGES_VALIDATION_WARNING)
         return
-      rescue Elastic::Transport::Transport::Error => e
+      rescue Elastic::Transport::Transport::Error
         warn(VALIDATION_WARNING)
         return
       end

elasticsearch/spec/unit/elasticsearch_product_validation_spec.rb

@@ -98,7 +98,7 @@ def valid_requests_and_expectations
     let(:status) { 413 }
     let(:body) { {}.to_json }
 
-    it 'Does not verify the request and shows a warning' do
+    it 'Verifies the request and shows a warning' do
       stderr      = $stderr
       fake_stderr = StringIO.new
       $stderr     = fake_stderr
@@ -107,18 +107,11 @@ def valid_requests_and_expectations
       assert_not_requested :get, host
       verify_request_stub
       expect { client.info }.to raise_error Elastic::Transport::Transport::Errors::RequestEntityTooLarge
-      assert_not_requested :post, "#{host}/_count"
-      expect(client.instance_variable_get('@verified')).to be false
+      expect(client.instance_variable_get('@verified')).to be true
 
       fake_stderr.rewind
-      expect(fake_stderr.string)
-        .to eq(
-          <<~MSG
-            The client is unable to verify that the server is \
-            Elasticsearch. Some functionality may not be compatible \
-            if the server is running an unsupported product.
-          MSG
-        )
+      expect(fake_stderr.string.delete("\n"))
+        .to eq(Elasticsearch::SECURITY_PRIVILEGES_VALIDATION_WARNING)
     ensure
       $stderr = stderr
     end