Merge pull request #342 from austinarbor/allow-first-at-in-past

Optionally allow first_at to be set in the past

lib/rufus/scheduler/jobs_repeat.rb

@@ -15,6 +15,8 @@ def initialize(scheduler, duration, opts, block)
 
     @times = opts[:times]
 
+    @first_at_no_error = opts[:first_at_no_error] || false
+
     fail ArgumentError.new(
       "cannot accept :times => #{@times.inspect}, not nil or an int"
     ) unless @times == nil || @times.is_a?(Integer)
@@ -43,6 +45,7 @@ def first_at=(first)
 
     @first_at = (fdur && (EoTime.now + fdur)) || EoTime.make(first)
     @first_at = n1 if @first_at >= n0 && @first_at < n1
+    @first_at = n0 if @first_at < n0 && @first_at_no_error
 
     fail ArgumentError.new(
       "cannot set first[_at|_in] in the past: " +
@@ -319,4 +322,3 @@ def set_next_time(trigger_time, is_post=false, now=nil)
       end
   end
 end
-

spec/job_repeat_spec.rb

@@ -191,6 +191,15 @@
       }.to raise_error(ArgumentError)
     end
 
+    it 'does not raise on points in the past when first_at_no_error is set' do
+      n = Time.now
+      job =
+          @scheduler.schedule_every '7s',
+                                    { :first => Time.now - 60, :first_at_no_error => true } do; end
+
+      expect(job.first_at).to be < n + 0.7
+    end
+
     context ':first_time => :now/:immediately/0' do
 
       it 'schedules the first execution immediately (:first => :now)' do
@@ -439,4 +448,3 @@
     end
   end
 end
-