delayed-method 0.2.0 → 0.2.1

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 98dd71101e47e443933ffca6f42c37d945c2f92e
+  data.tar.gz: c7c5d3f383502af0d2f0b07330ef6cc6076dbc11
+SHA512:
+  metadata.gz: 1eb232ea47d0324d3160e9eb12c5fabeeca457e35c343776054b47d1e5deffbe5de3a14a842e7a0e741ea8c88170140b802010654379726dfed4368b9334afef
+  data.tar.gz: 78d815fa211bd1c9a66ac4bff957f36b021a08f5863e5c5e81a77ee6b12040a8401e3440680d81f9cfb5785e4130b7bd15b7e21d85d68367c97cd13a45f8337d

data/lib/delayed-method.rb

@@ -37,7 +37,7 @@ class DelayedMethod
       if object.is_a? Class
         yield object.name, nil
       elsif object.is_a? ActiveRecord::Base
-        raise ArgumentError.new("object need to be persisted") unless object.persisted?
+        raise ArgumentError.new("object #{object.class}:#{object.id}##{method} needs to be persisted #{caller}") unless object.persisted?
         yield object.class.name, object.id
       else
         raise ArgumentError.new("Only class and ActiveRecord resource are supported")

data/spec/delayed_method_spec.rb

@@ -28,10 +28,11 @@ describe DelayedMethod do
       end
 
       def mock_armodel
-        mock(ActiveRecord::Base, { :id => 1, :execute => true, :persisted? => true }).tap do |armodel|
+        mock(ActiveRecord::Base, { :id => 1, :execute => true }).tap do |armodel|
           armodel.should_receive(:is_a?).with(Class).and_return(false)
           armodel.should_receive(:is_a?).with(ActiveRecord::Base).and_return(true)
           armodel.should_receive(:class).and_return ActiveRecord::Base
+          armodel.stub(:persisted?).and_return true
           ActiveRecord::Base.stub(:find).with(1).and_return armodel
         end
       end
@@ -43,6 +44,17 @@ describe DelayedMethod do
           klass, args = Resque.reserve(:delayed)
           klass.perform(*args)
         end
+
+        context "object is not persisted" do
+          it "raises exception with identifying info" do
+            armodel = mock_armodel
+            armodel.stub(:some_method)
+            armodel.stub(:persisted?).and_return(false)
+            expect {
+              DelayedMethod.enqueue(armodel, :some_method)
+            }.to raise_error(anything, /Base.*some_method/)
+          end
+        end
       end
     end
   end

data/spec/spec_helper.rb

@@ -3,7 +3,3 @@ require 'bundler'
 require 'rspec'
 require 'delayed-method'
 require 'debugger'
-
-Dir[File.expand_path("../support/*.rb", __FILE__)].each do |file|
-  require file
-end

metadata

@@ -1,94 +1,83 @@
 --- !ruby/object:Gem::Specification
 name: delayed-method
 version: !ruby/object:Gem::Version
-  version: 0.2.0
-  prerelease: 
+  version: 0.2.1
 platform: ruby
 authors:
 - Phuong Nguyen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-09-19 00:00:00.000000000 Z
+date: 2016-01-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: resque
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.0'
 - !ruby/object:Gem::Dependency
   name: debugger
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: Allow you to quickly move a long call to background which then executed
@@ -98,36 +87,34 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- LICENSE
 - README.md
 - Rakefile
-- LICENSE
 - lib/delayed-method.rb
 - spec/delayed_method_spec.rb
 - spec/spec_helper.rb
-- spec/support/redis.rb
 homepage: http://github.com/phuongnd08/delayed-method
 licenses: []
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.23
+rubygems_version: 2.2.2
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Allow you to quickly move a long call to background which then executed by
   resque
 test_files: []

data/spec/support/redis.rb

@@ -1,70 +0,0 @@
-TMP_PATH = File.expand_path("../../../tmp", __FILE__)
-PIDS_PATH = TMP_PATH + "/pids"
-REDIS_PID = "#{PIDS_PATH}/redis-test.pid"
-REDIS_CACHE_PATH = "#{TMP_PATH}/cache/"
-
-module RedisHelper
-  class << self
-    def start_server
-      FileUtils.mkdir_p REDIS_CACHE_PATH
-      FileUtils.mkdir_p PIDS_PATH
-
-      redis_options = {
-        "daemonize"     => 'yes',
-        "pidfile"       => REDIS_PID,
-        "port"          => 9736,
-        "timeout"       => 300,
-        "save 900"      => 1,
-        "save 300"      => 1,
-        "save 60"       => 10000,
-        "dbfilename"    => "dump.rdb",
-        "dir"           => REDIS_CACHE_PATH,
-        "loglevel"      => "debug",
-        "logfile"       => "stdout",
-        "databases"     => 16
-      }.map { |k, v| "#{k} #{v}" }.join('\n')
-      `echo '#{redis_options}' | redis-server -`
-      success = false
-
-      wait_time_remaining = 5
-      begin
-        TCPSocket.open("localhost", "9736")
-        success = true
-      rescue Exception => e
-        if wait_time_remaining > 0
-          wait_time_remaining -= 0.1
-          sleep 0.1
-        else
-          raise "Cannot start redis server in 5 seconds. Pinging server yield\n#{e.inspect}"
-        end
-      end while(!success)
-
-      Resque.redis = "localhost:9736"
-    end
-
-    def stop_server
-      %x{
-      cat #{REDIS_PID} | xargs kill -QUIT
-      rm -f #{REDIS_CACHE_PATH}dump.rdb
-      }
-    end
-
-    def clear
-      Resque.redis.flushdb
-    end
-  end
-end
-
-RSpec.configure do |config|
-  config.before(:each) do
-    RedisHelper.clear
-  end
-
-  config.before(:suite) do
-    RedisHelper.start_server
-  end
-
-  config.after(:suite) do
-    RedisHelper.stop_server
-  end
-end