scribe-rb 2.2.rc5 → 2.2.rc6

data/lib/extras/queued_log_message_handler.rb

@@ -16,7 +16,7 @@ module FacebookService
     end
 
     def Log(messages)
-      if self.message_limit.nil? || self.queue.size < self.message_limit
+      if self.message_limit.nil? || (messages.length + self.queue.size) <= self.message_limit
         messages.each do |message|
           self.queue.push(message)
         end

data/scribe-rb.gemspec

@@ -3,7 +3,7 @@ $:.push File.expand_path("../lib", __FILE__)
 
 Gem::Specification.new do |s|
   s.name        = "scribe-rb"
-  s.version     = "2.2.rc5"
+  s.version     = "2.2.rc6"
   s.authors     = ["Andy Lindeman"]
   s.email       = ["andy@highgroove.com"]
   s.homepage    = "http://github.com/highgroove/scribe-rb"

data/spec/lib/extras/queued_log_message_handler_spec.rb

@@ -35,5 +35,13 @@ describe FacebookService::QueuedLogMessageHandler do
       subject.Log(["foo"]).should eq(ResultCode::TRY_LATER)
       subject.queue.size.should be_zero
     end
+
+    it "does not accept messages and returns ResultCode::TRY_LATER if the size limit would be reached by adding the messages" do
+      subject.message_limit = 1
+
+      subject.Log(["foo"]).should eq(ResultCode::OK)
+      subject.Log(["bar"]).should eq(ResultCode::TRY_LATER)
+      subject.queue.size.should eq(1)
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: scribe-rb
 version: !ruby/object:Gem::Version
-  version: 2.2.rc5
+  version: 2.2.rc6
   prerelease: 4
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-09-20 00:00:00.000000000Z
+date: 2011-09-21 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thrift
-  requirement: &2156647800 !ruby/object:Gem::Requirement
+  requirement: &2164580280 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 0.7.0
   type: :runtime
   prerelease: false
-  version_requirements: *2156647800
+  version_requirements: *2164580280
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &2156647060 !ruby/object:Gem::Requirement
+  requirement: &2164579860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2156647060
+  version_requirements: *2164579860
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &2156645840 !ruby/object:Gem::Requirement
+  requirement: &2164579320 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: 2.6.0
   type: :development
   prerelease: false
-  version_requirements: *2156645840
+  version_requirements: *2164579320
 - !ruby/object:Gem::Dependency
   name: timecop
-  requirement: &2156645080 !ruby/object:Gem::Requirement
+  requirement: &2164578900 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,7 +54,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2156645080
+  version_requirements: *2164578900
 description: Everything you need to build a Ruby producer/client or consumer/server
   of Scribe messages
 email:
@@ -96,9 +96,6 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: -3771262154631515498
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: