hotseat 0.3.0 → 0.4.0

data/VERSION

@@ -1 +1 @@
-0.3.0
+0.4.0

data/hotseat.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "hotseat"
-  s.version = "0.3.0"
+  s.version = "0.4.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Elad Kehat"]
-  s.date = "2011-09-23"
+  s.date = "2012-02-16"
   s.email = "eladkehat@gmail.com"
   s.extra_rdoc_files = [
     "LICENSE.txt",

data/lib/hotseat/queue.rb

@@ -106,6 +106,12 @@ module Hotseat
       doc
     end
 
+    def done?(doc)
+      if obj = doc[config[:object_name]]
+        obj.has_key? 'done'
+      end
+    end
+
     def add(doc_id)
       @db.update_doc(doc_id) do |doc|
         patch doc
@@ -189,6 +195,26 @@ module Hotseat
       }
     end
 
+    def undo(doc_id)
+      @db.update_doc(doc_id) do |doc|
+        raise(QueueError, "Document is not done") unless done?(doc)
+        obj = doc[config[:object_name]]
+        obj.delete 'done'
+      end
+    end
+
+    def undo_bulk(doc_ids)
+      rows = @db.bulk_load(doc_ids)['rows']
+      docs, missing = rows.partition {|row| row['doc'] }
+      docs.map! {|row| row['doc'] }
+      done, not_done = docs.partition {|doc| done? doc }
+      done.each do |doc|
+        obj = doc[config[:object_name]]
+        obj.delete 'done'
+      end
+      @db.bulk_save done, use_uuids=false
+    end
+
     def num_done
       @db.view(done_view_name, :limit => 0)['total_rows']
     end

data/spec/hotseat/queue_spec.rb

@@ -114,6 +114,27 @@ module Hotseat
       end
     end
 
+    describe "#done?" do
+      before(:each) do
+        reset_test_queue!
+      end
+      it "should be true if the document was marked as done" do
+        doc = @q.mark_done( @q.patch( sample_doc ) )
+        @q.done?(doc).should be_true
+      end
+      it "should be false for a patched document that was not marked as done" do
+        doc = @q.patch( sample_doc )
+        @q.done?(doc).should be_false
+      end
+      it "should be false for a locked document" do
+        doc = @q.add_lock( @q.patch( sample_doc ) )
+        @q.done?(doc).should be_false
+      end
+      it "should be false for an un-patched document (not part of the queue)" do
+        @q.done?(sample_doc).should be_false
+      end
+    end
+
     describe "#add" do
       before(:each) do
         reset_test_queue!
@@ -418,6 +439,62 @@ module Hotseat
       end
     end
 
+    describe "#undo" do
+      before(:all) do
+        reset_test_queue!
+        doc_ids = create_some_docs(3)
+        enqueue doc_ids
+        @done_ids = [doc_ids[1]]
+        @undone_ids = [doc_ids[0], doc_ids[2]]
+        @done_ids.each do |done_id|
+          done_doc = DB.get done_id
+          @q.mark_done done_doc
+          done_doc.save
+        end
+      end
+
+      it "should mark the done document as pending again" do
+        @q.undo @done_ids.first
+        done_doc = DB.get @done_ids.first
+        patch = done_doc[@q.config[:object_name]]
+        patch.should_not have_key('done')
+        patch.should_not have_key('lock')
+      end
+
+      it "should raise an error if the document is not done" do
+        undone_doc = DB.get @undone_ids.first
+        expect {
+          @q.undo @undone_ids.first
+        }.to raise_error(Hotseat::QueueError)
+      end
+    end
+
+    describe "#undo_bulk" do
+      before(:each) do
+        reset_test_queue!
+        doc_ids = create_some_docs(5)
+        enqueue doc_ids
+        @done_docs = doc_ids[0..2].map{|id| DB.get id }
+        @done_docs.each do |doc|
+          @q.mark_done doc
+          doc.save
+        end
+      end
+
+      it "should mark all 'done' documents as pending again" do
+        @q.undo_bulk @done_docs.map{|doc| doc['_id']}
+        @q.num_done.should == 0
+        @q.num_pending.should == 5
+      end
+
+      it "should leave not-done documents intact" do
+        @q.lease
+        @q.undo_bulk @done_docs.map{|doc| doc['_id']}
+        @q.num_locked.should == 1
+        @q.num_pending.should == 4
+      end
+    end
+
     describe "#num_done" do
       it "should return the number of documents done" do
         reset_test_queue!

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hotseat
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-09-23 00:00:00.000000000Z
+date: 2012-02-16 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: couchrest
-  requirement: &87101760 !ruby/object:Gem::Requirement
+  requirement: &76368530 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: 1.1.0
   type: :runtime
   prerelease: false
-  version_requirements: *87101760
+  version_requirements: *76368530
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &87101330 !ruby/object:Gem::Requirement
+  requirement: &76368010 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *87101330
+  version_requirements: *76368010
 - !ruby/object:Gem::Dependency
   name: yard
-  requirement: &87100940 !ruby/object:Gem::Requirement
+  requirement: &76367640 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *87100940
+  version_requirements: *76367640
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &87100480 !ruby/object:Gem::Requirement
+  requirement: &76367340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *87100480
+  version_requirements: *76367340
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &87100060 !ruby/object:Gem::Requirement
+  requirement: &76367000 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,7 +65,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *87100060
+  version_requirements: *76367000
 description: 
 email: eladkehat@gmail.com
 executables: []
@@ -103,7 +103,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 841242607
+      hash: -1012521073
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: