opensearch-api 1.0.0 → 2.0.2

data/spec/opensearch/api/actions/get_document_spec.rb

@@ -43,7 +43,7 @@ describe 'client#get' do
   end
 
   let(:url) do
-    'foo/bar/1'
+    'foo/_doc/1'
   end
 
   let(:client) do
@@ -52,13 +52,13 @@ describe 'client#get' do
 
   it 'requires the :index argument' do
     expect {
-      client.get(type: 'bar', id: '1')
+      client.get(id: '1')
     }.to raise_exception(ArgumentError)
   end
 
   it 'requires the :id argument' do
     expect {
-      client.get(index: 'foo', type: 'bar')
+      client.get(index: 'foo')
     }.to raise_exception(ArgumentError)
   end
 
@@ -80,7 +80,7 @@ describe 'client#get' do
     end
 
     it 'Passes the URL params' do
-      expect(client_double.get(index: 'foo', type: 'bar', id: '1', routing: 'abc123')).to eq({})
+      expect(client_double.get(index: 'foo', id: '1', routing: 'abc123')).to eq({})
     end
   end
 
@@ -88,7 +88,7 @@ describe 'client#get' do
 
     it 'Passes the URL params' do
       expect {
-        client.get(index: 'foo', type: 'bar', id: '1', qwert: 'abc123')
+        client.get(index: 'foo', id: '1', qwert: 'abc123')
       }.to raise_exception(ArgumentError)
     end
   end
@@ -96,11 +96,11 @@ describe 'client#get' do
   context 'when the request needs to be URL-escaped' do
 
     let(:url) do
-      'foo%5Ebar/bar%2Fbam/1'
+      'foo%5Ebar/_doc/1'
     end
 
     it 'URL-escapes the parts' do
-      expect(client_double.get(index: 'foo^bar', type: 'bar/bam', id: '1')).to eq({})
+      expect(client_double.get(index: 'foo^bar', id: '1')).to eq({})
     end
   end
 
@@ -125,7 +125,7 @@ describe 'client#get' do
         end
 
         it 'returns false' do
-          expect(client.get(index: 'foo', type: 'bar', id: '1', ignore: 404)).to eq(false)
+          expect(client.get(index: 'foo', id: '1', ignore: 404)).to eq(false)
         end
       end
     end

data/spec/opensearch/api/actions/index_document_spec.rb

@@ -47,7 +47,7 @@ describe 'client#index' do
   end
 
   let(:url) do
-    'foo/bar'
+    'foo/_doc'
   end
 
   let(:body) do
@@ -60,12 +60,12 @@ describe 'client#index' do
 
   it 'requires the :index argument' do
     expect {
-      client.index(type: 'bar')
+      client.index()
     }.to raise_exception(ArgumentError)
   end
 
   it 'performs the request' do
-    expect(client_double.index(index: 'foo', type: 'bar', body: body)).to eq({})
+    expect(client_double.index(index: 'foo', body: body)).to eq({})
   end
 
   context 'when a specific id is provided' do
@@ -75,11 +75,11 @@ describe 'client#index' do
     end
 
     let(:url) do
-      'foo/bar/1'
+      'foo/_doc/1'
     end
 
     it 'performs the request' do
-      expect(client_double.index(index: 'foo', type: 'bar', id: '1', body: body)).to eq({})
+      expect(client_double.index(index: 'foo', id: '1', body: body)).to eq({})
     end
   end
 
@@ -90,7 +90,7 @@ describe 'client#index' do
     end
 
     let(:url) do
-      'foo/bar'
+      'foo/_doc'
     end
 
     let(:params) do
@@ -98,7 +98,7 @@ describe 'client#index' do
     end
 
     it 'passes the URL params' do
-      expect(client_double.index(index: 'foo', type: 'bar', op_type: 'create', body: body)).to eq({})
+      expect(client_double.index(index: 'foo', op_type: 'create', body: body)).to eq({})
     end
 
     context 'when a specific id is provided' do
@@ -108,7 +108,7 @@ describe 'client#index' do
       end
 
       let(:url) do
-        'foo/bar/1'
+        'foo/_doc/1'
       end
 
       let(:params) do
@@ -116,7 +116,7 @@ describe 'client#index' do
       end
 
       it 'passes the URL params' do
-        expect(client_double.index(index: 'foo', type: 'bar', id: '1', op_type: 'create', body: body)).to eq({})
+        expect(client_double.index(index: 'foo', id: '1', op_type: 'create', body: body)).to eq({})
       end
     end
   end
@@ -127,11 +127,11 @@ describe 'client#index' do
     end
 
     let(:url) do
-      'foo/bar%2Fbam/123'
+      'foo%5Ebar/_doc/123'
     end
 
     it 'URL-escapes the parts' do
-      expect(client_double.index(index: 'foo', type: 'bar/bam', id: '123', body: body)).to eq({})
+      expect(client_double.index(index: 'foo^bar', id: '123', body: body)).to eq({})
     end
   end
 
@@ -139,7 +139,7 @@ describe 'client#index' do
 
     it 'raises and ArgumentError' do
       expect {
-        client.index(index: 'foo', type: 'bar', id: '1', qwerty: 'yuiop')
+        client.index(index: 'foo', id: '1', qwerty: 'yuiop')
       }.to raise_exception(ArgumentError)
     end
   end

data/spec/opensearch/api/actions/indices/get_field_mapping_spec.rb

@@ -56,15 +56,4 @@ describe 'client.cluster#get_field_mapping' do
       expect(client_double.indices.get_field_mapping(index: 'foo', field: 'bam')).to eq({})
     end
   end
-
-  context 'when a type is specified' do
-
-    let(:url) do
-      'foo/_mapping/bar/field/bam'
-    end
-
-    it 'performs the request' do
-      expect(client_double.indices.get_field_mapping(index: 'foo', type: 'bar', field: 'bam')).to eq({})
-    end
-  end
 end

data/spec/opensearch/api/actions/indices/get_mapping_spec.rb

@@ -57,36 +57,25 @@ describe 'client.cluster#get_mapping' do
     end
   end
 
-  context 'when an index and type are specified' do
+  context 'when multiple indices are specified' do
 
     let(:url) do
-      'foo/_mapping/bar'
+      'foo,bar/_mapping'
     end
 
     it 'performs the request' do
-      expect(client_double.indices.get_mapping(index: 'foo', type: 'bar')).to eq({})
-    end
-  end
-
-  context 'when multiple indices and types are specified' do
-
-    let(:url) do
-      'foo,bar/_mapping/bam,baz'
-    end
-
-    it 'performs the request' do
-      expect(client_double.indices.get_mapping(index: ['foo', 'bar'], type: ['bam', 'baz'])).to eq({})
+      expect(client_double.indices.get_mapping(index: ['foo', 'bar'])).to eq({})
     end
   end
 
   context 'when the path must be URL-escaped' do
 
     let(:url) do
-      'foo%5Ebar/_mapping/bar%2Fbam'
+      'foo%5Ebar/_mapping'
     end
 
     it 'performs the request' do
-      expect(client_double.indices.get_mapping(index: 'foo^bar', type: 'bar/bam')).to eq({})
+      expect(client_double.indices.get_mapping(index: 'foo^bar')).to eq({})
     end
   end
 end

data/spec/opensearch/api/actions/indices/put_mapping_spec.rb

@@ -39,7 +39,7 @@ describe 'client.cluster#put_mapping' do
   end
 
   let(:url) do
-    'foo/bar/_mappings'
+    'foo/_mappings'
   end
 
   let(:body) do
@@ -47,20 +47,7 @@ describe 'client.cluster#put_mapping' do
   end
 
   it 'performs the request' do
-    expect(client_double.indices.put_mapping(index: 'foo', type: 'bar', body: {})).to eq({})
-  end
-
-  context 'when there is no type specified' do
-
-    let(:client) do
-      Class.new { include OpenSearch::API }.new
-    end
-
-    it 'raises an exception' do
-      expect {
-        client.indices.put_mapping(type: 'foo')
-      }.to raise_exception(ArgumentError)
-    end
+    expect(client_double.indices.put_mapping(index: 'foo', body: {})).to eq({})
   end
 
   context 'when there is no body specified' do
@@ -71,7 +58,7 @@ describe 'client.cluster#put_mapping' do
 
     it 'raises an exception' do
       expect {
-        client.indices.put_mapping(index: 'foo', type: 'bar')
+        client.indices.put_mapping(index: 'foo')
       }.to raise_exception(ArgumentError)
     end
   end
@@ -83,29 +70,29 @@ describe 'client.cluster#put_mapping' do
     end
 
     it 'performs the request' do
-      expect(client_double.indices.put_mapping(index: 'foo', type: 'bar', body: { filter: 'foo' })).to eq({})
+      expect(client_double.indices.put_mapping(index: 'foo', body: { filter: 'foo' })).to eq({})
     end
   end
 
   context 'when multiple indices are specified' do
 
     let(:url) do
-      'foo,bar/bam/_mappings'
+      'foo,bar/_mappings'
     end
 
     it 'performs the request' do
-      expect(client_double.indices.put_mapping(index: ['foo','bar'], type: 'bam', body: {})).to eq({})
+      expect(client_double.indices.put_mapping(index: ['foo','bar'], body: {})).to eq({})
     end
   end
 
   context 'when the path needs to be URL-escaped' do
 
     let(:url) do
-      'foo%5Ebar/bar%2Fbam/_mappings'
+      'foo%5Ebar/_mappings'
     end
 
     it 'performs the request' do
-      expect(client_double.indices.put_mapping(index: 'foo^bar', type: 'bar/bam', body: {})).to eq({})
+      expect(client_double.indices.put_mapping(index: 'foo^bar', body: {})).to eq({})
     end
   end
 end

data/spec/opensearch/api/actions/indices/validate_query_spec.rb

@@ -64,16 +64,6 @@ describe 'client.cluster#validate_query' do
     end
   end
 
-  context 'when a type and index are specified' do
-    let(:url) do
-      'foo/bar/_validate/query'
-    end
-
-    it 'performs the request' do
-      expect(client_double.indices.validate_query(index: 'foo', type: 'bar')).to eq({})
-    end
-  end
-
   context 'when multiple indicies are specified as a list' do
     let(:url) do
       'foo,bar/_validate/query'

data/spec/opensearch/api/actions/mget_spec.rb

@@ -64,21 +64,6 @@ describe 'client#mget' do
     end
   end
 
-  context 'when a type is specified' do
-
-    let(:url) do
-      'foo/bar/_mget'
-    end
-
-    let(:body) do
-      { ids: [ '1', '2' ]}
-    end
-
-    it 'performs the request' do
-      expect(client_double.mget(index: 'foo', type: 'bar', body: { :ids => [ '1', '2'] })).to eq({})
-    end
-  end
-
   context 'when url parameters are provided' do
 
     let(:params) do
@@ -97,7 +82,7 @@ describe 'client#mget' do
   context 'when the request needs to be URL-escaped' do
 
     let(:url) do
-      'foo%5Ebar/bar%2Fbam/_mget'
+      'foo%5Ebar/_mget'
     end
 
     let(:body) do
@@ -105,7 +90,7 @@ describe 'client#mget' do
     end
 
     it 'performs the request' do
-      expect(client_double.mget(index: 'foo^bar', type: 'bar/bam', body: { :ids => [ '1', '2'] })).to eq({})
+      expect(client_double.mget(index: 'foo^bar', body: { :ids => [ '1', '2'] })).to eq({})
     end
   end
 end

data/spec/opensearch/api/actions/mtermvectors_spec.rb

@@ -30,7 +30,7 @@ describe 'client#mtermvectors' do
   let(:expected_args) do
     [
       method,
-      'my-index/my-type/_mtermvectors',
+      'my-index/_mtermvectors',
       {},
       body,
       {}
@@ -43,13 +43,13 @@ describe 'client#mtermvectors' do
   end
 
   it 'performs the request' do
-    expect(client_double.mtermvectors(index: 'my-index', type: 'my-type', body: { ids: [1, 2, 3] })).to eq({})
+    expect(client_double.mtermvectors(index: 'my-index', body: { ids: [1, 2, 3] })).to eq({})
   end
 
   context 'when a list of ids is passed instead of a body' do
     let(:method) { 'GET' }
     it 'performs the request' do
-      expect(client_double.mtermvectors(index: 'my-index', type: 'my-type', ids: [1, 2, 3])).to eq({})
+      expect(client_double.mtermvectors(index: 'my-index', ids: [1, 2, 3])).to eq({})
     end
   end
 end

data/spec/opensearch/api/actions/search_spec.rb

@@ -47,11 +47,11 @@ describe 'client#search' do
   end
 
   let(:url) do
-    '_all/foo/_search'
+    '_search'
   end
 
   it 'has a default value for index' do
-    expect(client_double.search(type: 'foo'))
+    expect(client_double.search())
   end
 
   context 'when a request definition is specified' do
@@ -80,14 +80,14 @@ describe 'client#search' do
     end
   end
 
-  context 'when an index and type are specified' do
+  context 'when an index is specified' do
 
     let(:url) do
-      'foo/bar/_search'
+      'foo/_search'
     end
 
     it 'performs the request' do
-      expect(client_double.search(index: 'foo', type: 'bar'))
+      expect(client_double.search(index: 'foo'))
     end
   end
 
@@ -102,17 +102,6 @@ describe 'client#search' do
     end
   end
 
-  context 'when multiple indices and types are specified' do
-
-    let(:url) do
-      'foo,bar/lam,bam/_search'
-    end
-
-    it 'performs the request' do
-      expect(client_double.search(index: ['foo', 'bar'], type: ['lam', 'bam']))
-    end
-  end
-
   context 'when there are URL params' do
 
     let(:url) do

data/spec/opensearch/api/actions/termvectors_spec.rb

@@ -38,7 +38,7 @@ describe 'client#termvectors' do
   end
 
   let(:url) do
-    'foo/bar/123/_termvectors'
+    'foo/_termvectors/123'
   end
 
   let(:params) do
@@ -55,21 +55,21 @@ describe 'client#termvectors' do
 
   it 'requires the :index argument' do
     expect {
-      client.termvectors(type: 'bar', id: '1')
+      client.termvectors(id: '1')
     }.to raise_exception(ArgumentError)
   end
 
   it 'performs the request' do
-    expect(client_double.termvectors(index: 'foo', type: 'bar', id: '123', body: {})).to eq({})
+    expect(client_double.termvectors(index: 'foo', id: '123', body: {})).to eq({})
   end
 
   context 'when the older api name \'termvector\' is used' do
     let(:url) do
-      'foo/bar/123/_termvector'
+      'foo/_termvector/123'
     end
 
     it 'performs the request' do
-      expect(client_double.termvector(index: 'foo', type: 'bar', id: '123', body: {})).to eq({})
+      expect(client_double.termvector(index: 'foo', id: '123', body: {})).to eq({})
     end
   end
 end

data/spec/opensearch/api/actions/update_document_spec.rb

@@ -43,7 +43,7 @@ describe 'client#update' do
   end
 
   let(:url) do
-    'foo/bar/1/_update'
+    'foo/_update/1'
   end
 
   let(:client) do
@@ -56,24 +56,24 @@ describe 'client#update' do
 
   it 'requires the :index argument' do
     expect {
-      client.update(type: 'bar', id: '1')
+      client.update(id: '1')
     }.to raise_exception(ArgumentError)
   end
 
   it 'requires the :id argument' do
     expect {
-      client.update(index: 'foo', type: 'bar')
+      client.update(index: 'foo')
     }.to raise_exception(ArgumentError)
   end
 
   it 'performs the request' do
-    expect(client_double.update(index: 'foo', type: 'bar', id: '1', body: { doc: {} })).to eq({})
+    expect(client_double.update(index: 'foo', id: '1', body: { doc: {} })).to eq({})
   end
 
   context 'when URL parameters are provided' do
 
     let(:url) do
-      'foo/bar/1/_update'
+      'foo/_update/1'
     end
 
     let(:body) do
@@ -81,7 +81,7 @@ describe 'client#update' do
     end
 
     it 'performs the request' do
-      expect(client_double.update(index: 'foo', type: 'bar', id: '1', body: {}))
+      expect(client_double.update(index: 'foo', id: '1', body: {}))
     end
   end
 
@@ -89,7 +89,7 @@ describe 'client#update' do
 
     it 'raises an ArgumentError' do
       expect {
-        client.update(index: 'foo', type: 'bar', id: '1', body: { doc: {} }, qwertypoiuy: 'asdflkjhg')
+        client.update(index: 'foo', id: '1', body: { doc: {} }, qwertypoiuy: 'asdflkjhg')
       }.to raise_exception(ArgumentError)
     end
   end
@@ -97,7 +97,7 @@ describe 'client#update' do
   context 'when the request needs to be URL-escaped' do
 
     let(:url) do
-      'foo%5Ebar/bar%2Fbam/1/_update'
+      'foo%5Ebar/_update/1'
     end
 
     let(:body) do
@@ -105,7 +105,7 @@ describe 'client#update' do
     end
 
     it 'escapes the parts' do
-      expect(client_double.update(index: 'foo^bar', type: 'bar/bam', id: '1', body: {}))
+      expect(client_double.update(index: 'foo^bar', id: '1', body: {}))
     end
   end
 
@@ -117,14 +117,14 @@ describe 'client#update' do
 
     it 'raises it to the user' do
       expect {
-        client.update(index: 'foo', type: 'bar', id: 'XXX', body: {})
+        client.update(index: 'foo', id: 'XXX', body: {})
       }.to raise_exception(NotFound)
     end
 
     context 'when the :ignore parameter is specified' do
 
       it 'does not raise the error to the user' do
-        expect(client.update(index: 'foo', type: 'bar', id: 'XXX', body: {}, ignore: 404)).to eq(false)
+        expect(client.update(index: 'foo', id: 'XXX', body: {}, ignore: 404)).to eq(false)
       end
     end
   end

data/spec/opensearch/api/utils_spec.rb

@@ -48,17 +48,10 @@ describe OpenSearch::API::Utils do
       expect(utils.__escape('*')).to eq('*')
     end
 
-    it 'users CGI.escape by default' do
+    it 'uses CGI.escape by default' do
       expect(CGI).to receive(:escape).and_call_original
       expect(utils.__escape('foo bar')).to eq('foo+bar')
     end
-
-    it 'uses the escape_utils gem when available', unless: defined?(JRUBY_VERSION) do
-      require 'escape_utils'
-      expect(CGI).not_to receive(:escape)
-      expect(EscapeUtils).to receive(:escape_url).and_call_original
-      expect(utils.__escape('foo bar')).to eq('foo+bar')
-    end
   end
 
   describe '#__listify' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: opensearch-api
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 2.0.2
 platform: ruby
 authors:
 - Jayesh Hathila
@@ -33,7 +33,7 @@ cert_chain:
   yXikuH6LEVykA8pgOcB9gKsB2/zMd2ZlSj2monM8Qw9EfB14ZSDTYS8VYuwWCeF0
   eFmXXk0ufQFKl1Yll7quHkmQ0PzKkvXTpONBT6qPkXE=
   -----END CERTIFICATE-----
-date: 2021-12-06 00:00:00.000000000 Z
+date: 2022-06-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json
@@ -245,20 +245,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: escape_utils
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: jbuilder
   requirement: !ruby/object:Gem::Requirement
@@ -431,10 +417,8 @@ files:
 - lib/opensearch/api/actions/indices/exists_alias.rb
 - lib/opensearch/api/actions/indices/exists_index_template.rb
 - lib/opensearch/api/actions/indices/exists_template.rb
-- lib/opensearch/api/actions/indices/exists_type.rb
 - lib/opensearch/api/actions/indices/field_usage_stats.rb
 - lib/opensearch/api/actions/indices/flush.rb
-- lib/opensearch/api/actions/indices/flush_synced.rb
 - lib/opensearch/api/actions/indices/forcemerge.rb
 - lib/opensearch/api/actions/indices/get.rb
 - lib/opensearch/api/actions/indices/get_alias.rb
@@ -605,10 +589,8 @@ files:
 - spec/opensearch/api/actions/indices/exists_alias_spec.rb
 - spec/opensearch/api/actions/indices/exists_spec.rb
 - spec/opensearch/api/actions/indices/exists_template_spec.rb
-- spec/opensearch/api/actions/indices/exists_type_spec.rb
 - spec/opensearch/api/actions/indices/field_usage_stats_spec.rb
 - spec/opensearch/api/actions/indices/flush_spec.rb
-- spec/opensearch/api/actions/indices/flush_synced_spec.rb
 - spec/opensearch/api/actions/indices/forcemerge_spec.rb
 - spec/opensearch/api/actions/indices/get_alias_spec.rb
 - spec/opensearch/api/actions/indices/get_field_mapping_spec.rb
@@ -778,10 +760,8 @@ test_files:
 - spec/opensearch/api/actions/indices/exists_alias_spec.rb
 - spec/opensearch/api/actions/indices/exists_spec.rb
 - spec/opensearch/api/actions/indices/exists_template_spec.rb
-- spec/opensearch/api/actions/indices/exists_type_spec.rb
 - spec/opensearch/api/actions/indices/field_usage_stats_spec.rb
 - spec/opensearch/api/actions/indices/flush_spec.rb
-- spec/opensearch/api/actions/indices/flush_synced_spec.rb
 - spec/opensearch/api/actions/indices/forcemerge_spec.rb
 - spec/opensearch/api/actions/indices/get_alias_spec.rb
 - spec/opensearch/api/actions/indices/get_field_mapping_spec.rb