elasticsearch-api 7.15.0 → 8.0.0.pre1

data/spec/elasticsearch/api/actions/create_document_spec.rb

@@ -22,7 +22,7 @@ describe 'client#create_document' do
   let(:expected_args) do
     [
         'PUT',
-        'foo/bar/123',
+        'foo/_doc/123',
         { op_type: 'create' },
         { foo: 'bar' },
         {}
@@ -30,7 +30,7 @@ describe 'client#create_document' do
   end
 
   it 'performs the request' do
-    expect(client_double.create(index: 'foo', type: 'bar', id: '123', body: { foo: 'bar'})).to eq({})
+    expect(client_double.create(index: 'foo', id: '123', body: { foo: 'bar'})).to be_a Elasticsearch::API::Response
   end
 
   context 'when the request needs to be URL-escaped' do
@@ -38,7 +38,7 @@ describe 'client#create_document' do
     let(:expected_args) do
       [
           'PUT',
-          'foo/bar%2Fbam/123',
+          'foo/_doc/123',
           { op_type: 'create' },
           {},
           {}
@@ -46,7 +46,7 @@ describe 'client#create_document' do
     end
 
     it 'performs the request' do
-      expect(client_double.create(index: 'foo', type: 'bar/bam', id: '123', body: {})).to eq({})
+      expect(client_double.create(index: 'foo', id: '123', body: {})).to be_a Elasticsearch::API::Response
     end
   end
 
@@ -55,7 +55,7 @@ describe 'client#create_document' do
     let(:expected_args) do
       [
           'PUT',
-          'foo/bar/1',
+          'foo/_doc/1',
           { op_type: 'create' },
           { foo: 'bar' },
           {}
@@ -63,7 +63,7 @@ describe 'client#create_document' do
     end
 
     it 'updates the arguments with the `op_type`' do
-      expect(client_double.create(index: 'foo', type: 'bar', id: 1, body: { foo: 'bar' })).to eq({})
+      expect(client_double.create(index: 'foo', id: 1, body: { foo: 'bar' })).to be_a Elasticsearch::API::Response
     end
   end
 
@@ -72,7 +72,7 @@ describe 'client#create_document' do
     let(:expected_args) do
       [
           'POST',
-          'foo/bar',
+          'foo/_doc',
           { },
           { foo: 'bar' },
           {}
@@ -80,7 +80,7 @@ describe 'client#create_document' do
     end
 
     it 'updates the arguments with the `op_type`' do
-      expect(client_double.create(index: 'foo', type: 'bar', body: { foo: 'bar' })).to eq({})
+      expect(client_double.create(index: 'foo', body: { foo: 'bar' })).to be_a Elasticsearch::API::Response
     end
   end
 end

data/spec/elasticsearch/api/actions/dangling_indices/delete_dangling_indices_spec.rb

@@ -31,7 +31,7 @@ describe 'dangling_indices#delete_dangling_index' do
   it 'performs the request' do
     expect(
       client_double.dangling_indices.delete_dangling_index(index_uuid: 'foo')
-    ).to eq({})
+    ).to be_a Elasticsearch::API::Response
   end
 
   context 'when no index_uuid is specified' do

data/spec/elasticsearch/api/actions/dangling_indices/import_dangling_indices_spec.rb

@@ -31,7 +31,7 @@ describe 'dangling_indices#import_dangling_index' do
   it 'performs the request' do
     expect(
       client_double.dangling_indices.import_dangling_index(index_uuid: 'foo')
-    ).to eq({})
+    ).to be_a Elasticsearch::API::Response
   end
 
   context 'when no index_uuid is specified' do

data/spec/elasticsearch/api/actions/dangling_indices/list_dangling_indices_spec.rb

@@ -31,6 +31,6 @@ describe 'dangling_indices#list_dangling_index' do
   it 'performs the request' do
     expect(
       client_double.dangling_indices.list_dangling_indices
-    ).to eq({})
+    ).to be_a Elasticsearch::API::Response
   end
 end

data/spec/elasticsearch/api/actions/data_frame/update_data_frame_transform_spec.rb

@@ -0,0 +1,81 @@
+# 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#update_transform' do
+
+  let(:expected_args) do
+    [
+      'POST',
+      '_transform/foo/_update',
+      params,
+      {},
+      {}
+    ]
+  end
+
+  let(:params) do
+    {}
+  end
+
+  it 'performs the request' do
+    expect(
+      client_double.transform
+        .update_transform(transform_id: 'foo', body: {})
+    ).to be_a Elasticsearch::API::Response
+  end
+
+  context 'when body is not provided' do
+    let(:client) do
+      Class.new { include Elasticsearch::API }.new
+    end
+
+    it 'raises an exception' do
+      expect {
+        client.transform.update_transform(transform_id: 'foo')
+      }.to raise_exception(ArgumentError)
+    end
+  end
+
+  context 'when a transform_id is not provided' do
+
+    let(:client) do
+      Class.new { include Elasticsearch::API }.new
+    end
+
+    it 'raises an exception' do
+      expect {
+        client.transform.update_transform(body: {})
+      }.to raise_exception(ArgumentError)
+    end
+  end
+
+  context 'when params are specified' do
+
+    let(:params) do
+      { defer_validation: true }
+    end
+
+    it 'performs the request' do
+      expect(
+        client_double.transform
+          .update_transform(transform_id: 'foo', body: {}, defer_validation: true)
+      ).to be_a Elasticsearch::API::Response
+    end
+  end
+end

data/spec/elasticsearch/api/actions/delete_by_query_spec.rb

@@ -36,24 +36,7 @@ describe 'client#delete_by_query' do
   end
 
   it 'performs the request' do
-    expect(client_double.delete_by_query(index: 'foo', body: { term: {} })).to eq({})
-  end
-
-  context 'when the type argument is provided' do
-
-    let(:expected_args) do
-      [
-        'POST',
-        'foo/tweet,post/_delete_by_query',
-        {},
-        { term: {} },
-        {}
-      ]
-    end
-
-    it 'performs the request' do
-      expect(client_double.delete_by_query(index: 'foo', type: ['tweet', 'post'], body: { term: {} })).to eq({})
-    end
+    expect(client_double.delete_by_query(index: 'foo', body: { term: {} })).to be_a Elasticsearch::API::Response
   end
 
   context 'when a query is provided' do
@@ -74,7 +57,7 @@ describe 'client#delete_by_query' do
           q: 'foo:bar',
           body: { query: 'query' }
         )
-      ).to eq({})
+      ).to be_a Elasticsearch::API::Response
     end
   end
 end

data/spec/elasticsearch/api/actions/delete_document_spec.rb

@@ -50,31 +50,20 @@ describe 'client#delete' do
   end
 
   it 'performs the request' do
-    expect(client_double.delete(index: 'foo', type: 'bar', id: '1')).to eq({})
+    expect(client_double.delete(index: 'foo', type: 'bar', id: '1')).to be_a Elasticsearch::API::Response
   end
 
   context 'when url params are provided' do
-
     let(:params) do
       { routing: 'abc123' }
     end
 
     it 'performs the request' do
-      expect(client_double.delete(index: 'foo', type: 'bar', id: '1', routing: 'abc123')).to eq({})
-    end
-  end
-
-  context 'when invalid url params are provided' do
-
-    it 'raises an ArgumentError' do
-      expect {
-        client.delete(index: 'foo', type: 'bar', id: '1', qwertypoiuy: 'asdflkjhg')
-      }.to raise_exception(ArgumentError)
+      expect(client_double.delete(index: 'foo', type: 'bar', id: '1', routing: 'abc123')).to be_a Elasticsearch::API::Response
     end
   end
 
   context 'when the url params need to be escaped' do
-
     let(:expected_args) do
       [
           'DELETE',
@@ -86,12 +75,11 @@ describe 'client#delete' do
     end
 
     it 'escapes the url params' do
-      expect(client_double.delete(index: 'foo^bar', type: 'bar/bam', id: 1)).to eq({})
+      expect(client_double.delete(index: 'foo^bar', type: 'bar/bam', id: 1)).to be_a Elasticsearch::API::Response
     end
   end
 
   context 'when the index is not found' do
-
     before do
       expect(client).to receive(:perform_request).and_raise(NotFound)
     end
@@ -103,7 +91,6 @@ describe 'client#delete' do
     end
 
     context 'when the :ignore option is provided' do
-
       it 'does not raise the NotFound exception' do
         expect(client.delete(index: 'foo', type: 'bar', id: 1, ignore: 404)).to eq(false)
       end

data/spec/elasticsearch/api/actions/delete_script_spec.rb

@@ -31,7 +31,7 @@ describe 'client#delete_script' do
     end
 
     it 'performs the request' do
-      expect(client_double.delete_script(id: 'foo')).to eq({})
+      expect(client_double.delete_script(id: 'foo')).to be_a Elasticsearch::API::Response
     end
   end
 end

data/spec/elasticsearch/api/actions/exists_document_spec.rb

@@ -34,7 +34,7 @@ describe 'client#exists' do
   end
 
   let(:url) do
-    'foo/bar/1'
+    'foo/_doc/1'
   end
 
   let(:client) do
@@ -43,29 +43,18 @@ describe 'client#exists' do
 
   it 'requires the :index argument' do
     expect {
-      client.exists(type: 'bar', id: '1')
+      client.exists(id: '1')
     }.to raise_exception(ArgumentError)
   end
 
   it 'requires the :id argument' do
     expect {
-      client.exists(index: 'foo', type: 'bar')
+      client.exists(index: 'foo')
     }.to raise_exception(ArgumentError)
   end
 
-  context 'when the type parameter is not provided' do
-
-    let(:url) do
-      'foo/_doc/1'
-    end
-
-    it 'performs the request' do
-      expect(client_double.exists(index: 'foo', id: '1')).to eq(true)
-    end
-  end
-
   it 'is aliased to a predicated method' do
-    expect(client_double.exists?(index: 'foo', type: 'bar', id: '1')).to eq(true)
+    expect(client_double.exists?(index: 'foo', id: '1')).to eq(true)
   end
 
   context 'when URL parameters are provided' do
@@ -75,18 +64,18 @@ describe 'client#exists' do
     end
 
     it 'passes the parameters' do
-      expect(client_double.exists(index: 'foo', type: 'bar', id: '1', routing: 'abc123')).to eq(true)
+      expect(client_double.exists(index: 'foo', id: '1', routing: 'abc123')).to eq(true)
     end
   end
 
   context 'when the request needs to be URL-escaped' do
 
     let(:url) do
-      'foo/bar%2Fbam/1'
+      'foo/_doc/1'
     end
 
     it 'URL-escapes the characters' do
-      expect(client_double.exists(index: 'foo', type: 'bar/bam', id: '1')).to eq(true)
+      expect(client_double.exists(index: 'foo', id: '1')).to eq(true)
     end
   end
 
@@ -97,7 +86,7 @@ describe 'client#exists' do
     end
 
     it 'returns false' do
-      expect(client_double.exists(index: 'foo', type: 'bar', id: '1')).to eq(false)
+      expect(client_double.exists(index: 'foo', id: '1')).to eq(false)
     end
   end
 
@@ -108,7 +97,7 @@ describe 'client#exists' do
     end
 
     it 'returns false' do
-      expect(client_double.exists(index: 'foo', type: 'bar', id: '1')).to eq(false)
+      expect(client_double.exists(index: 'foo', id: '1')).to eq(false)
     end
   end
 
@@ -120,7 +109,7 @@ describe 'client#exists' do
 
     it 'raises the error' do
       expect {
-        client_double.exists(index: 'foo', type: 'bar', id: '1')
+        client_double.exists(index: 'foo', id: '1')
       }.to raise_exception(StandardError)
     end
   end

data/spec/elasticsearch/api/actions/explain_document_spec.rb

@@ -39,7 +39,7 @@ describe 'client#explain' do
   end
 
   let(:url) do
-    'foo/bar/1/_explain'
+    'foo/_explain/1'
   end
 
   let(:client) do
@@ -48,18 +48,18 @@ describe 'client#explain' do
 
   it 'requires the :index argument' do
     expect {
-      client.explain(type: 'bar', id: '1')
+      client.explain(id: '1')
     }.to raise_exception(ArgumentError)
   end
 
   it 'requires the :id argument' do
     expect {
-      client.explain(index: 'foo', type: 'bar')
+      client.explain(index: 'foo')
     }.to raise_exception(ArgumentError)
   end
-  
+
   it 'performs the request' do
-    expect(client_double.explain(index: 'foo', type: 'bar', id: 1, body: {})).to eq({})
+    expect(client_double.explain(index: 'foo', id: 1, body: {})).to be_a Elasticsearch::API::Response
   end
 
   context 'when a query is provided' do
@@ -73,7 +73,7 @@ describe 'client#explain' do
     end
 
     it 'passes the query' do
-      expect(client_double.explain(index: 'foo', type: 'bar', id: '1', q: 'abc123')).to eq({})
+      expect(client_double.explain(index: 'foo', id: '1', q: 'abc123')).to be_a Elasticsearch::API::Response
     end
   end
 
@@ -83,17 +83,17 @@ describe 'client#explain' do
     end
 
     it 'passes the query definition' do
-      expect(client_double.explain(index: 'foo', type: 'bar', id: '1', body: { query: { match: {} } })).to eq({})
+      expect(client_double.explain(index: 'foo', id: '1', body: { query: { match: {} } })).to be_a Elasticsearch::API::Response
     end
   end
 
   context 'when the request needs to be URL-escaped' do
     let(:url) do
-      'foo%5Ebar/bar%2Fbam/1/_explain'
+      'foo%5Ebar/_explain/1'
     end
 
     it 'URL-escapes the parts' do
-      expect(client_double.explain(index: 'foo^bar', type: 'bar/bam', id: '1', body: { })).to eq({})
+      expect(client_double.explain(index: 'foo^bar', id: '1', body: { })).to be_a Elasticsearch::API::Response
     end
   end
 end

data/spec/elasticsearch/api/actions/features/get_features_spec.rb

@@ -17,18 +17,22 @@
 
 require 'spec_helper'
 
-describe 'client.featues#get_features' do
+describe 'client.features#get_features' do
   let(:expected_args) do
     [
-      'GET',
-      '_features',
-      {},
-      nil,
-      {}
+        'GET',
+        '_features',
+        {},
+        nil,
+        {}
     ]
   end
 
+  let(:client) do
+    Class.new { include Elasticsearch::API }.new
+  end
+
   it 'performs the request' do
-    expect(client_double.features.get_features).to eq({})
+    expect(client_double.features.get_features).to be_a Elasticsearch::API::Response
   end
 end

data/spec/elasticsearch/api/actions/features/reset_features_spec.rb

@@ -17,18 +17,22 @@
 
 require 'spec_helper'
 
-describe 'client.featues#reset_features' do
+describe 'client.features#reset_features' do
   let(:expected_args) do
     [
-      'POST',
-      '_features/_reset',
-      {},
-      nil,
-      {}
+        'POST',
+        '_features/_reset',
+        {},
+        nil,
+        {}
     ]
   end
 
+  let(:client) do
+    Class.new { include Elasticsearch::API }.new
+  end
+
   it 'performs the request' do
-    expect(client_double.features.reset_features).to eq({})
+    expect(client_double.features.reset_features).to be_a Elasticsearch::API::Response
   end
 end

data/spec/elasticsearch/api/actions/field_caps_spec.rb

@@ -30,6 +30,6 @@ describe 'client#field_caps' do
   end
 
   it 'performs the request' do
-    expect(client_double.field_caps(index: 'foo', fields: 'bar')).to eq({})
+    expect(client_double.field_caps(index: 'foo', fields: 'bar')).to be_a Elasticsearch::API::Response
   end
 end

data/spec/elasticsearch/api/actions/fleet/global_checkpoints_spec.rb

@@ -0,0 +1,44 @@
+# 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#fleet.global_checkpoints' do
+  let(:expected_args) do
+    [
+      'GET',
+      'foo/_fleet/global_checkpoints',
+      {},
+      nil,
+      {}
+    ]
+  end
+
+  it 'performs the request' do
+    expect(client_double.fleet.global_checkpoints(index: 'foo')).to be_a Elasticsearch::API::Response
+  end
+
+  let(:client) do
+    Class.new { include Elasticsearch::API }.new
+  end
+
+  it 'requires the :index argument' do
+    expect {
+      client.fleet.global_checkpoints
+    }.to raise_exception(ArgumentError)
+  end
+end

data/spec/elasticsearch/api/actions/get_document_source_spec.rb

@@ -34,7 +34,7 @@ describe 'client#get_source' do
   end
 
   let(:url) do
-    'foo/bar/1/_source'
+    'foo/_source/1'
   end
 
   let(:client) do
@@ -43,46 +43,34 @@ describe 'client#get_source' do
 
   it 'requires the :index argument' do
     expect {
-      client.get_source(type: 'bar', id: '1')
+      client.get_source(id: '1')
     }.to raise_exception(ArgumentError)
   end
 
   it 'requires the :id argument' do
     expect {
-      client.get_source(index: 'foo', type: 'bar')
+      client.get_source(index: 'foo')
     }.to raise_exception(ArgumentError)
   end
 
-  context 'when the type parameter is not provided' do
-
-    let(:url) do
-      'foo/_source/1'
-    end
-
-    it 'performs the request' do
-      expect(client_double.get_source(index: 'foo', id: '1')).to eq({})
-    end
-  end
-
   context 'when URL parameters are provided' do
-
     let(:params) do
       { routing: 'abc123' }
     end
 
     it 'Passes the URL params' do
-      expect(client_double.get_source(index: 'foo', type: 'bar', id: '1', routing: 'abc123')).to eq({})
+      expect(client_double.get_source(index: 'foo', id: '1', routing: 'abc123')).to be_a Elasticsearch::API::Response
     end
   end
 
   context 'when the request needs to be URL-escaped' do
 
     let(:url) do
-      'foo%5Ebar/bar%2Fbam/1/_source'
+      'foo%5Ebar/_source/1'
     end
 
     it 'URL-escapes the parts' do
-      expect(client_double.get_source(index: 'foo^bar', type: 'bar/bam', id: '1')).to eq({})
+      expect(client_double.get_source(index: 'foo^bar', id: '1')).to be_a Elasticsearch::API::Response
     end
   end