klomp 1.0.6 → 1.0.7

data/ChangeLog.md

@@ -1,6 +1,12 @@
 Klomp Changes
 --------------------------------------------------------------------------------
 
+1.0.7 (2013/02/04)
+================================================================================
+
+- Ruby 1.8 support for the laggards
+- Add empty .gemtest and fix gemspec so klomp may be used directly from git
+
 1.0.6 (2012/11/19)
 ================================================================================
 

data/Gemfile

@@ -15,6 +15,7 @@ gem "rspec-given", "~>1.0", :group => [:development, :test]
 gem "simplecov", "~>0.6.0", :group => [:development, :test]
 gem "em-proxy", "~>0.1.0", :group => [:development, :test]
 gem "ci_reporter", "~>1.7.0", :group => [:development, :test]
+gem "activesupport", ">=2.3.0", :group => [:development, :test]
 gem "hoe", "~>3.0", :group => [:development, :test]
 
 # vim: syntax=ruby

data/Gemfile.lock

@@ -2,6 +2,7 @@ GEM
   remote: http://rubygems.org/
   specs:
     ZenTest (4.8.0)
+    activesupport (2.3.16)
     builder (3.1.4)
     ci_reporter (1.7.3)
       builder (>= 2.1.2)
@@ -42,6 +43,7 @@ PLATFORMS
 
 DEPENDENCIES
   ZenTest (~> 4.8.0)
+  activesupport (>= 2.3.0)
   ci_reporter (~> 1.7.0)
   em-proxy (~> 0.1.0)
   hoe (~> 3.0)

data/Manifest.txt

@@ -1,3 +1,4 @@
+.gemtest
 .rspec
 .simplecov
 ChangeLog.md

data/Rakefile

@@ -13,15 +13,16 @@ Hoe.spec 'klomp' do
   self.clean_globs << 'spec/reports'
 
   ### dependencies!
-  self.extra_dev_deps << [ 'hoe-bundler',  '~> 1.1.0'  ]
-  self.extra_dev_deps << [ 'hoe-gemspec',  '~> 1.0.0'  ]
-  self.extra_dev_deps << [ 'hoe-git',      '~> 1.5.0'  ]
-  self.extra_dev_deps << [ 'rspec',        '~> 2.11.0' ]
-  self.extra_dev_deps << [ 'ZenTest',      '~> 4.8.0'  ]
-  self.extra_dev_deps << [ 'rspec-given',  '~> 1.0'    ]
-  self.extra_dev_deps << [ 'simplecov',    '~> 0.6.0'  ]
-  self.extra_dev_deps << [ 'em-proxy',     '~> 0.1.0'  ]
-  self.extra_dev_deps << [ 'ci_reporter',  '~> 1.7.0'  ]
+  self.extra_dev_deps << [ 'hoe-bundler',   '~> 1.1.0'  ]
+  self.extra_dev_deps << [ 'hoe-gemspec',   '~> 1.0.0'  ]
+  self.extra_dev_deps << [ 'hoe-git',       '~> 1.5.0'  ]
+  self.extra_dev_deps << [ 'rspec',         '~> 2.11.0' ]
+  self.extra_dev_deps << [ 'ZenTest',       '~> 4.8.0'  ]
+  self.extra_dev_deps << [ 'rspec-given',   '~> 1.0'    ]
+  self.extra_dev_deps << [ 'simplecov',     '~> 0.6.0'  ]
+  self.extra_dev_deps << [ 'em-proxy',      '~> 0.1.0'  ]
+  self.extra_dev_deps << [ 'ci_reporter',   '~> 1.7.0'  ]
+  self.extra_dev_deps << [ 'activesupport', '>= 2.3.0'  ]
 end
 
 require 'ci/reporter/rake/rspec'

data/klomp.gemspec

@@ -2,15 +2,15 @@
 
 Gem::Specification.new do |s|
   s.name = "klomp"
-  s.version = "1.0.6"
+  s.version = "1.0.7"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Nick Sieger"]
-  s.date = "2012-10-17"
+  s.date = "2013-02-04"
   s.description = "Klomp is a simple [Stomp] messaging client that keeps your sanity intact.\n\nThe purpose of Klomp is to be the simplest possible Stomp client. No in-memory\nbuffering of outgoing messages, no fanout subscriptions in-process, no\ntransactions, no complicated messaging patterns. Code simple enough so that when\nsomething goes wrong, the problem is obvious.\n\n[Stomp]: http://stomp.github.com/"
   s.email = ["nick.sieger@livingsocial.com"]
   s.extra_rdoc_files = ["Manifest.txt"]
-  s.files = [".rspec", ".simplecov", "ChangeLog.md", "Gemfile", "Gemfile.lock", "Manifest.txt", "README.md", "Rakefile", "klomp.gemspec", "lib/klomp.rb", "lib/klomp/connection.rb", "lib/klomp/frames.rb", "lib/klomp/sentinel.rb", "spec/acceptance/acceptance_spec.rb", "spec/frames/auth_error.txt", "spec/frames/connect.txt", "spec/frames/connect_vhost.txt", "spec/frames/connected.txt", "spec/frames/disconnect.txt", "spec/frames/error.txt", "spec/frames/greeting.txt", "spec/frames/message.txt", "spec/frames/receipt.txt", "spec/frames/subscribe.txt", "spec/frames/unsubscribe.txt", "spec/klomp/connection_spec.rb", "spec/klomp/frames_spec.rb", "spec/klomp/sentinel_spec.rb", "spec/klomp_spec.rb", "spec/spec_helper.rb", "spec/support/have_received.rb", ".gemtest"]
+  s.files = [".gemtest", ".rspec", ".simplecov", "ChangeLog.md", "Gemfile", "Gemfile.lock", "Manifest.txt", "README.md", "Rakefile", "klomp.gemspec", "lib/klomp.rb", "lib/klomp/connection.rb", "lib/klomp/frames.rb", "lib/klomp/sentinel.rb", "spec/acceptance/acceptance_spec.rb", "spec/acceptance/reconnect_spec.rb", "spec/frames/auth_error.txt", "spec/frames/connect.txt", "spec/frames/connect_vhost.txt", "spec/frames/connected.txt", "spec/frames/disconnect.txt", "spec/frames/error.txt", "spec/frames/greeting.txt", "spec/frames/message.txt", "spec/frames/receipt.txt", "spec/frames/subscribe.txt", "spec/frames/unsubscribe.txt", "spec/klomp/connection_spec.rb", "spec/klomp/frames_spec.rb", "spec/klomp/sentinel_spec.rb", "spec/klomp_spec.rb", "spec/spec_helper.rb", "spec/support/have_received.rb", "spec/support/shared_contexts.rb"]
   s.homepage = "http://github.com/livingsocial/klomp"
   s.rdoc_options = ["--main", "README.md"]
   s.require_paths = ["lib"]
@@ -32,6 +32,7 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<simplecov>, ["~> 0.6.0"])
       s.add_development_dependency(%q<em-proxy>, ["~> 0.1.0"])
       s.add_development_dependency(%q<ci_reporter>, ["~> 1.7.0"])
+      s.add_development_dependency(%q<activesupport>, [">= 2.3.0"])
       s.add_development_dependency(%q<hoe>, ["~> 3.0"])
     else
       s.add_dependency(%q<rdoc>, ["~> 3.10"])
@@ -44,6 +45,7 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<simplecov>, ["~> 0.6.0"])
       s.add_dependency(%q<em-proxy>, ["~> 0.1.0"])
       s.add_dependency(%q<ci_reporter>, ["~> 1.7.0"])
+      s.add_dependency(%q<activesupport>, [">= 2.3.0"])
       s.add_dependency(%q<hoe>, ["~> 3.0"])
     end
   else
@@ -57,6 +59,7 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<simplecov>, ["~> 0.6.0"])
     s.add_dependency(%q<em-proxy>, ["~> 0.1.0"])
     s.add_dependency(%q<ci_reporter>, ["~> 1.7.0"])
+    s.add_dependency(%q<activesupport>, [">= 2.3.0"])
     s.add_dependency(%q<hoe>, ["~> 3.0"])
   end
 end

data/lib/klomp.rb

@@ -1,5 +1,5 @@
 class Klomp
-  VERSION = '1.0.6'
+  VERSION = '1.0.7'
 
   class Error < StandardError; end
 
@@ -14,7 +14,7 @@ class Klomp
   def publish(queue, body, headers = {})
     connections_remaining = connections.dup
     begin
-      conn = connections_remaining.sample
+      conn = connections_remaining[rand(connections_remaining.size)]
       conn.publish(queue, body, headers)
     rescue
       connections_remaining.delete conn

data/lib/klomp/connection.rb

@@ -84,7 +84,7 @@ class Klomp
     private
     def connect
       @socket  = TCPSocket.new *options['server']
-      @socket.set_encoding 'UTF-8'
+      @socket.set_encoding 'UTF-8' if @socket.respond_to?(:set_encoding)
       write Frames::Connect.new(options)
       frame = read Frames::Connected, @select_timeout
       log_frame frame if logger

data/lib/klomp/frames.rb

@@ -5,9 +5,14 @@ class Klomp
     class Frame
       def name; @name ||= self.class.name.split('::').last.upcase; end
 
-      def headers;         @headers ||= {};       end
-      def [](key);          headers[key];         end
-      def []=(key, value);  headers[key] = value; end
+      def new_headers
+        # Dependency injection point for tests.
+        {}
+      end
+
+      def headers;          @headers ||= new_headers; end
+      def [](key);          headers[key];             end
+      def []=(key, value);  headers[key] = value;     end
 
       def body; @body ||= ""; end
       def body=(b); @body = b; end
@@ -44,7 +49,7 @@ class Klomp
 
       def parse_headers(data)
         frame = nil
-        {}.tap do |headers|
+        new_headers.tap do |headers|
           data.lines.each do |line|
             next if line == "\n"
             unless frame

data/spec/acceptance/acceptance_spec.rb

@@ -16,7 +16,7 @@ describe "Klomp acceptance", :acceptance => true do
 
   context "publish" do
 
-    When { klomp.publish "/queue/greeting", "hello" }
+    When { klomp.publish("/queue/greeting", "hello") }
 
     Then do
       vhosts = apollo_api_get_json "/broker/virtual-hosts.json"

data/spec/klomp/connection_spec.rb

@@ -4,18 +4,24 @@ describe Klomp::Connection do
 
   Given(:data)       { frame(:connected) }
   Given(:server)     { "127.0.0.1:61613" }
-  Given(:options)    { { "login" => "admin", "passcode" => "password", "logger" => logger } }
-  Given(:socket)     { double(TCPSocket, gets:data, write:nil, set_encoding:nil, close:nil) }
-  Given(:logger)     { double("Logger", error:nil, warn:nil, info:nil, debug:nil).as_null_object }
-  Given(:subscriber) { double "subscriber", call:nil }
+  Given(:socket)     { double(TCPSocket, :gets => data, :write => nil, :set_encoding => nil, :close => nil) }
+  Given(:logger)     { double("Logger", :error => nil, :warn => nil, :info => nil, :debug => nil).as_null_object }
+  Given(:subscriber) { double "subscriber", :call => nil }
   Given(:thread)     { double Thread }
-  Given(:sentinel)   { double Klomp::Sentinel, alive?:true }
+  Given(:sentinel)   { double Klomp::Sentinel, :alive? => true }
+  Given(:options)    {
+    hsh = ActiveSupport::OrderedHash.new
+    hsh['login']    = 'admin'
+    hsh['passcode'] = 'password'
+    hsh['logger']   = logger
+    hsh
+  }
 
   Given do
     IO.stub!(:select).and_return([[socket], [socket]])
     TCPSocket.stub!(:new).and_return socket
     Thread.stub!(:new).and_return {|*args,&blk| thread.stub!(:block => blk); thread }
-    Klomp::Sentinel.stub!(new: sentinel)
+    Klomp::Sentinel.stub!(:new => sentinel)
   end
 
   context "new" do
@@ -78,7 +84,7 @@ describe Klomp::Connection do
 
     context "logs when logger level is debug" do
 
-      Given(:logger) { double("Logger").as_null_object.tap {|l| l.stub!(debug?: true) } }
+      Given(:logger) { double("Logger").as_null_object.tap {|l| l.stub!(:debug? => true) } }
 
       Then { logger.should have_received(:debug) }
 
@@ -100,7 +106,7 @@ describe Klomp::Connection do
 
       context "and logs when logger level is debug" do
 
-        Given(:logger) { double("Logger").as_null_object.tap {|l| l.stub!(debug?: true) } }
+        Given(:logger) { double("Logger").as_null_object.tap {|l| l.stub!(:debug? => true) } }
 
         Then { logger.should have_received(:debug) }
 
@@ -112,7 +118,7 @@ describe Klomp::Connection do
 
       When do
         connection.subscribe "/queue/greeting", subscriber
-        connection.subscribe "/queue/greeting", double("another subscriber that replaces the first", call:nil)
+        connection.subscribe "/queue/greeting", double("another subscriber that replaces the first", :call => nil)
       end
 
       Then { socket.should have_received(:write).with(frame(:subscribe)).once }
@@ -225,7 +231,7 @@ describe Klomp::Connection do
 
   context "disconnect" do
 
-    Given!(:connection) { Klomp::Connection.new server, options }
+    Given!(:connection) { Klomp::Connection.new server, options}
 
     When(:result) { connection.disconnect }
 
@@ -293,7 +299,7 @@ describe Klomp::Connection do
     Given!(:connection) { Klomp::Connection.new server, options }
 
     Given do
-      thread.stub!(:raise).and_return {|e| raise e }
+      thread.stub!(:raise).and_return {|e, _| raise e }
       socket.stub!(:gets).and_raise SystemCallError.new("some socket error")
     end
 

data/spec/klomp/frames_spec.rb

@@ -22,7 +22,7 @@ describe Klomp::Frames do
 
   context "stringifies all header keys and values" do
 
-    Given(:headers) { { timeout:42 } }
+    Given(:headers) { { :timeout => 42 } }
 
     When(:send_frame) { Klomp::Frames::Send.new("/queue/q", "", headers) }
 

data/spec/klomp/sentinel_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 describe Klomp::Sentinel do
 
-  Given(:connection) { double "Connection", connected?:false, reconnect:nil }
+  Given(:connection) { double "Connection", :connected? => false, :reconnect => nil }
   Given(:sentinel) { Klomp::Sentinel.new connection }
   Given(:thread) { double Thread }
   Given do
@@ -11,7 +11,7 @@ describe Klomp::Sentinel do
 
   context "does nothing if the connection is already connected" do
 
-    Given { connection.stub!(connected?: true) }
+    Given { connection.stub!(:connected? => true) }
 
     When { sentinel }
 

data/spec/klomp_spec.rb

@@ -4,7 +4,7 @@ describe Klomp do
 
   Given(:servers) { ["127.0.0.1:61613", "127.0.0.1:67673"] }
   Given(:connections) { Hash[*servers.map {|s| [s, double("connection #{s}")] }.flatten] }
-  Given { Klomp::Connection.stub!(:new).and_return {|s| connections[s] } }
+  Given { Klomp::Connection.stub!(:new).and_return {|s, _| connections[s] } }
   Given(:klomp) { Klomp.new servers }
 
   context "new" do
@@ -113,7 +113,7 @@ describe Klomp do
 
     context "calls subscribe on all of the servers" do
 
-      Given { connections.values.each {|conn| conn.stub!(subscribe: 42) } }
+      Given { connections.values.each {|conn| conn.stub!(:subscribe => 42) } }
 
       When(:result) { klomp.subscribe("/queue/greeting") { true } }
 
@@ -143,7 +143,7 @@ describe Klomp do
 
     context "calls unsubscribe on all the servers" do
 
-      Given { connections.values.each {|conn| conn.stub!(unsubscribe: 42) } }
+      Given { connections.values.each {|conn| conn.stub!(:unsubscribe => 42) } }
 
       When(:result) { klomp.unsubscribe("/queue/greeting") }
 
@@ -197,7 +197,7 @@ describe Klomp do
 
     context "disconnects all the servers" do
 
-      Given { connections.values.each {|conn| conn.stub!(disconnect: 42) } }
+      Given { connections.values.each {|conn| conn.stub!(:disconnect => 42) } }
 
       When(:result) { klomp.disconnect }
 

data/spec/spec_helper.rb

@@ -1,7 +1,8 @@
-require 'simplecov'
+require 'simplecov' if RUBY_VERSION >= '1.9'
 require 'klomp'
 require 'rspec'
 require 'rspec-given'
+require 'active_support'
 
 Dir[File.expand_path('../support/', __FILE__) + '/*.rb'].each {|f| require f }
 
@@ -32,5 +33,11 @@ RSpec.configure do |config|
     config.filter_run_excluding :performance
   end
 
+  config.before :each do
+    Klomp::Frames::Frame.any_instance.stub(:new_headers) do
+      ActiveSupport::OrderedHash.new
+    end
+  end
+
   config.include Frames
 end

data/spec/support/have_received.rb

@@ -54,7 +54,7 @@ module RSpec::Matchers::HaveReceived
     def initialize(message, expected_from)
       @message, @mock = message, RSpec::Mocks::Mock.new
       @mock.stub!(message)
-      @expectation = @mock.should_receive(message, expected_from: expected_from)
+      @expectation = @mock.should_receive(message, expected_from => expected_from)
     end
 
     def matches?(actual)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: klomp
 version: !ruby/object:Gem::Version
-  version: 1.0.6
+  version: 1.0.7
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-19 00:00:00.000000000 Z
+date: 2013-02-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rdoc
@@ -171,6 +171,22 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: 1.7.0
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 2.3.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 2.3.0
 - !ruby/object:Gem::Dependency
   name: hoe
   requirement: !ruby/object:Gem::Requirement
@@ -208,6 +224,7 @@ extensions: []
 extra_rdoc_files:
 - Manifest.txt
 files:
+- .gemtest
 - .rspec
 - .simplecov
 - ChangeLog.md
@@ -241,7 +258,6 @@ files:
 - spec/spec_helper.rb
 - spec/support/have_received.rb
 - spec/support/shared_contexts.rb
-- .gemtest
 homepage: http://github.com/livingsocial/klomp
 licenses: []
 post_install_message: 
@@ -258,7 +274,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 30151873419123398
+      hash: 2933496602827248730
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: