seam 0.0.16 → 0.0.17

data/lib/seam/persistence.rb

@@ -4,18 +4,12 @@ module Seam
       effort = Seam::InMemory.records.select { |x| x.id == effort_id }.first
       return nil unless effort
       effort.clone
-      #document = Seam::MongoDb.collection.find( { id: effort_id } ).first
-      #return nil unless document
-      #Seam::Effort.parse document
     end
 
     def self.find_all_pending_executions_by_step step
       Seam::InMemory.records
         .select { |x| x.next_step == step && x.next_execute_at <= Time.now }
         .map { |x| x.clone }
-      #Seam::MongoDb.collection
-        #.find( { next_step: step, next_execute_at: { '$lte' => Time.now } } )
-        #.map { |x| Seam::Effort.parse x }
     end
 
     def self.save effort
@@ -24,13 +18,10 @@ module Seam
         Seam::InMemory.records = Seam::InMemory.records.select { |x| x.id != effort.id }.to_a
       end
       create effort
-      #Seam::MongoDb.collection.find( { id: effort.id } )
-      #    .update("$set" => effort.to_hash)
     end
 
     def self.create effort
       Seam::InMemory.records = [Seam::InMemory.records, effort].flatten
-      #Seam::MongoDb.collection.insert(effort.to_hash)
     end
 
     def self.all

data/lib/seam/version.rb

@@ -1,3 +1,3 @@
 module Seam
-  VERSION = "0.0.16"
+  VERSION = "0.0.17"
 end

data/lib/seam/wait_worker.rb

@@ -13,13 +13,18 @@ module Seam
     private
 
     def the_time_to_move_on
-      Time.now + the_amount_of_time_to_wait
+      time_of_last_execution + the_amount_of_time_to_wait
     end
 
     def the_amount_of_time_to_wait
       current_step[:arguments][0]
     end
 
+    def time_of_last_execution
+      return effort.created_at if effort.history.count == 0
+      effort.history.last[:stopped_at]
+    end
+
   end
 
 end

data/spec/seam/wait_worker_spec.rb

@@ -15,16 +15,16 @@ describe Seam::WaitWorker do
     Seam::WaitWorker.new.step.must_equal 'wait'
   end
 
-  describe "using it in a flow" do
+  let(:today) do
+    Time.parse '1/1/2011'
+  end
 
-    let(:effort_id) do
-      effort = flow.start
-      effort.id
-    end
+  let(:effort_id) do
+    effort = flow.start
+    effort.id
+  end
 
-    let(:today) do
-      Time.parse '1/1/2011'
-    end
+  describe "using it in a flow" do
 
     [:length_of_time].to_objects { [
       [3.days],
@@ -59,6 +59,120 @@ describe Seam::WaitWorker do
       end
 
     end
+
+    describe "time has passed since the flow was started and the wait worker was called" do
+
+      let(:today)                              { Time.parse('1/1/2011') }
+      let(:time_to_wait)                       { 3.days }
+      let(:time_before_wait_worker_was_called) { 1.day }
+      let(:expected_start)                     { today + time_to_wait }
+
+      let(:flow) do
+        f = Seam::Flow.new
+        f.wait time_to_wait
+        f.do_something
+        f
+      end
+
+      before do
+        Timecop.freeze today
+        effort_id
+
+        Timecop.freeze today + time_before_wait_worker_was_called
+      end
+
+      it "should move to the next step" do
+        Seam::WaitWorker.new.execute_all
+        Seam::Effort.find(effort_id).next_step.must_equal "do_something"
+      end
+       
+      it "should set the next execute date" do
+        Seam::WaitWorker.new.execute_all
+        Seam::Effort.find(effort_id).next_execute_at.must_equal expected_start
+      end
+
+    end
+
+    describe "time has passed since the last step was started and the wait worker was called" do
+
+      let(:today)                              { Time.parse('1/1/2011') }
+      let(:time_to_wait)                       { 8.days }
+      let(:time_it_took_for_previous_step)     { 1.days }
+      let(:time_before_wait_worker_was_called) { 3.day }
+      let(:expected_start)                     { today + time_to_wait + time_it_took_for_previous_step }
+
+      let(:flow) do
+        f = Seam::Flow.new
+        f.do_something
+        f.wait time_to_wait
+        f.do_something
+        f
+      end
+
+      before do
+        Timecop.freeze today
+        effort_id
+
+        Timecop.freeze Time.now + time_it_took_for_previous_step
+        DoSomething.new.execute_all
+        Timecop.freeze Time.now + time_before_wait_worker_was_called
+      end
+
+      it "should move to the next step" do
+        Seam::WaitWorker.new.execute_all
+        Seam::Effort.find(effort_id).next_step.must_equal "do_something"
+      end
+       
+      it "should set the next execute date" do
+        Seam::WaitWorker.new.execute_all
+        Seam::Effort.find(effort_id).next_execute_at.must_equal expected_start
+      end
+
+    end
+    j
+    describe "a more complex example of waiting" do
+
+      let(:today)                              { Time.parse('1/1/2011') }
+      let(:time_to_wait)                       { 8.days }
+      let(:time_it_took_for_first_step)        { 1.days }
+      let(:time_it_took_for_second_step)       { 4.days }
+      let(:time_before_wait_worker_was_called) { 3.day }
+      let(:expected_start)                     { today + 
+                                                 time_to_wait + 
+                                                 time_it_took_for_first_step +
+                                                 time_it_took_for_second_step  }
+
+      let(:flow) do
+        f = Seam::Flow.new
+        f.do_something
+        f.do_something
+        f.wait time_to_wait
+        f.do_something
+        f
+      end
+
+      before do
+        Timecop.freeze today
+        effort_id
+
+        Timecop.freeze Time.now + time_it_took_for_first_step
+        DoSomething.new.execute_all
+        Timecop.freeze Time.now + time_it_took_for_second_step
+        DoSomething.new.execute_all
+        Timecop.freeze Time.now + time_before_wait_worker_was_called
+      end
+
+      it "should move to the next step" do
+        Seam::WaitWorker.new.execute_all
+        Seam::Effort.find(effort_id).next_step.must_equal "do_something"
+      end
+       
+      it "should set the next execute date" do
+        Seam::WaitWorker.new.execute_all
+        Seam::Effort.find(effort_id).next_execute_at.must_equal expected_start
+      end
+
+    end
     
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: seam
 version: !ruby/object:Gem::Version
-  version: 0.0.16
+  version: 0.0.17
   prerelease: 
 platform: ruby
 authors:
@@ -230,7 +230,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -4363898252696775104
+      hash: -3036186920062924532
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -239,10 +239,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -4363898252696775104
+      hash: -3036186920062924532
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.25
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: Simple workflows