elastic_searchable 0.3.0 → 0.3.1

data/Gemfile

@@ -1,7 +1,7 @@
 source "http://rubygems.org"
 # Add dependencies required to use your gem here.
 gem "activerecord", "~> 2.3.5"
-gem 'httparty', '~> 0.7.3'
+gem 'httparty', '~> 0.6.0'
 gem 'backgrounded', '~> 0.7.0'
 gem 'will_paginate', '~> 2.3.15'
 gem 'larsklevan-after_commit', '~> 1.0.5', :require => 'after_commit'

data/VERSION

@@ -1 +1 @@
-0.3.0
+0.3.1

data/elastic_searchable.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{elastic_searchable}
-  s.version = "0.3.0"
+  s.version = "0.3.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Ryan Sonnek"]
@@ -48,7 +48,7 @@ Gem::Specification.new do |s|
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
       s.add_runtime_dependency(%q<activerecord>, ["~> 2.3.5"])
-      s.add_runtime_dependency(%q<httparty>, ["~> 0.7.3"])
+      s.add_runtime_dependency(%q<httparty>, ["~> 0.6.0"])
       s.add_runtime_dependency(%q<backgrounded>, ["~> 0.7.0"])
       s.add_runtime_dependency(%q<will_paginate>, ["~> 2.3.15"])
       s.add_runtime_dependency(%q<larsklevan-after_commit>, ["~> 1.0.5"])
@@ -60,7 +60,7 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<sqlite3-ruby>, ["~> 1.3.2"])
     else
       s.add_dependency(%q<activerecord>, ["~> 2.3.5"])
-      s.add_dependency(%q<httparty>, ["~> 0.7.3"])
+      s.add_dependency(%q<httparty>, ["~> 0.6.0"])
       s.add_dependency(%q<backgrounded>, ["~> 0.7.0"])
       s.add_dependency(%q<will_paginate>, ["~> 2.3.15"])
       s.add_dependency(%q<larsklevan-after_commit>, ["~> 1.0.5"])
@@ -73,7 +73,7 @@ Gem::Specification.new do |s|
     end
   else
     s.add_dependency(%q<activerecord>, ["~> 2.3.5"])
-    s.add_dependency(%q<httparty>, ["~> 0.7.3"])
+    s.add_dependency(%q<httparty>, ["~> 0.6.0"])
     s.add_dependency(%q<backgrounded>, ["~> 0.7.0"])
     s.add_dependency(%q<will_paginate>, ["~> 2.3.15"])
     s.add_dependency(%q<larsklevan-after_commit>, ["~> 1.0.5"])

data/lib/elastic_searchable.rb

@@ -22,7 +22,7 @@ module ElasticSearchable
     #perform a request to the elasticsearch server
     def request(method, url, options = {})
       response = self.send(method, url, options)
-      puts "elasticsearch request: #{method} #{url} #{" finished in #{response['took']}ms" if response['took']}"
+      #puts "elasticsearch request: #{method} #{url} #{" finished in #{response['took']}ms" if response['took']}"
       assert_ok_response response
       response
     end
@@ -30,7 +30,7 @@ module ElasticSearchable
     private
     def assert_ok_response(response)
       error = response['error'] || "Error executing request: #{response.inspect}"
-      raise ElasticSearchable::ElasticError.new(error) if response['error'] || !response.success?
+      raise ElasticSearchable::ElasticError.new(error) if response['error'] || !response.response.is_a?(Net::HTTPOK)
     end
   end
 end

data/test/test_elastic_searchable.rb

@@ -67,6 +67,14 @@ class TestElasticSearchable < Test::Unit::TestCase
     end
   end
 
+  context 'requesting invalid url' do
+    should 'raise error' do
+      assert_raises ElasticSearchable::ElasticError do
+        ElasticSearchable.request :get, '/elastic_searchable/foobar/notfound'
+      end
+    end
+  end
+
   context 'with empty index' do
     setup do
       begin
@@ -154,7 +162,7 @@ class TestElasticSearchable < Test::Unit::TestCase
       end
       should 'be removed from the index' do
         @request = ElasticSearchable.get "/elastic_searchable/posts/#{@first_post.id}"
-        assert @request.not_found?, @request.inspect
+        assert @request.response.is_a?(Net::HTTPNotFound), @request.inspect
       end
     end
   end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: elastic_searchable
 version: !ruby/object:Gem::Version 
-  hash: 19
+  hash: 17
   prerelease: 
   segments: 
   - 0
   - 3
-  - 0
-  version: 0.3.0
+  - 1
+  version: 0.3.1
 platform: ruby
 authors: 
 - Ryan Sonnek
@@ -41,12 +41,12 @@ dependencies:
     requirements: 
     - - ~>
       - !ruby/object:Gem::Version 
-        hash: 5
+        hash: 7
         segments: 
         - 0
-        - 7
-        - 3
-        version: 0.7.3
+        - 6
+        - 0
+        version: 0.6.0
   prerelease: false
   type: :runtime
   requirement: *id002