hotseat 0.5.1 → 0.6.0

data/VERSION

@@ -1 +1 @@
-0.5.1
+0.6.0

data/hotseat.gemspec

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

data/lib/hotseat/queue.rb

@@ -164,6 +164,21 @@ module Hotseat
       end
     end
 
+    def unlease_bulk(doc_ids)
+      rows = @db.bulk_load(doc_ids)['rows']
+      docs, missing = rows.partition {|row| row['doc'] }
+      docs.map! {|row| row['doc'] }
+      locked, unlocked = docs.partition {|doc| locked? doc }
+      locked.each do |doc|
+        remove_lock( doc )
+      end
+      @db.bulk_save locked, use_uuids=false
+      {'errors' =>
+        unlocked.map {|doc| {'id' => doc['_id'], 'error' => 'unlocked' } } +
+        missing.map {|row| {'id' => row['key'], 'error' => row['error']} }
+      }
+    end
+
     def remove(doc_id, opts={})
       @db.update_doc(doc_id) do |doc|
         raise(QueueError, "Document was already removed") unless locked?(doc)

data/spec/hotseat/queue_spec.rb

@@ -318,6 +318,41 @@ module Hotseat
       end
     end
 
+    describe "#unlease_bulk" do
+      before(:each) do
+        reset_test_queue!
+        enqueue( create_some_docs(10) )
+        @leased = @q.lease 8
+        @doc_ids = @leased.take(5).map{|doc| doc['_id'] }
+      end
+
+      it "should unlock multiple leased documents" do
+        @q.unlease_bulk @doc_ids
+        docs = DB.get_bulk(@doc_ids)['rows'].map{|row| row['doc']}
+        docs.each do |doc|
+          doc.should have_key(@q.config[:object_name])
+          doc[@q.config[:object_name]].should_not have_key('lock')
+        end
+      end
+
+      it "should report docs whose lock was already removed" do
+        rem_ids = @doc_ids.take(2)
+        @q.unlease_bulk rem_ids
+        res = @q.unlease_bulk @doc_ids
+        res['errors'].should have(2).errors
+        res['errors'].map{|err| err['id']}.should == rem_ids
+      end
+
+      it "should report docs that are missing from the database" do
+        rem_ids = @doc_ids.take(2)
+        docs = rem_ids.map{|id| @q.db.get(id) }
+        docs.each {|doc| @q.db.delete_doc(doc) }
+        res = @q.unlease_bulk @doc_ids
+        res['errors'].should have(2).errors
+        res['errors'].map{|err| err['id']}.should == rem_ids
+      end
+    end
+
     describe "#remove" do
       before(:each) do
         reset_test_queue!

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hotseat
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.6.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-02-21 00:00:00.000000000Z
+date: 2012-02-27 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: couchrest
-  requirement: &84428420 !ruby/object:Gem::Requirement
+  requirement: &76050580 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: 1.1.0
   type: :runtime
   prerelease: false
-  version_requirements: *84428420
+  version_requirements: *76050580
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &84427770 !ruby/object:Gem::Requirement
+  requirement: &76049460 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *84427770
+  version_requirements: *76049460
 - !ruby/object:Gem::Dependency
   name: yard
-  requirement: &84414610 !ruby/object:Gem::Requirement
+  requirement: &76048250 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *84414610
+  version_requirements: *76048250
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &84413400 !ruby/object:Gem::Requirement
+  requirement: &76030030 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *84413400
+  version_requirements: *76030030
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &84412810 !ruby/object:Gem::Requirement
+  requirement: &76028720 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,7 +65,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *84412810
+  version_requirements: *76028720
 description: 
 email: eladkehat@gmail.com
 executables: []
@@ -103,7 +103,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 121392801
+      hash: -832976563
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: