seam 0.0.14 → 0.0.15

data/lib/seam/flow.rb

@@ -1,8 +1,11 @@
 module Seam
   class Flow
 
+    attr_accessor :stamp_data_history
+
     def initialize
       @steps = []
+      @stamp_data_history = false
     end
 
     def method_missing(meth, *args, &blk)
@@ -23,7 +26,10 @@ module Seam
     end
 
     def to_hash
-      { steps: self.steps.map { |x| x.to_hash } }
+      { 
+        steps: self.steps.map { |x| x.to_hash },
+        stamp_data_history: @stamp_data_history
+      }
     end
 
     def steps

data/lib/seam/version.rb

@@ -1,3 +1,3 @@
 module Seam
-  VERSION = "0.0.14"
+  VERSION = "0.0.15"
 end

data/lib/seam/worker.rb

@@ -98,7 +98,7 @@ module Seam
       run = { 
               started_at:  Time.now,
               step:        step, 
-              data_before: effort.data.clone
+              data_before: stamping_the_history? ? effort.data.clone : nil
             }
       @current_run = HashWithIndifferentAccess.new run
     end
@@ -120,12 +120,19 @@ module Seam
 
     def stamp_the_new_history_record
       history[:result]     = @operation_to_execute
-      history[:data_after] = effort.data.clone
       history[:stopped_at] = Time.now
 
+      if stamping_the_history?
+        history[:data_after] = effort.data.clone
+      end
+
       effort.history << history
     end
 
+    def stamping_the_history?
+      effort.flow['stamp_data_history']
+    end
+
     def save_the_effort
       effort.save
     end

data/spec/seam/flow_spec.rb

@@ -129,4 +129,36 @@ describe "flow" do
       end
     end
   end
+
+  describe "stamping history" do
+    describe "default" do
+      it "should should be false" do
+        flow = Seam::Flow.new
+        flow.stamp_data_history.must_equal false
+      end
+    end
+
+    describe "setting it to true" do
+      it "allow to be set to true" do
+        flow = Seam::Flow.new
+        flow.stamp_data_history = true
+        flow.stamp_data_history.must_equal true
+      end
+    end
+
+    describe "carrying the value through the serialization" do
+
+      it "should be able to persist false" do
+        flow = Seam::Flow.new
+        flow.stamp_data_history = false
+        flow.to_hash[:stamp_data_history].must_equal false
+      end
+
+      it "should be able to persist true" do
+        flow = Seam::Flow.new
+        flow.stamp_data_history = true
+        flow.to_hash[:stamp_data_history].must_equal true
+      end
+    end
+  end
 end

data/spec/seam/worker_spec.rb

@@ -4,6 +4,7 @@ describe "worker" do
 
   before do
     Seam::Persistence.destroy
+    @stamp_data_history = true
   end
 
   after do
@@ -260,6 +261,7 @@ describe "worker" do
     let(:effort_creator) do
       ->() do
         e = flow.start
+        flow.stamp_data_history = @stamp_data_history
         Seam::Effort.find(e.id)
       end
     end
@@ -550,8 +552,6 @@ describe "worker" do
                                           "started_at"=> Time.now, 
                                           "step"=>"wait_for_attempting_contact_stage",
                                           "stopped_at" => Time.now, 
-                                          "data_before" => { "first_name" => "DARREN" } ,
-                                          "data_after"  => { "first_name" => "DARREN", "hit 1" => 1 } 
                                         } )
 
       send_postcard_if_necessary_worker.execute_all
@@ -735,6 +735,85 @@ describe "worker" do
       orange_worker.execute_all
     end
   end
+
+  describe "data history" do
+    describe "stamping the history" do
+      let(:effort) do
+        flow = Seam::Flow.new
+        flow.stamp_data_history = true
+        flow.apple
+
+        e = flow.start( { first_name: 'John' } )
+        Seam::Effort.find(e.id)
+      end
+
+      before do
+        Timecop.freeze Time.parse('3/4/2013')
+        effort.next_step.must_equal "apple"
+
+        apple_worker = Seam::Worker.new
+        apple_worker.handles(:apple)
+        def apple_worker.process
+          effort.data['something'] = 'else'
+        end
+
+        apple_worker.execute effort
+      end
+
+      it "should not update the next step" do
+        fresh_effort = Seam::Effort.find(effort.id)
+        fresh_effort.history.count.must_equal 1
+      end
+
+      it "should set the data_before history" do
+        fresh_effort = Seam::Effort.find(effort.id)
+        fresh_effort.history.first["data_before"].must_equal( { "first_name" => 'John' } )
+      end
+
+      it "should set the data_after history" do
+        fresh_effort = Seam::Effort.find(effort.id)
+        fresh_effort.history.first["data_after"].must_equal( { "first_name" => 'John', "something" => 'else' } )
+      end
+    end
+
+    describe "not stamping the history" do
+      let(:effort) do
+        flow = Seam::Flow.new
+        flow.stamp_data_history = false
+        flow.apple
+
+        e = flow.start( { first_name: 'John' } )
+        Seam::Effort.find(e.id)
+      end
+
+      before do
+        Timecop.freeze Time.parse('3/4/2013')
+
+        apple_worker = Seam::Worker.new
+        apple_worker.handles(:apple)
+        def apple_worker.process
+          effort.data['something'] = 'else'
+        end
+
+        apple_worker.execute effort
+      end
+
+      it "should not update the next step" do
+        fresh_effort = Seam::Effort.find(effort.id)
+        fresh_effort.history.count.must_equal 1
+      end
+
+      it "should set the data_before history" do
+        fresh_effort = Seam::Effort.find(effort.id)
+        fresh_effort.history.first["data_before"].nil?.must_equal true
+      end
+
+      it "should set the data_after history" do
+        fresh_effort = Seam::Effort.find(effort.id)
+        fresh_effort.history.first["data_after"].nil?.must_equal true
+      end
+    end
+  end
 end
 
 class IWillNotCallHandlesWorker < Seam::Worker

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: seam
 version: !ruby/object:Gem::Version
-  version: 0.0.14
+  version: 0.0.15
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-29 00:00:00.000000000 Z
+date: 2013-09-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
@@ -228,7 +228,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2234950607663767066
+      hash: -3794428632218759846
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -237,7 +237,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2234950607663767066
+      hash: -3794428632218759846
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.25