seam-mongodb 0.0.2 → 0.0.3

data/.gitignore

@@ -15,3 +15,4 @@ spec/reports
 test/tmp
 test/version_tmp
 tmp
+.DS_Store

data/lib/seam/mongodb/version.rb

@@ -1,5 +1,5 @@
 module Seam
   module Mongodb
-    VERSION = "0.0.2"
+    VERSION = "0.0.3"
   end
 end

data/lib/seam/mongodb.rb

@@ -1,6 +1,7 @@
 require 'seam'
 require "seam/mongodb/version"
 require 'moped'
+require 'subtle'
 
 module Seam
   module Mongodb
@@ -25,7 +26,11 @@ module Seam
         def self.find_all_pending_executions_by_step step
           Seam::Mongodb.collection
             .find( { next_step: step, next_execute_at: { '$lte' => Time.now } } )
-            .map { |x| Seam::Effort.parse x }
+            .map do |x|
+              -> do
+                Seam::Effort.parse x
+              end.to_object
+            end
         end
 
         def self.save effort

data/seam-mongodb.gemspec

@@ -24,9 +24,9 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency "moped"
   spec.add_runtime_dependency "json"
   spec.add_runtime_dependency "seam"
+  spec.add_runtime_dependency 'subtle'
   spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "rake"
-  spec.add_development_dependency "subtle"
   spec.add_development_dependency "contrast"
   spec.add_development_dependency "mocha"
   spec.add_development_dependency "contrast"

data/spec/seam/flow_spec.rb

@@ -94,4 +94,39 @@ describe "flow" do
       end
     end
   end
+
+  describe "adding steps" do
+    describe "new steps" do
+      it "should return true" do
+        flow = Seam::Flow.new
+        flow.do_something.must_equal true
+        flow.do_something_else.must_equal true
+      end
+    end
+
+    describe "repeating steps" do
+      it "should only add the step once" do
+        flow = Seam::Flow.new
+        flow.do_something.must_equal true
+        flow.steps.count.must_equal 1
+        flow.do_something.must_equal true
+        flow.steps.count.must_equal 2
+      end
+    end
+
+    describe "repeating steps with different data" do
+      it "should only add the step once" do
+        flow = Seam::Flow.new
+        flow.do_something(special_id: 'one').must_equal true
+        flow.do_something( { special_id: 'two' }, 4).must_equal true
+        flow.steps.count.must_equal 2
+
+        flow.steps[0].arguments.count.must_equal 1
+        flow.steps[0].arguments[0].contrast_with!( { special_id: 'one' } )
+        flow.steps[1].arguments.count.must_equal 2
+        flow.steps[1].arguments[0].contrast_with!( { special_id: 'two' } )
+        flow.steps[1].arguments[1].must_equal 4
+      end
+    end
+  end
 end

data/spec/seam/step_spec.rb

@@ -0,0 +1,30 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe Seam::Step do
+  describe "initialize" do
+    it "should allow values to be set with the constructor" do
+      step = Seam::Step.new( { name: 'a name',
+                               type: 'a type',
+                               arguments: ['1234', 2] } )
+      step.name.must_equal 'a name'
+      step.type.must_equal 'a type'
+      step.arguments.count.must_equal 2
+      step.arguments[0].must_equal '1234'
+      step.arguments[1].must_equal 2
+    end
+  end
+
+  describe "to hash" do
+    it "should allow values to be set with the constructor" do
+      step = Seam::Step.new( { name: 'a name',
+                               type: 'a type',
+                               arguments: ['1234', 2] } )
+      step = Seam::Step.new step.to_hash
+      step.name.must_equal 'a name'
+      step.type.must_equal 'a type'
+      step.arguments.count.must_equal 2
+      step.arguments[0].must_equal '1234'
+      step.arguments[1].must_equal 2
+    end
+  end
+end

data/spec/seam/worker_spec.rb

@@ -10,6 +10,14 @@ describe "worker" do
     Timecop.return
   end
 
+  describe "step" do
+    it "should match the name" do
+      worker = Seam::Worker.new
+      worker.handles(:darren)
+      worker.step.must_equal "darren"
+    end
+  end
+
   describe "move_to_next_step" do
     it "should work" do
       flow = Seam::Flow.new
@@ -34,6 +42,29 @@ describe "worker" do
     end
   end
 
+  describe "move_to_next_step as a default" do
+    it "should go to move_to_next_step by default" do
+      flow = Seam::Flow.new
+      flow.apple
+      flow.orange
+
+      effort = flow.start( { first_name: 'John' } )
+      effort = Seam::Effort.find(effort.id)
+
+      effort.next_step.must_equal "apple"
+
+      apple_worker = Seam::Worker.new
+      apple_worker.handles(:apple)
+      def apple_worker.process
+      end
+
+      apple_worker.execute effort
+
+      effort = Seam::Effort.find(effort.id)
+      effort.next_step.must_equal "orange"
+    end
+  end
+
   describe "try_again_in" do
 
     let(:effort) do
@@ -63,12 +94,54 @@ describe "worker" do
       fresh_effort.next_step.must_equal "apple"
     end
 
-    it "should not update the next execute date" do
+    it "should update the next execute date" do
       fresh_effort = Seam::Effort.find(effort.id)
       fresh_effort.next_execute_at.must_equal Time.parse('4/4/2013')
     end
   end
 
+  describe "try_again_on" do
+
+    describe "putting it off for one day" do
+      let(:effort) do
+        flow = Seam::Flow.new
+        flow.apple
+        flow.orange
+
+        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
+          try_again_on Time.parse('4/4/2013')
+        end
+
+        apple_worker.execute effort
+      end
+
+      it "should not update the next step" do
+        fresh_effort = Seam::Effort.find(effort.id)
+        fresh_effort.next_step.must_equal "apple"
+      end
+
+      it "should update the next execute date" do
+        fresh_effort = Seam::Effort.find(effort.id)
+        fresh_effort.next_execute_at.must_equal Time.parse('4/4/2013')
+      end
+
+      it "should update the history" do
+        fresh_effort = Seam::Effort.find(effort.id)
+        fresh_effort.history[0]['try_again_on'].must_equal Time.parse('4/4/2013')
+      end
+    end
+  end
+
   describe "more copmlex example" do
 
     let(:effort1) do
@@ -271,7 +344,7 @@ describe "worker" do
       effort.data['hit 3'].must_equal 1
 
       effort.complete?.must_equal true
-      #effort.completed_at.must_equal Time.now
+      effort.completed_at.must_equal Time.now
       
       # FUTURE WAVES
       send_postcard_if_necessary_worker.execute_all
@@ -575,10 +648,96 @@ describe "worker" do
       fresh_effort = Seam::Effort.find(effort.id)
       fresh_effort.next_step.nil?.must_equal true
     end
+
+    it "should mark the completed_at date" do
+      fresh_effort = Seam::Effort.find(effort.id)
+      fresh_effort.completed_at.must_equal Time.now
+    end
     
     it "should mark the history" do
       effort.history[0].contrast_with!({"step"=>"apple", "result" => "eject" } )
     end
 
   end
+
+  describe "use the name of the worker to tie to a step" do
+
+    let(:effort) do
+      flow = Seam::Flow.new
+      flow.i_will_not_call_handles
+
+      e = flow.start
+      Seam::Effort.find(e.id)
+    end
+
+    before do
+      effort
+      worker = IWillNotCallHandlesWorker.new
+      IWillNotCallHandlesWorker.new.execute_all
+    end
+
+    it "should complete the effort" do
+      fresh_effort = Seam::Effort.find effort.id
+      fresh_effort.complete?.must_equal true
+    end
+
+  end
+
+  describe "making the current step available" do
+    it "should return the first step if on the first step" do
+      flow = Seam::Flow.new
+      flow.apple("test")
+      flow.orange
+
+      effort = flow.start( { first_name: 'John' } )
+      effort = Seam::Effort.find(effort.id)
+
+      effort.next_step.must_equal "apple"
+
+      apple_worker = Seam::Worker.new
+      apple_worker.handles(:apple)
+      def apple_worker.process
+        current_step.nil?.must_equal false
+        current_step["name"].must_equal "apple"
+        current_step["arguments"].must_equal ["test"]
+      end
+
+      apple_worker.execute effort
+    end
+
+    it "should return the second step if on the second step" do
+      flow = Seam::Flow.new
+      flow.apple("test")
+      flow.orange("another test")
+
+      effort = flow.start( { first_name: 'John' } )
+      effort = Seam::Effort.find(effort.id)
+
+      effort.next_step.must_equal "apple"
+
+      apple_worker = Seam::Worker.new
+      apple_worker.handles(:apple)
+      def apple_worker.process
+        current_step.nil?.must_equal false
+        current_step["name"].must_equal "apple"
+        current_step["arguments"].must_equal ["test"]
+      end
+
+      orange_worker = Seam::Worker.new
+      orange_worker.handles(:orange)
+      def orange_worker.process
+        current_step.nil?.must_equal false
+        current_step["name"].must_equal "orange"
+        current_step["arguments"].must_equal ["another test"]
+      end
+
+      apple_worker.execute_all
+      orange_worker.execute_all
+    end
+  end
+end
+
+class IWillNotCallHandlesWorker < Seam::Worker
+  # no calling handles here
+  def process; end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: seam-mongodb
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-08 00:00:00.000000000 Z
+date: 2013-09-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
@@ -108,39 +108,39 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: subtle
   requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - ! '>='
       - !ruby/object:Gem::Version
-        version: '1.3'
-  type: :development
+        version: '0'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - ! '>='
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: bundler
   requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ! '>='
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ! '>='
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.3'
 - !ruby/object:Gem::Dependency
-  name: subtle
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
@@ -236,6 +236,7 @@ files:
 - seam-mongodb.gemspec
 - spec/seam/effort_spec.rb
 - spec/seam/flow_spec.rb
+- spec/seam/step_spec.rb
 - spec/seam/worker_spec.rb
 - spec/spec_helper.rb
 homepage: ''
@@ -253,7 +254,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2083924864139500401
+      hash: -2871184592722645062
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -262,7 +263,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2083924864139500401
+      hash: -2871184592722645062
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.25
@@ -272,5 +273,6 @@ summary: MongoDB support for seam
 test_files:
 - spec/seam/effort_spec.rb
 - spec/seam/flow_spec.rb
+- spec/seam/step_spec.rb
 - spec/seam/worker_spec.rb
 - spec/spec_helper.rb