activesearch 0.1.2 → 0.1.3

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NDUxNjIzNjY2NmQxYjIzYmQ4NzA0NjRmYThjNThmY2ZhNjk4N2VlNw==
+    NWZiODUwMjI1M2M2OTQwZTBjOWI4ZWIzZTRiODFhYmEyMzg5MjAwNw==
   data.tar.gz: !binary |-
-    ZjZmZDhkODU0NzI4MDU0ZjU5YWUzODcyMGFlZDhlNGMyNDk1NjRhMw==
+    NmRmNTIwNjA1M2Y4NTJhZGQ2NzYyMTAwOTdlZmQyM2YyYzA4YmRkOA==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    MTA2OTEwMzMxYWE2ZTI4MTg3MzcyNTE2MmE4ZGU1ZDc4YjZjMmU2MjBjOTJm
-    ODc2MTNiNmUzZjhkM2E2ZTNlOGQ0OGYzOWJhYTdiNTUxMTYzNzA1N2JmZjcz
-    YTc3YTM5Nzc1OWYxYzU1YzZmZGUyYTliZmE1NDllY2YxNDYzZGY=
+    M2M1ZGNmMTVjYjA1YmM1YTUxNDk5ZWIxNzI0MTZjOWE4ZjE5ZTJmZmEwOWYx
+    OTNkZDFlZDM2NDA0NWYzMGE5ZjYxMGI3OGNmMzExMGE5YTkwNmM5Y2JjNWFh
+    OTY5ZDZiNzA2YWYzYWQ2M2I3MDc5NTBiZmVlZjY1MzY5NGIyNzI=
   data.tar.gz: !binary |-
-    MzM2NGFkNWZhZmZmMjY1MDNmZjJmMzI4MzBhNTI0OGJlMWRhOTg0YmJlNDRh
-    MjI3OWFiYzIxYjU2NTM4OWRlMTFlZGFmYjkyNzYwYzAzMjdkM2FhN2U5Zjg4
-    NjFlYTBmZTFjNjJkMDRiYTA4MDFjYTkxOTI3YWQzZWVmZjQ5MjU=
+    NzdhYjBmNDNkYzFhMjdmNDhlMzczOWViOWEzMzFjMTAyMDVhNjY2MjMzMjI0
+    ZDA2MmY0YmNjY2E5MzEzYzZlNWQxMDc4OWNiMjc0OGQ3N2RlNDYxMWFkNDQy
+    Yzg3Yjg4NzQzMjBlOTgzOTE0ZjQ5ZmUyYjFiM2E5Zjg2MWUxOWI=

data/Gemfile

@@ -1,4 +1,4 @@
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in activesearch.gemspec
-gemspec
+gemspec

data/activesearch.gemspec

@@ -18,6 +18,7 @@ Gem::Specification.new do |gem|
   gem.require_paths = ["lib"]
   
   gem.add_dependency "activesupport"
+  gem.add_dependency "sucker_punch"
   
   gem.add_development_dependency "rspec"
   gem.add_development_dependency "rspec-mocks"

data/lib/activesearch/algolia.rb

@@ -1,4 +1,5 @@
 require "activesearch/algolia/client"
+require "activesearch/algolia/worker"
 require "activesearch/base"
 require "activesearch/proxy"
 
@@ -26,7 +27,7 @@ module ActiveSearch
     conditions.map { |c| c.join(':') }.join(',')
   end
   
-  module Algolia
+  module Algolia      
     def self.included(base)
       base.class_eval do
         include Base
@@ -35,14 +36,11 @@ module ActiveSearch
     
     protected
     def reindex
-      algolia_client.save(indexable_id, self.to_indexable)
-    rescue
-      self.touch
-      false
+      Worker.new.async.perform(task: :reindex, id: indexable_id, doc: self.to_indexable)
     end
     
     def deindex
-      algolia_client.delete(indexable_id)
+      Worker.new.async.perform(task: :deindex, id: indexable_id)
     end
     
     def to_indexable
@@ -57,9 +55,5 @@ module ActiveSearch
       end
       doc
     end
-    
-    def algolia_client
-      @algolia_client ||= Client.new
-    end
   end
 end

data/lib/activesearch/algolia/worker.rb

@@ -0,0 +1,18 @@
+require "sucker_punch"
+
+class ActiveSearch::Algolia::Worker
+  include SuckerPunch::Job
+  
+  def perform(msg)
+    begin
+      case msg[:task]
+      when :reindex
+        ::ActiveSearch::Algolia::Client.new.save(msg[:id], msg[:doc])
+      when :deindex
+        ::ActiveSearch::Algolia::Client.new.delete(msg[:id])
+      end
+    rescue Exception => e
+      perform(msg.merge!(retries: msg[:retries].to_i + 1)) unless msg[:retries].to_i >= 3
+    end
+  end
+end

data/lib/activesearch/version.rb

@@ -1,3 +1,3 @@
 module ActiveSearch
-  VERSION = "0.1.2"
+  VERSION = "0.1.3"
 end

data/spec/algolia_spec.rb

@@ -12,11 +12,14 @@ describe ActiveSearch::Algolia do
     ActiveSearch::Algolia::Client.new.delete_index
   end
   
-  context "errors on save" do
+  context "retry on errors" do
     before do
+      times_called = 0
       @instance = AlgoliaModel.new(title: "Example")
-      @instance.should_receive(:touch).once
-      ActiveSearch::Algolia::Client.any_instance.stub(:save).and_raise(Errno::ECONNRESET)
+      ActiveSearch::Algolia::Client.should_receive(:put).exactly(3).times.and_return do
+        times_called += 1
+        raise Errno::ECONNRESET if times_called <= 2
+      end
     end
     
     subject { -> { @instance.save } }

data/spec/spec_helper.rb

@@ -1,5 +1,7 @@
 require 'active_model'
 require 'active_attr'
+require 'sucker_punch'
+require 'sucker_punch/testing/inline'
 
 class ActiveMimic
   extend ActiveModel::Callbacks

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activesearch
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Rodrigo Alvarez
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-10 00:00:00.000000000 Z
+date: 2013-10-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -24,6 +24,20 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: sucker_punch
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -154,6 +168,7 @@ files:
 - lib/activesearch.rb
 - lib/activesearch/algolia.rb
 - lib/activesearch/algolia/client.rb
+- lib/activesearch/algolia/worker.rb
 - lib/activesearch/base.rb
 - lib/activesearch/elastic_search.rb
 - lib/activesearch/mongoid.rb