lookout-statsd 0.7.0 → 0.7.1

data/Gemfile

@@ -1,16 +1,16 @@
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
-gem 'rake'
+gem "rake"
 
 group :test do
   if RUBY_VERSION > "1.9"
-    gem 'debugger'
+    gem "ruby-debug19", :require => 'ruby-debug'
   else
-    gem 'ruby-debug'
+    gem "ruby-debug"
   end
 
-  gem 'rspec'
-  gem 'cucumber'
+  gem "rspec"
+  gem "cucumber"
 end
 
 gemspec

data/lib/statsd.rb

@@ -52,24 +52,19 @@ module Statsd
 
     # +stats+ can be a string or an array of strings
     def increment(stats, sample_rate = 1)
-      if @prefix
-        stats = "#{@prefix}.#{stats}"
-      end
       update_counter stats, 1, sample_rate
     end
 
     # +stats+ can be a string or an array of strings
     def decrement(stats, sample_rate = 1)
-      if @prefix
-        stats = "#{@prefix}.#{stats}"
-      end
       update_counter stats, -1, sample_rate
     end
 
     # +stats+ can be a string or array of strings
     def update_counter(stats, delta = 1, sample_rate = 1)
       stats = Array(stats)
-      send_stats(stats.map { |s| "#{s}:#{delta}|c" }, sample_rate)
+      p = @prefix ? "#{@prefix}." : '' # apply prefix to each
+      send_stats(stats.map { |s| "#{p}#{s}:#{delta}|c" }, sample_rate)
     end
 
     # +stats+ is a hash

data/spec/statsd_spec.rb

@@ -93,9 +93,8 @@ describe Statsd::Client do
   describe '#increment' do
     let(:c) { Statsd::Client.new }
 
-    it 'should prepend the prefix if it has one' do
-      c.prefix = 'dev'
-      c.should_receive(:update_counter).with('dev.foo', anything(), anything())
+    it 'should update the counter by 1' do
+      c.should_receive(:update_counter).with('foo', 1, anything())
       c.increment('foo')
     end
   end
@@ -103,10 +102,25 @@ describe Statsd::Client do
   describe '#decrement' do
     let(:c) { Statsd::Client.new }
 
+    it 'should update the counter by -1' do
+      c.should_receive(:update_counter).with('foo', -1, anything())
+      c.decrement('foo')
+    end
+  end
+
+  describe '#update_counter' do
+    let(:c) { Statsd::Client.new }
+
     it 'should prepend the prefix if it has one' do
       c.prefix = 'dev'
-      c.should_receive(:update_counter).with('dev.foo', anything(), anything())
-      c.decrement('foo')
+      c.should_receive(:send_stats).with(['dev.foo:123|c'], anything())
+      c.update_counter('foo', 123)
+    end
+
+    it 'should prepend multiple prefixes if it has one' do
+      c.prefix = 'dev'
+      c.should_receive(:send_stats).with(['dev.foo:123|c', 'dev.bar:123|c'], anything())
+      c.update_counter(['foo', 'bar'], 123)
     end
   end
 

data/statsd.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
   s.name        = "lookout-statsd"
-  s.version     = "0.7.#{ENV['BUILD_NUMBER'] || 'dev'}"
+  s.version     = "0.7.1"
   s.platform    = Gem::Platform::RUBY
 
   s.authors     = ['R. Tyler Croy', 'Andrew Coldham', 'Ben VandenBos']

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: lookout-statsd
 version: !ruby/object:Gem::Version 
-  hash: 3
+  hash: 1
   prerelease: 
   segments: 
   - 0
   - 7
-  - 0
-  version: 0.7.0
+  - 1
+  version: 0.7.1
 platform: ruby
 authors: 
 - R. Tyler Croy
@@ -17,9 +17,11 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2013-04-08 00:00:00 Z
+date: 2013-05-14 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
+  type: :runtime
+  name: eventmachine
   version_requirements: &id001 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
@@ -31,11 +33,11 @@ dependencies:
         - 12
         - 10
         version: 0.12.10
-  prerelease: false
-  type: :runtime
   requirement: *id001
-  name: eventmachine
+  prerelease: false
 - !ruby/object:Gem::Dependency 
+  type: :runtime
+  name: erubis
   version_requirements: &id002 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
@@ -47,10 +49,8 @@ dependencies:
         - 6
         - 6
         version: 2.6.6
-  prerelease: false
-  type: :runtime
   requirement: *id002
-  name: erubis
+  prerelease: false
 description: A network daemon for aggregating statistics (counters and timers), rolling them up, then sending them to graphite.
 email: 
 - rtyler.croy@mylookout.com