bumbleworks 0.0.48 → 0.0.50

data/spec/lib/bumbleworks/{local_participant_spec.rb → participant/local_participant_spec.rb}

@@ -1,4 +1,4 @@
-describe Bumbleworks::LocalParticipant do
+describe Bumbleworks::Participant::LocalParticipant do
   it 'includes Ruote::LocalParticipant' do
     described_class.included_modules.should include(Ruote::LocalParticipant)
   end

data/spec/lib/bumbleworks/process_spec.rb

@@ -0,0 +1,147 @@
+describe Bumbleworks::Process do
+  before :each do
+    Bumbleworks.reset!
+    Bumbleworks.storage = {}
+    Bumbleworks::Ruote.register_participants
+    Bumbleworks.start_worker!
+
+    Bumbleworks.define_process 'going_to_the_dance' do
+      concurrence do
+        wait_for_event :an_invitation
+        await :left_tag => 'a_friend'
+      end
+    end
+    Bumbleworks.define_process 'straightening_the_rocks' do
+      concurrence do
+        wait_for_event :rock_caliper_delivery
+        wait_for_event :speedos
+      end
+    end
+  end
+
+  describe '.new' do
+    it 'sets workflow id' do
+      bp = described_class.new('apples')
+      bp.id.should == 'apples'
+    end
+  end
+
+  describe '#wfid' do
+    it 'is aliased to id' do
+      bp = described_class.new('smorgatoof')
+      bp.wfid.should == 'smorgatoof'
+    end
+  end
+
+  describe '#tasks' do
+    it 'returns task query filtered for this process' do
+      bp = described_class.new('chumpy')
+      Bumbleworks::Task.stub(:for_process).with('chumpy').and_return(:my_task_query)
+      bp.tasks.should == :my_task_query
+    end
+  end
+
+  describe '#trackers' do
+    it 'lists all trackers this process is waiting on' do
+      bp1 = Bumbleworks.launch!('going_to_the_dance')
+      bp2 = Bumbleworks.launch!('straightening_the_rocks')
+      wait_until { bp1.trackers.count == 2 && bp2.trackers.count == 2 }
+      bp1.trackers.map { |t| t['msg']['fei']['wfid'] }.should == [bp1.wfid, bp1.wfid]
+      bp2.trackers.map { |t| t['msg']['fei']['wfid'] }.should == [bp2.wfid, bp2.wfid]
+      bp1.trackers.map { |t| t['action'] }.should == ['left_tag', 'left_tag']
+      bp2.trackers.map { |t| t['action'] }.should == ['left_tag', 'left_tag']
+      bp1.trackers.map { |t| t['conditions']['tag'] }.should == [['an_invitation'], ['a_friend']]
+      bp2.trackers.map { |t| t['conditions']['tag'] }.should == [['rock_caliper_delivery'], ['speedos']]
+    end
+  end
+
+  describe '#all_subscribed_tags' do
+    it 'lists all tags this process is waiting on' do
+      bp1 = Bumbleworks.launch!('going_to_the_dance')
+      bp2 = Bumbleworks.launch!('straightening_the_rocks')
+      wait_until { bp1.trackers.count == 2 && bp2.trackers.count == 2 }
+      bp1.all_subscribed_tags.should == { :global => ['an_invitation'], bp1.wfid => ['a_friend'] }
+      bp2.all_subscribed_tags.should == { :global => ['rock_caliper_delivery', 'speedos'] }
+    end
+  end
+
+  describe '#subscribed_events' do
+    it 'lists all events (global tags) this process is waiting on' do
+      bp1 = Bumbleworks.launch!('going_to_the_dance')
+      bp2 = Bumbleworks.launch!('straightening_the_rocks')
+      wait_until { bp1.trackers.count == 2 && bp2.trackers.count == 2 }
+      bp1.subscribed_events.should == ['an_invitation']
+      bp2.subscribed_events.should == ['rock_caliper_delivery', 'speedos']
+    end
+  end
+
+  describe '#is_waiting_for?' do
+    it 'returns true if event is in subscribed events' do
+      bp = described_class.new('whatever')
+      bp.stub(:subscribed_events => ['ghosts', 'mouses'])
+      bp.is_waiting_for?('mouses').should be_true
+    end
+
+    it 'converts symbolized queries' do
+      bp = described_class.new('whatever')
+      bp.stub(:subscribed_events => ['ghosts', 'mouses'])
+      bp.is_waiting_for?(:ghosts).should be_true
+    end
+
+    it 'returns false if event is not in subscribed events' do
+      bp = described_class.new('whatever')
+      bp.stub(:subscribed_events => ['ghosts', 'mouses'])
+      bp.is_waiting_for?('organs').should be_false
+    end
+  end
+
+  describe '#kill!' do
+    it 'kills process' do
+      bp = described_class.new('frogheads')
+      Bumbleworks.should_receive(:kill_process!).with('frogheads')
+      bp.kill!
+    end
+  end
+
+  describe '#cancel!' do
+    it 'cancels process' do
+      bp = described_class.new('frogheads')
+      Bumbleworks.should_receive(:cancel_process!).with('frogheads')
+      bp.cancel!
+    end
+  end
+
+  describe '#==' do
+    it 'returns true if other object has same wfid' do
+      bp1 = described_class.new('in_da_sky')
+      bp2 = described_class.new('in_da_sky')
+      bp1.should == bp2
+    end
+  end
+
+  describe '#process_status' do
+    it 'returns a process_status instance for the wfid' do
+      bp = described_class.new('frogheads')
+      Bumbleworks.dashboard.stub(:process).with('frogheads').and_return(:the_status)
+      bp.process_status.should == :the_status
+    end
+  end
+
+  describe '#method_missing' do
+    it 'calls method on object returned by #process_status' do
+      ps = double('process_status')
+      ps.stub(:nuffle).with(:yay).and_return(:its_a_me)
+      bp = described_class.new('frogheads')
+      bp.stub(:process_status => ps)
+      bp.nuffle(:yay).should == :its_a_me
+    end
+
+    it 'falls back to method missing if no process status method' do
+      bp = described_class.new('blah')
+      bp.stub(:process_status => double('process status'))
+      expect {
+        bp.kerplunk!(:oh_no)
+      }.to raise_error
+    end
+  end
+end

data/spec/lib/bumbleworks/ruote/exp/broadcast_event_expression_spec.rb

@@ -18,7 +18,7 @@ describe Ruote::Exp::BroadcastEventExpression do
 
     waiter = Bumbleworks.launch!('waiter')
     sender = Bumbleworks.launch!('sender')
-    Bumbleworks.dashboard.wait_for(waiter)
+    Bumbleworks.dashboard.wait_for(waiter.wfid)
     @tracer.should == ['amazing']
   end
 end

data/spec/lib/bumbleworks/ruote/exp/wait_for_event_expression_spec.rb

@@ -33,7 +33,7 @@ describe Ruote::Exp::WaitForEventExpression do
 
     waiter = Bumbleworks.launch!('waiter')
     sender = Bumbleworks.launch!('sender')
-    Bumbleworks.dashboard.wait_for(waiter)
+    Bumbleworks.dashboard.wait_for(waiter.wfid)
     @tracer.should == ['get ready', 'oh my gosh almost there', 'hello', 'yay', 'yay2']
   end
 
@@ -53,7 +53,7 @@ describe Ruote::Exp::WaitForEventExpression do
     sender1 = Bumbleworks.launch!('sender', 'not_special' => 1)
     sender2 = Bumbleworks.launch!('sender', 'special' => 3)
 
-    Bumbleworks.dashboard.wait_for(waiter3)
+    Bumbleworks.dashboard.wait_for(waiter3.wfid)
     @tracer.should == ['specials! 3']
   end
 
@@ -73,7 +73,7 @@ describe Ruote::Exp::WaitForEventExpression do
     sender1 = Bumbleworks.launch!('sender', 'entity_type' => 'Pigeon', 'entity_id' => '13-6zoop')
     sender2 = Bumbleworks.launch!('sender', 'entity_type' => 'Rhubarb', 'entity_id' => 'spitpickle-4boof')
 
-    Bumbleworks.dashboard.wait_for(waiter3)
+    Bumbleworks.dashboard.wait_for(waiter3.wfid)
     @tracer.should == ['entities! Rhubarb,spitpickle-4boof']
   end
 end

data/spec/lib/bumbleworks/ruote_spec.rb

@@ -13,11 +13,11 @@ describe Bumbleworks::Ruote do
       Bumbleworks.define_process 'do_nothing' do
         lazy_guy :task => 'absolutely_nothing'
       end
-      wfid = Bumbleworks.launch!('do_nothing')
+      process = Bumbleworks.launch!('do_nothing')
       Bumbleworks.dashboard.wait_for(:lazy_guy)
-      Bumbleworks.dashboard.process(wfid).should_not be_nil
-      described_class.cancel_process!(wfid)
-      Bumbleworks.dashboard.process(wfid).should be_nil
+      Bumbleworks.dashboard.process(process.wfid).should_not be_nil
+      described_class.cancel_process!(process.wfid)
+      Bumbleworks.dashboard.process(process.wfid).should be_nil
     end
 
     it 'times out if process is not cancelled in time' do
@@ -29,11 +29,11 @@ describe Bumbleworks::Ruote do
           wait '1s'
         end
       end
-      wfid = Bumbleworks.launch!('time_hog')
+      process = Bumbleworks.launch!('time_hog')
       Bumbleworks.dashboard.wait_for(:pigheaded)
-      Bumbleworks.dashboard.process(wfid).should_not be_nil
+      Bumbleworks.dashboard.process(process.wfid).should_not be_nil
       expect {
-        described_class.cancel_process!(wfid, :timeout => 0.5)
+        described_class.cancel_process!(process.wfid, :timeout => 0.5)
       }.to raise_error(Bumbleworks::Ruote::CancelTimeout)
     end
   end
@@ -52,11 +52,11 @@ describe Bumbleworks::Ruote do
           wait '10s'
         end
       end
-      wfid = Bumbleworks.launch!('do_nothing')
+      process = Bumbleworks.launch!('do_nothing')
       Bumbleworks.dashboard.wait_for(:lazy_guy)
-      Bumbleworks.dashboard.process(wfid).should_not be_nil
-      described_class.kill_process!(wfid)
-      Bumbleworks.dashboard.process(wfid).should be_nil
+      Bumbleworks.dashboard.process(process.wfid).should_not be_nil
+      described_class.kill_process!(process.wfid)
+      Bumbleworks.dashboard.process(process.wfid).should be_nil
     end
 
     it 'times out if process is not killed in time' do
@@ -235,7 +235,10 @@ describe Bumbleworks::Ruote do
       described_class.register_participants
       described_class.start_worker!
       described_class.dashboard.participant_list.map(&:classname).should == [
-        'Bumbleworks::ErrorHandlerParticipant', 'Bumbleworks::StorageParticipant']
+        'Bumbleworks::Participant::ErrorHandler',
+        'Bumbleworks::Participant::EntityInteractor',
+        'Bumbleworks::Participant::StorageParticipant'
+      ]
       Bumbleworks.dashboard.on_error.flatten[2].should == 'error_handler_participant'
     end
   end
@@ -279,9 +282,13 @@ describe Bumbleworks::Ruote do
 
       described_class.dashboard.participant_list.should be_empty
       described_class.register_participants &registration_block
-      described_class.dashboard.participant_list.should have(5).items
+      described_class.dashboard.participant_list.should have(6).items
       described_class.dashboard.participant_list.map(&:classname).should == [
-        'Bumbleworks::ErrorHandlerParticipant', 'BeesHoney', 'MapleSyrup', 'NewCatchall', 'Bumbleworks::StorageParticipant']
+        'Bumbleworks::Participant::ErrorHandler',
+        'Bumbleworks::Participant::EntityInteractor',
+        'BeesHoney', 'MapleSyrup', 'NewCatchall',
+        'Bumbleworks::Participant::StorageParticipant'
+      ]
     end
 
     it 'does not add storage participant catchall if already exists' do
@@ -292,25 +299,25 @@ describe Bumbleworks::Ruote do
 
       described_class.dashboard.participant_list.should be_empty
       described_class.register_participants &registration_block
-      described_class.dashboard.participant_list.should have(3).items
+      described_class.dashboard.participant_list.should have(4).items
       described_class.dashboard.participant_list.map(&:classname).should == [
-        'Bumbleworks::ErrorHandlerParticipant', 'BeesHoney', 'Ruote::StorageParticipant'
+        'Bumbleworks::Participant::ErrorHandler', 'Bumbleworks::Participant::EntityInteractor', 'BeesHoney', 'Ruote::StorageParticipant'
       ]
     end
 
     it 'adds catchall and error_handler participants if block is nil' do
       described_class.dashboard.participant_list.should be_empty
       described_class.register_participants &nil
-      described_class.dashboard.participant_list.should have(2).item
+      described_class.dashboard.participant_list.should have(3).item
       described_class.dashboard.participant_list.map(&:classname).should ==
-        ['Bumbleworks::ErrorHandlerParticipant', 'Bumbleworks::StorageParticipant']
+        ['Bumbleworks::Participant::ErrorHandler', 'Bumbleworks::Participant::EntityInteractor', 'Bumbleworks::Participant::StorageParticipant']
     end
   end
 
   describe '.register_error_handler', dev:true do
     it 'registers the error handler participant' do
       described_class.register_error_handler
-      Bumbleworks.dashboard.participant_list.map(&:classname).should include('Bumbleworks::ErrorHandlerParticipant')
+      Bumbleworks.dashboard.participant_list.map(&:classname).should include('Bumbleworks::Participant::ErrorHandler')
     end
 
     it 'it sets the global Ruote on_error to the error_handler_participant' do
@@ -324,7 +331,7 @@ describe Bumbleworks::Ruote do
       end
       described_class.register_error_handler
       Bumbleworks.dashboard.participant_list.map(&:classname).should ==
-        ['Whatever', 'Bumbleworks::StorageParticipant']
+        ['Bumbleworks::Participant::EntityInteractor', 'Whatever', 'Bumbleworks::Participant::StorageParticipant']
 
     end
   end
@@ -362,7 +369,7 @@ describe Bumbleworks::Ruote do
       described_class.dashboard.participant_list.should be_empty
       described_class.launch('foo')
       described_class.dashboard.participant_list.should have(1).item
-      described_class.dashboard.participant_list.first.classname.should == 'Bumbleworks::StorageParticipant'
+      described_class.dashboard.participant_list.first.classname.should == 'Bumbleworks::Participant::StorageParticipant'
     end
   end
 

data/spec/lib/bumbleworks/task/finder_spec.rb

@@ -31,4 +31,13 @@ describe Bumbleworks::Task::Finder do
       Object.send(:remove_const, :MyOwnTask)
     end
   end
+
+  describe '#available' do
+    it 'adds both unclaimed and completable filters' do
+      query = Bumbleworks::Task::Finder.new
+      query.should_receive(:unclaimed).and_return(query)
+      query.should_receive(:completable).and_return(query)
+      query.available
+    end
+  end
 end

data/spec/lib/bumbleworks/task_spec.rb

@@ -131,7 +131,7 @@ describe Bumbleworks::Task do
     it 'extends with base module and task module' do
       task = described_class.new(workflow_item)
       task.should_receive(:task_module).and_return(:task_module_double)
-      task.should_receive(:extend).with(Bumbleworks::Tasks::Base).ordered
+      task.should_receive(:extend).with(Bumbleworks::Task::Base).ordered
       task.should_receive(:extend).with(:task_module_double).ordered
       task.extend_module
     end
@@ -139,13 +139,13 @@ describe Bumbleworks::Task do
     it 'extends only with base module if no nickname' do
       task = described_class.new(workflow_item)
       task.stub(:nickname).and_return(nil)
-      task.should_receive(:extend).with(Bumbleworks::Tasks::Base)
+      task.should_receive(:extend).with(Bumbleworks::Task::Base)
       task.extend_module
     end
 
     it 'extends only with base module if task module does not exist' do
       task = described_class.new(workflow_item)
-      task.should_receive(:extend).with(Bumbleworks::Tasks::Base)
+      task.should_receive(:extend).with(Bumbleworks::Task::Base)
       task.extend_module
     end
   end
@@ -250,6 +250,61 @@ describe Bumbleworks::Task do
     end
   end
 
+  describe '.for_processes' do
+    before :each do
+      Bumbleworks.define_process 'spunking' do
+        concurrence do
+          spunker :task => 'spunk'
+          nonspunker :task => 'complain'
+        end
+      end
+      Bumbleworks.define_process 'rooting' do
+        concurrence do
+          rooter :task => 'get_the_rooting_on'
+          armchair_critic :task => 'scoff'
+        end
+      end
+      @spunking_process = Bumbleworks.launch!('spunking')
+      @rooting_process_1 = Bumbleworks.launch!('rooting')
+      @rooting_process_2 = Bumbleworks.launch!('rooting')
+      Bumbleworks.dashboard.wait_for(:armchair_critic)
+    end
+
+    it 'returns tasks for given processes' do
+      spunking_tasks = described_class.for_processes([@spunking_process])
+      rooting_tasks = described_class.for_processes([@rooting_process_1])
+      tasks_for_both = described_class.for_processes([@spunking_process, @rooting_process_1])
+
+      spunking_tasks.map(&:nickname).should =~ ['spunk', 'complain']
+      rooting_tasks.map(&:nickname).should =~ ['get_the_rooting_on', 'scoff']
+      tasks_for_both.map(&:nickname).should =~ ['spunk', 'complain', 'get_the_rooting_on', 'scoff']
+    end
+
+    it 'works with process ids as well' do
+      spunking_tasks = described_class.for_processes([@spunking_process.id])
+      spunking_tasks.map(&:nickname).should =~ ['spunk', 'complain']
+    end
+
+    it 'returns empty array when no tasks for given process id' do
+      described_class.for_processes(['boop']).should be_empty
+    end
+
+    it 'returns empty array if given empty array' do
+      described_class.for_processes([]).should be_empty
+    end
+
+    it 'returns empty array if given nil' do
+      described_class.for_processes(nil).should be_empty
+    end
+  end
+
+  describe '.for_process' do
+    it 'acts as shortcut to .for_processes with one process' do
+      described_class::Finder.any_instance.should_receive(:for_processes).with([:one_guy]).and_return(:aha)
+      described_class.for_process(:one_guy).should == :aha
+    end
+  end
+
   describe '.for_role' do
     it 'returns all tasks for given role' do
       Bumbleworks.define_process 'chalking' do
@@ -315,6 +370,32 @@ describe Bumbleworks::Task do
     end
   end
 
+  describe '.completable' do
+    it 'returns only completable tasks' do
+      module WuggleHandsTask
+        def completable?
+          false
+        end
+      end
+
+      Bumbleworks.define_process 'hand_waggling' do
+        concurrence do
+          a_fella :task => 'waggle_hands'
+          a_monkey :task => 'wuggle_hands'
+          a_lady :task => 'wiggle_hands'
+        end
+      end
+      Bumbleworks.launch!('hand_waggling')
+      Bumbleworks.dashboard.wait_for(:a_lady)
+      tasks = described_class.completable
+      tasks.should have(2).items
+      tasks.map { |t| [t.role, t.nickname] }.should == [
+        ['a_fella', 'waggle_hands'],
+        ['a_lady', 'wiggle_hands']
+      ]
+    end
+  end
+
   describe '.all' do
     before :each do
       Bumbleworks.define_process 'dog-lifecycle' do
@@ -668,6 +749,12 @@ describe Bumbleworks::Task do
 
   describe 'chained queries' do
     it 'allows for AND-ed chained finders' do
+      module BeProudTask
+        def completable?
+          role == 'pink'
+        end
+      end
+
       Bumbleworks.define_process 'the_big_kachunko' do
         concurrence do
           red :task => 'be_really_mad'
@@ -690,6 +777,14 @@ describe Bumbleworks::Task do
       tasks.should have(2).items
       tasks.map(&:nickname).should =~ ['be_proud', 'be_proud']
 
+      tasks = described_class.
+        for_roles(['green', 'pink', 'blue']).
+        completable.
+        by_nickname('be_proud')
+      tasks.should have(1).items
+      tasks.map(&:nickname).should =~ ['be_proud']
+      tasks.first.role.should == 'pink'
+
       tasks = described_class.
         for_claimant('crayon_box').
         for_roles(['red', 'yellow', 'green'])