[API] Tests: Renames integration tests helper, updates some unit tests

Author: picandocodigo

elasticsearch-api/spec/integration/api_key_spec.rb

@@ -18,7 +18,7 @@
 # under the License.
 
 require 'base64'
-require_relative './spec_helper'
+require_relative './integration_helper'
 
 describe 'API keys' do
   before do

elasticsearch-api/spec/integration/health_spec.rb

@@ -17,7 +17,7 @@
 # specific language governing permissions and limitations
 # under the License.
 
-require_relative './spec_helper'
+require_relative './integration_helper'
 
 describe 'Health basic test' do
   it 'performs the request' do

elasticsearch-api/spec/integration/spec_helper.rb renamed to elasticsearch-api/spec/integration/integration_helper.rb

@@ -18,15 +18,14 @@
 require 'spec_helper'
 
 describe 'client.nodes#info' do
-
   let(:expected_args) do
     [
-        'GET',
-        url,
-        params,
-        nil,
-        {},
-        { endpoint: 'nodes.info' }
+      'GET',
+      url,
+      params,
+      nil,
+      {},
+      { endpoint: 'nodes.info' }
     ]
   end
 
@@ -43,7 +42,6 @@
   end
 
   context 'when the node id is specified' do
-
     let(:url) do
       '_nodes/foo'
     end
@@ -65,7 +63,6 @@
   end
 
   context 'when multiple node ids are specified as a list' do
-
     let(:url) do
       '_nodes/A,B,C'
     end
@@ -87,7 +84,6 @@
   end
 
   context 'when multiple node ids are specified as a String' do
-
     let(:url) do
       '_nodes/A,B,C'
     end
@@ -109,7 +105,6 @@
   end
 
   context 'when URL params are specified' do
-
     let(:url) do
       '_nodes'
     end
@@ -124,7 +119,6 @@
   end
 
   context 'when metrics are specified' do
-
     let(:url) do
       '_nodes/http,network'
     end

elasticsearch-api/spec/unit/actions/nodes/info_spec.rb

@@ -0,0 +1,106 @@
+# Licensed to Elasticsearch B.V. under one or more contributor
+# license agreements. See the NOTICE file distributed with
+# this work for additional information regarding copyright
+# ownership. Elasticsearch B.V. licenses this file to you under
+# the Apache License, Version 2.0 (the "License"); you may
+# not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+require 'spec_helper'
+
+describe 'client.nodes#usage' do
+  let(:expected_args) do
+    [
+      'GET',
+      url,
+      params,
+      nil,
+      {},
+      { endpoint: 'nodes.usage' }
+    ]
+  end
+
+  let(:url) do
+    '_nodes/usage'
+  end
+
+  it 'performs the request' do
+    expect(client_double.nodes.usage).to be_a Elasticsearch::API::Response
+  end
+
+  let(:params) do
+    {}
+  end
+
+  context 'when the node id is specified' do
+    let(:url) do
+      '_nodes/foo/usage'
+    end
+
+    let(:expected_args) do
+      [
+        'GET',
+        url,
+        params,
+        nil,
+        {},
+        { defined_params: { node_id: 'foo' }, endpoint: 'nodes.usage' }
+      ]
+    end
+
+    it 'performs the request' do
+      expect(client_double.nodes.usage(node_id: 'foo')).to be_a Elasticsearch::API::Response
+    end
+  end
+
+  context 'when the metric is specified' do
+    let(:url) do
+      '_nodes/usage/metric'
+    end
+
+    let(:expected_args) do
+      [
+        'GET',
+        url,
+        params,
+        nil,
+        {},
+        { defined_params: { metric: 'metric' }, endpoint: 'nodes.usage' }
+      ]
+    end
+
+    it 'performs the request' do
+      expect(client_double.nodes.usage(metric: 'metric')).to be_a Elasticsearch::API::Response
+    end
+  end
+
+  context 'when both are specified' do
+    let(:url) do
+      '_nodes/foo/usage/metric'
+    end
+
+    let(:expected_args) do
+      [
+        'GET',
+        url,
+        params,
+        nil,
+        {},
+        { defined_params: { node_id: 'foo', metric: 'metric' }, endpoint: 'nodes.usage' }
+      ]
+    end
+
+    it 'performs the request' do
+      expect(client_double.nodes.usage(metric: 'metric', node_id: 'foo')).to be_a Elasticsearch::API::Response
+    end
+  end
+end

elasticsearch-api/spec/unit/actions/nodes/usage_spec.rb

@@ -72,7 +72,11 @@
     end
 
     it 'performs the request' do
-      expect(client_double.termvector(index: 'foo', id: '123', body: {})).to be_a Elasticsearch::API::Response
+      message = '[DEPRECATION] `termvector` is deprecated. Please use the plural version, `termvectors` instead.
+'
+      expect do
+        client_double.termvector(index: 'foo', id: '123', body: {})
+      end.to output(message).to_stderr
     end
   end
 end

elasticsearch-api/spec/unit/actions/termvectors_spec.rb

@@ -62,4 +62,8 @@
   it 'returns the body which' do
     expect(response.body).to eq response_body
   end
+
+  it 'returns the body with to_s' do
+    expect(response.to_s).to eq response.body.to_s
+  end
 end