Fix tests

Author: Kevin Kirsche

elasticsearch-api/test/unit/exists_document_test.rb

@@ -9,19 +9,19 @@ class ExistsTest < ::Test::Unit::TestCase
 
         should "require the :index argument" do
           assert_raise ArgumentError do
-            subject.exists :type => 'bar', :id => '1'
+            subject.exists? :type => 'bar', :id => '1'
           end
         end
 
         should "NOT require the :type argument" do
           assert_nothing_raised do
-            subject.exists :index => 'foo', :id => '1'
+            subject.exists? :index => 'foo', :id => '1'
           end
         end
 
         should "require the :id argument" do
           assert_raise ArgumentError do
-            subject.exists :index => 'foo', :type => 'bar'
+            subject.exists? :index => 'foo', :type => 'bar'
           end
         end
 
@@ -34,7 +34,7 @@ class ExistsTest < ::Test::Unit::TestCase
             true
           end.returns(FakeResponse.new)
 
-          subject.exists :index => 'foo', :type => 'bar', :id => '1'
+          subject.exists? :index => 'foo', :type => 'bar', :id => '1'
         end
 
         should "pass the URL parameters" do
@@ -44,7 +44,7 @@ class ExistsTest < ::Test::Unit::TestCase
             true
           end.returns(FakeResponse.new)
 
-          subject.exists :index => 'foo', :type => 'bar', :id => '1', :routing => 'abc123'
+          subject.exists? :index => 'foo', :type => 'bar', :id => '1', :routing => 'abc123'
         end
 
         should "URL-escape the parts" do
@@ -53,28 +53,28 @@ class ExistsTest < ::Test::Unit::TestCase
             true
           end.returns(FakeResponse.new)
 
-          subject.exists :index => 'foo', :type => 'bar/bam', :id => '1'
+          subject.exists? :index => 'foo', :type => 'bar/bam', :id => '1'
         end
 
         should "return true for successful response" do
           subject.expects(:perform_request).returns(FakeResponse.new 200, 'OK')
-          assert_equal true, subject.exists(:index => 'foo', :type => 'bar', :id => '1')
+          assert_equal true, subject.exists?(:index => 'foo', :type => 'bar', :id => '1')
         end
 
         should "return false for 404 response" do
           subject.expects(:perform_request).returns(FakeResponse.new 404, 'Not Found')
-          assert_equal false, subject.exists(:index => 'foo', :type => 'bar', :id => '1')
+          assert_equal false, subject.exists?(:index => 'foo', :type => 'bar', :id => '1')
         end
 
         should "return false on 'not found' exceptions" do
           subject.expects(:perform_request).raises(StandardError.new '404 NotFound')
-          assert_equal false, subject.exists(:index => 'foo', :type => 'bar', :id => '1')
+          assert_equal false, subject.exists?(:index => 'foo', :type => 'bar', :id => '1')
         end
 
         should "re-raise generic exceptions" do
           subject.expects(:perform_request).raises(StandardError)
           assert_raise(StandardError) do
-            assert_equal false, subject.exists(:index => 'foo', :type => 'bar', :id => '1')
+            assert_equal false, subject.exists?(:index => 'foo', :type => 'bar', :id => '1')
           end
         end
 

elasticsearch-api/test/unit/indices/exists_alias_test.rb

@@ -16,7 +16,7 @@ class IndicesExistsAliasTest < ::Test::Unit::TestCase
             true
           end.returns(FakeResponse.new)
 
-          subject.indices.exists_alias :name => 'foo'
+          subject.indices.exists_alias? :name => 'foo'
         end
 
         should "perform request against multiple indices" do
@@ -25,7 +25,7 @@ class IndicesExistsAliasTest < ::Test::Unit::TestCase
             true
           end.returns(FakeResponse.new)
 
-          subject.indices.exists_alias :index => ['foo','bar'], :name => 'bam'
+          subject.indices.exists_alias? :index => ['foo','bar'], :name => 'bam'
         end
 
         should "URL-escape the parts" do
@@ -34,30 +34,30 @@ class IndicesExistsAliasTest < ::Test::Unit::TestCase
             true
           end.returns(FakeResponse.new)
 
-          subject.indices.exists_alias :index => 'foo^bar', :name => 'bar/bam'
+          subject.indices.exists_alias? :index => 'foo^bar', :name => 'bar/bam'
         end
 
         should "return true for successful response" do
           subject.expects(:perform_request).returns(FakeResponse.new 200, 'OK')
-          assert_equal true, subject.indices.exists_alias(:name => 'foo')
+          assert_equal true, subject.indices.exists_alias?(:name => 'foo')
         end
 
         should "return false for 404 response" do
           subject.expects(:perform_request).returns(FakeResponse.new 404, 'Not Found')
-          assert_equal false, subject.indices.exists_alias(:name => 'none')
+          assert_equal false, subject.indices.exists_alias?(:name => 'none')
         end
 
         should "return false on 'not found' exceptions" do
           subject.expects(:perform_request).raises(StandardError.new '404 NotFound')
           assert_nothing_raised do
-            assert_equal false, subject.indices.exists_alias(:name => 'none')
+            assert_equal false, subject.indices.exists_alias?(:name => 'none')
           end
         end
 
         should "re-raise generic exceptions" do
           subject.expects(:perform_request).raises(StandardError)
           assert_raise(StandardError) do
-            assert_equal false, subject.indices.exists_alias(:name => 'none')
+            assert_equal false, subject.indices.exists_alias?(:name => 'none')
           end
         end
 

elasticsearch-api/test/unit/indices/exists_test.rb

@@ -16,7 +16,7 @@ class IndicesExistsTest < ::Test::Unit::TestCase
             true
           end.returns(FakeResponse.new)
 
-          subject.indices.exists(:index => 'foo')
+          subject.indices.exists?(:index => 'foo')
         end
 
         should "perform the request against multiple indices" do
@@ -25,7 +25,7 @@ class IndicesExistsTest < ::Test::Unit::TestCase
             true
           end.returns(FakeResponse.new)
 
-          subject.indices.exists(:index => ['foo', 'bar'])
+          subject.indices.exists?(:index => ['foo', 'bar'])
         end
 
         should "URL-escape the parts" do
@@ -34,28 +34,28 @@ class IndicesExistsTest < ::Test::Unit::TestCase
             true
           end.returns(FakeResponse.new)
 
-          subject.indices.exists :index => 'foo^bar,bar/bam'
+          subject.indices.exists? :index => 'foo^bar,bar/bam'
         end
 
         should "return true for successful response" do
           subject.expects(:perform_request).returns(FakeResponse.new 200, 'OK')
-          assert_equal true, subject.indices.exists(:index => 'foo')
+          assert_equal true, subject.indices.exists?(:index => 'foo')
         end
 
         should "return false for 404 response" do
           subject.expects(:perform_request).returns(FakeResponse.new 404, 'Not Found')
-          assert_equal false, subject.indices.exists(:index => 'none')
+          assert_equal false, subject.indices.exists?(:index => 'none')
         end
 
         should "return false on 'not found' exceptions" do
           subject.expects(:perform_request).raises(StandardError.new '404 NotFound')
-          assert_equal false, subject.indices.exists(:index => 'none')
+          assert_equal false, subject.indices.exists?(:index => 'none')
         end
 
         should "re-raise generic exceptions" do
           subject.expects(:perform_request).raises(StandardError)
           assert_raise(StandardError) do
-            assert_equal false, subject.indices.exists(:index => 'none')
+            assert_equal false, subject.indices.exists?(:index => 'none')
           end
         end
 

elasticsearch-api/test/unit/indices/exists_type_test.rb

@@ -16,7 +16,7 @@ class IndicesExistsTypeTest < ::Test::Unit::TestCase
             true
           end.returns(FakeResponse.new)
 
-          subject.indices.exists_type :index => 'foo', :type => 'bar'
+          subject.indices.exists_type? :index => 'foo', :type => 'bar'
         end
 
         should "perform request against multiple indices" do
@@ -25,7 +25,7 @@ class IndicesExistsTypeTest < ::Test::Unit::TestCase
             true
           end.returns(FakeResponse.new)
 
-          subject.indices.exists_type :index => ['foo','bar'], :type => 'bam'
+          subject.indices.exists_type? :index => ['foo','bar'], :type => 'bam'
         end
 
         should "URL-escape the parts" do
@@ -34,30 +34,30 @@ class IndicesExistsTypeTest < ::Test::Unit::TestCase
             true
           end.returns(FakeResponse.new)
 
-          subject.indices.exists_type :index => 'foo^bar', :type => 'bar/bam'
+          subject.indices.exists_type? :index => 'foo^bar', :type => 'bar/bam'
         end
 
         should "return true for successful response" do
           subject.expects(:perform_request).returns(FakeResponse.new 200, 'OK')
-          assert_equal true, subject.indices.exists_type(:index => 'foo', :type => 'bar')
+          assert_equal true, subject.indices.exists_type?(:index => 'foo', :type => 'bar')
         end
 
         should "return false for 404 response" do
           subject.expects(:perform_request).returns(FakeResponse.new 404, 'Not Found')
-          assert_equal false, subject.indices.exists_type(:index => 'foo', :type => 'none')
+          assert_equal false, subject.indices.exists_type?(:index => 'foo', :type => 'none')
         end
 
         should "return false on 'not found' exceptions" do
           subject.expects(:perform_request).raises(StandardError.new '404 NotFound')
           assert_nothing_raised do
-            assert_equal false, subject.indices.exists_type(:index => 'foo', :type => 'none')
+            assert_equal false, subject.indices.exists_type?(:index => 'foo', :type => 'none')
           end
         end
 
         should "re-raise generic exceptions" do
           subject.expects(:perform_request).raises(StandardError)
           assert_raise(StandardError) do
-            assert_equal false, subject.indices.exists_type(:index => 'foo', :type => 'none')
+            assert_equal false, subject.indices.exists_type?(:index => 'foo', :type => 'none')
           end
         end