delayed_cron_job 0.7.0 → 0.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1a6855db526a23fa48933504b568fded11df2722
-  data.tar.gz: c28537792117c9937fb5deb5c18f287ebf90596f
+  metadata.gz: ef66952c36796b2dfa66c343a31631d061d09bd1
+  data.tar.gz: fba9ad110c34d1e475bc8057d818b9d0c0fa02d4
 SHA512:
-  metadata.gz: 5abd5f9a6d32b08f7f92c274d355daaddebf32d9e93ec9197982c044c6a9928fc15a79a3412fa6c6589f98356d0183c768d20c6b8ad3cbbabd56bf256c2fa35c
-  data.tar.gz: 8abb91541f341a7e620ba2c602370e384b3093a0e6c4a7fd87ab0d6510c4610dc5be7dee8cdb562140320074805f3f1a0ff7c6d05969ca1ffe4c305087e60f09
+  metadata.gz: da230b6a7fef72f96129ff81f32f8e1345b025a6f0b54cc6bcc482d44a11e823480a2f3a0715c46687a98226d8f9c7e210ccf3f6892b08723ce6f78fd884e498
+  data.tar.gz: 150cd81fe161e5cd030a99924345902e47788877c861db22975ad6db80aa9eeb593e09abbd11a67c35054eb13603eddcb8313b85f4698368e78e879e0e1229ec

data/lib/delayed_cron_job.rb

@@ -3,6 +3,7 @@ require 'English'
 require 'delayed_cron_job/cronline'
 require 'delayed_cron_job/plugin'
 require 'delayed_cron_job/version'
+require 'delayed_cron_job/backend/updatable_cron'
 
 module DelayedCronJob
 
@@ -11,10 +12,14 @@ end
 if defined?(Delayed::Backend::Mongoid)
   Delayed::Backend::Mongoid::Job.field :cron, :type => String
   Delayed::Backend::Mongoid::Job.attr_accessible(:cron) if Delayed::Backend::Mongoid::Job.respond_to?(:attr_accessible)
+  Delayed::Backend::Mongoid::Job.send(:include, DelayedCronJob::Backend::UpdatableCron)
 end
 
-if defined?(Delayed::Backend::ActiveRecord) && Delayed::Backend::ActiveRecord::Job.respond_to?(:attr_accessible)
-  Delayed::Backend::ActiveRecord::Job.attr_accessible(:cron)
+if defined?(Delayed::Backend::ActiveRecord)
+  Delayed::Backend::ActiveRecord::Job.send(:include, DelayedCronJob::Backend::UpdatableCron)
+  if Delayed::Backend::ActiveRecord::Job.respond_to?(:attr_accessible)
+    Delayed::Backend::ActiveRecord::Job.attr_accessible(:cron)
+  end
 end
 
 Delayed::Worker.plugins << DelayedCronJob::Plugin

data/lib/delayed_cron_job/backend/updatable_cron.rb

@@ -0,0 +1,17 @@
+module DelayedCronJob
+  module Backend
+    module UpdatableCron
+
+      def self.included(klass)
+        klass.send(:before_save, :set_next_run_at, :if => :cron_changed?)
+      end
+
+      def set_next_run_at
+        if cron.present?
+          self.run_at = Cronline.new(cron).next_time(Delayed::Job.db_time_now)
+        end
+      end
+
+    end
+  end
+end

data/lib/delayed_cron_job/plugin.rb

@@ -2,20 +2,12 @@ module DelayedCronJob
   class Plugin < Delayed::Plugin
 
     class << self
-      def next_run_at(job)
-        job.run_at = Cronline.new(job.cron).next_time(Delayed::Job.db_time_now)
-      end
-
       def cron?(job)
         job.cron.present?
       end
     end
 
     callbacks do |lifecycle|
-      # Calculate the next run_at based on the cron attribute before enqueue.
-      lifecycle.before(:enqueue) do |job|
-        next_run_at(job) if cron?(job)
-      end
 
       # Prevent rescheduling of failed jobs as this is already done
       # after perform.
@@ -39,6 +31,13 @@ module DelayedCronJob
         end
       end
 
+      # Update the cron expression from the database in case it was updated.
+      lifecycle.after(:invoke_job) do |job|
+        if cron?(job)
+          job.cron = job.class.where(:id => job.id).pluck(:cron).first
+        end
+      end
+
       # Schedule the next run based on the cron attribute.
       lifecycle.after(:perform) do |worker, job|
         if cron?(job)
@@ -48,7 +47,6 @@ module DelayedCronJob
           next_job.locked_at = nil
           next_job.locked_by = nil
           next_job.attempts += 1
-          next_run_at(next_job)
           next_job.save!
         end
       end

data/lib/delayed_cron_job/version.rb

@@ -1,3 +1,3 @@
 module DelayedCronJob
-  VERSION = '0.7.0'
+  VERSION = '0.7.1'
 end

data/spec/delayed_cron_job_spec.rb

@@ -31,7 +31,7 @@ describe DelayedCronJob do
 
     it 'schedules a new job after success' do
       job.update_column(:run_at, now)
-      job.reload
+      job.reload # adjusts granularity of run_at datetime
 
       worker.work_off
 
@@ -48,7 +48,7 @@ describe DelayedCronJob do
     it 'schedules a new job after failure' do
       allow_any_instance_of(TestJob).to receive(:perform).and_raise('Fail!')
       job.update(run_at: now)
-      job.reload
+      job.reload # adjusts granularity of run_at datetime
 
       worker.work_off
 
@@ -98,7 +98,7 @@ describe DelayedCronJob do
       expect(j.last_error).to eq(nil)
     end
 
-    it 'has correct last_error after success' do
+    it 'has updated last_error after failure' do
       allow_any_instance_of(TestJob).to receive(:perform).and_raise('Fail!')
       job.update(run_at: now, last_error: 'Last error')
 
@@ -138,6 +138,36 @@ describe DelayedCronJob do
       j = Delayed::Job.first
       expect(j.attempts).to eq(job.attempts + 1)
     end
+
+    it 'updates run_at if cron is changed' do
+      job.update!(cron: '1 10 * * *')
+      expect(job.run_at.min).to eq(1)
+    end
+
+    it 'uses new cron when this is updated while job is running' do
+      job.update_column(:run_at, now)
+      allow_any_instance_of(TestJob).to receive(:perform) { job.update!(cron: '1 10 * * *') }
+
+      worker.work_off
+
+      j = Delayed::Job.first
+      expect(j.run_at.min).to eq(1)
+    end
+
+    it 'does not reschedule job if cron is cleared while job is running' do
+      job.update_column(:run_at, now)
+      allow_any_instance_of(TestJob).to receive(:perform) { job.update!(cron: '') }
+
+      expect { worker.work_off }.to change { Delayed::Job.count }.by(-1)
+    end
+
+    it 'does not reschedule job if model is deleted while job is running' do
+      job.update_column(:run_at, now)
+      allow_any_instance_of(TestJob).to receive(:perform) { job.destroy! }
+
+      expect { worker.work_off }.to change { Delayed::Job.count }.by(-1)
+    end
+
   end
 
   context 'without cron' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: delayed_cron_job
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.7.1
 platform: ruby
 authors:
 - Pascal Zumkehr
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-04-29 00:00:00.000000000 Z
+date: 2016-05-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: delayed_job
@@ -129,6 +129,7 @@ files:
 - lib/delayed_cron_job.rb
 - lib/delayed_cron_job/active_job/enqueuing.rb
 - lib/delayed_cron_job/active_job/queue_adapter.rb
+- lib/delayed_cron_job/backend/updatable_cron.rb
 - lib/delayed_cron_job/cronline.rb
 - lib/delayed_cron_job/plugin.rb
 - lib/delayed_cron_job/version.rb
@@ -157,7 +158,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5.1
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
 summary: An extension to Delayed::Job that allows you to set cron expressions for