action_subscriber 1.0.4-java → 1.0.5-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b4d9b95b805582ecb7730fabdfa3d99adac708f2
-  data.tar.gz: 2a016a851347d77f3c2ebdbf83c6860455e0599b
+  metadata.gz: 1f4c2c6846b39cea78859c8fbaa677c35df08c84
+  data.tar.gz: 41c186e3e7465d4a7feb1252ae7ca983948bb630
 SHA512:
-  metadata.gz: 98a8fbb5c315e381cf079a3e909991e1be22f2ef644e2204727356c85d509a01faf50496e58281d0b6f157e2eda89f3d148f549427372e62f9baf830dea1cba6
-  data.tar.gz: 3aec2d0022224cb122acc2fd43d53513aa14bf491bc15076493aa4b77c80a72744e4cb345d87f5996fcae0fc0666a3abbfec7243af804520e2d483afa94537a2
+  metadata.gz: 4c3c4091f8bc4a243cfb5cb0639d9fcaf4c6aba62e99908e1a812a03f233aea2c09a5f66a962dfd59166326a128d40582b67a4519b297f2e00765734ce0ca32c
+  data.tar.gz: b7497d9da032a879f1506ac122d4ecc342e79020dd4886ee60464b35ee8c009e6f897aa9872c92523ef361bfb3e4fa600f20eb7151478f94b026e3160b671e9b

data/.travis.yml

@@ -1,12 +1,17 @@
 language: ruby
 rvm:
- - 1.9
  - 2.0
  - 2.1
  - 2.2
  - jruby
+ - jruby-1.7
  - jruby-head
+ - rbx-2
 services:
  - rabbitmq
 sudo: false
 cache: bundler
+matrix:
+  allow_failures:
+    - rvm: jruby-head
+    - rvm: rbx-2

data/README.md

@@ -1,6 +1,7 @@
 [![Build Status](https://travis-ci.org/moneydesktop/action_subscriber.svg?branch=master)](https://travis-ci.org/moneydesktop/action_subscriber)
 [![Code Climate](https://codeclimate.com/github/moneydesktop/action_subscriber/badges/gpa.svg)](https://codeclimate.com/github/moneydesktop/action_subscriber)
 [![Dependency Status](https://gemnasium.com/moneydesktop/action_subscriber.svg)](https://gemnasium.com/moneydesktop/action_subscriber)
+[![Join the chat at https://gitter.im/moneydesktop/action_subscriber](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/moneydesktop/action_subscriber?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
 
 ActionSubscriber
 =================

data/action_subscriber.gemspec

@@ -4,19 +4,20 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'action_subscriber/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "action_subscriber"
-  spec.version       = ActionSubscriber::VERSION
-  spec.authors       = ["Brian Stien","Adam Hutchison","Brandon Dewitt","Devin Christensen","Michael Ries"]
-  spec.email         = ["brianastien@gmail.com","liveh2o@gmail.com","brandonsdewitt@gmail.com","quixoten@gmail.com","michael@riesd.com"]
-  spec.description   = %q{ActionSubscriber is a DSL that allows a rails app to consume messages from a RabbitMQ broker.}
-  spec.summary       = %q{ActionSubscriber is a DSL that allows a rails app to consume messages from a RabbitMQ broker.}
-  spec.homepage      = "https://github.com/moneydesktop/action_subscriber"
-  spec.license       = "MIT"
+  spec.name                  = "action_subscriber"
+  spec.version               = ActionSubscriber::VERSION
+  spec.authors               = ["Brian Stien","Adam Hutchison","Brandon Dewitt","Devin Christensen","Michael Ries"]
+  spec.email                 = ["brianastien@gmail.com","liveh2o@gmail.com","brandonsdewitt@gmail.com","quixoten@gmail.com","michael@riesd.com"]
+  spec.description           = %q{ActionSubscriber is a DSL that allows a rails app to consume messages from a RabbitMQ broker.}
+  spec.summary               = %q{ActionSubscriber is a DSL that allows a rails app to consume messages from a RabbitMQ broker.}
+  spec.homepage              = "https://github.com/moneydesktop/action_subscriber"
+  spec.license               = "MIT"
 
-  spec.files         = `git ls-files`.split($/)
-  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
-  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
-  spec.require_paths = ["lib"]
+  spec.files                 = `git ls-files`.split($/)
+  spec.executables           = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
+  spec.test_files            = spec.files.grep(%r{^(test|spec|features)/})
+  spec.require_paths         = ["lib"]
+  spec.required_ruby_version = Gem::Requirement.new(">= 2.0")
 
   spec.add_dependency 'activesupport', '>= 3.2'
 

data/lib/action_subscriber/base.rb

@@ -32,7 +32,7 @@ module ActionSubscriber
     #
 
     def self.connection
-      ::ActionSubscriber::RabbitConnection.connection
+      ::ActionSubscriber::RabbitConnection.subscriber_connection
     end
 
     # Inherited callback, save a reference to our descendents

data/lib/action_subscriber/default_routing.rb

@@ -8,7 +8,7 @@ module ActionSubscriber
       queue_name = queue_name_for_method(method_name)
       routing_key_name = routing_key_name_for_method(method_name)
 
-      channel = ::ActionSubscriber::RabbitConnection.connection.create_channel
+      channel = connection.create_channel
       exchange = channel.topic(exchange_name)
       queue = channel.queue(queue_name)
       queue.bind(exchange, :routing_key => routing_key_name)

data/lib/action_subscriber/rabbit_connection.rb

@@ -2,29 +2,63 @@ require 'thread'
 
 module ActionSubscriber
   module RabbitConnection
-    CONNECTION_MUTEX = ::Mutex.new
-
-    def self.connect!
-      CONNECTION_MUTEX.synchronize do
-        return @connection if @connection
-        if ::RUBY_PLATFORM == "java"
-          @connection = ::MarchHare.connect(connection_options)
-        else
-          @connection = ::Bunny.new(connection_options)
-          @connection.start
+    SUBSCRIBER_CONNECTION_MUTEX = ::Mutex.new
+    PUBLISHER_CONNECTION_MUTEX = ::Mutex.new
+
+    def self.publisher_connected?
+      publisher_connection.try(:connected?)
+    end
+
+    def self.publisher_connection
+      SUBSCRIBER_CONNECTION_MUTEX.synchronize do
+        return @publisher_connection if @publisher_connection
+        @publisher_connection = create_connection
+      end
+    end
+
+    def self.publisher_disconnect!
+      SUBSCRIBER_CONNECTION_MUTEX.synchronize do
+        if @publisher_connection && @publisher_connection.connected?
+          @publisher_connection.close
         end
-        @connection
+
+        @publisher_connection = nil
       end
     end
 
-    def self.connected?
-      connection && connection.connected?
+    def self.subscriber_connected?
+      subscriber_connection.try(:connected?)
     end
 
-    def self.connection
-      connect!
+    def self.subscriber_connection
+      SUBSCRIBER_CONNECTION_MUTEX.synchronize do
+        return @subscriber_connection if @subscriber_connection
+        @subscriber_connection = create_connection
+      end
     end
 
+    def self.subscriber_disconnect!
+      SUBSCRIBER_CONNECTION_MUTEX.synchronize do
+        if @subscriber_connection && @subscriber_connection.connected?
+          @subscriber_connection.close
+        end
+
+        @subscriber_connection = nil
+      end
+    end
+
+    # Private API
+    def self.create_connection
+      if ::RUBY_PLATFORM == "java"
+        connection = ::MarchHare.connect(connection_options)
+      else
+        connection = ::Bunny.new(connection_options)
+        connection.start
+        connection
+      end
+    end
+    private_class_method :create_connection
+
     def self.connection_options
       {
         :heartbeat                     => ::ActionSubscriber.configuration.heartbeat,
@@ -36,15 +70,6 @@ module ActionSubscriber
         :recover_from_connection_close => true,
       }
     end
-
-    def self.disconnect!
-      CONNECTION_MUTEX.synchronize do
-        if @connection && @connection.connected?
-          @connection.close
-        end
-
-        @connection = nil
-      end
-    end
+    private_class_method :connection_options
   end
 end

data/lib/action_subscriber/version.rb

@@ -1,3 +1,3 @@
 module ActionSubscriber
-  VERSION = "1.0.4"
+  VERSION = "1.0.5"
 end

data/spec/integration/around_filters_spec.rb

@@ -31,8 +31,9 @@ describe "subscriber filters", :integration => true do
     channel = connection.create_channel
     exchange = channel.topic("events")
     exchange.publish("hEY Guyz!", :routing_key => "insta.first")
-    sleep 0.1
 
-    expect($messages).to eq [:whisper_before, :yell_before, "hEY Guyz!", :yell_after, :whisper_after]
+    verify_expectation_within(1.0) do
+      expect($messages).to eq [:whisper_before, :yell_before, "hEY Guyz!", :yell_after, :whisper_after]
+    end
   end
 end

data/spec/integration/at_least_once_spec.rb

@@ -16,8 +16,9 @@ describe "at_least_once! mode", :integration => true do
     channel = connection.create_channel
     exchange = channel.topic("events")
     exchange.publish("GrumpFace", :routing_key => "gorby_puff.grumpy")
-    sleep 0.1
 
-    expect($messages).to eq Set.new(["GrumpFace::0","GrumpFace::1","GrumpFace::2"])
+    verify_expectation_within(1.0) do
+      expect($messages).to eq Set.new(["GrumpFace::0","GrumpFace::1","GrumpFace::2"])
+    end
   end
 end

data/spec/integration/at_most_once_spec.rb

@@ -16,8 +16,9 @@ describe "at_most_once! mode", :integration => true do
     channel = connection.create_channel
     exchange = channel.topic("events")
     exchange.publish("All Pokemon have been caught", :routing_key => "pokemon.caught_em_all")
-    sleep 0.1
 
-    expect($messages.size).to eq 1
+    verify_expectation_within(1.0) do
+      expect($messages.size).to eq 1
+    end
   end
 end

data/spec/integration/automatic_reconnect_spec.rb

@@ -16,15 +16,22 @@ describe "Automatically reconnect on connection failure", :integration => true,
     channel = connection.create_channel
     exchange = channel.topic("events")
     exchange.publish("First", :routing_key => "gus.spoke")
-    sleep 0.1
+    verify_expectation_within(5.0) do
+      expect($messages).to eq(Set.new(["First"]))
+    end
 
     close_all_connections!
-    sleep_until_reconnected
+    sleep 5.0
+    verify_expectation_within(5.0) do
+      expect(connection).to be_open
+    end
 
+    channel = connection.create_channel
+    exchange = channel.topic("events")
     exchange.publish("Second", :routing_key => "gus.spoke")
-    sleep 0.1
-
-    expect($messages).to eq(Set.new(["First", "Second"]))
+    verify_expectation_within(5.0) do
+      expect($messages).to eq(Set.new(["First", "Second"]))
+    end
   end
 
   def close_all_connections!
@@ -32,12 +39,4 @@ describe "Automatically reconnect on connection failure", :integration => true,
       http_client.close_connection(conn_info.name)
     end
   end
-
-  def sleep_until_reconnected
-    100.times do
-      sleep 0.1
-      break if connection.open?
-    end
-    sleep 0.2
-  end
 end

data/spec/integration/basic_subscriber_spec.rb

@@ -25,11 +25,11 @@ describe "A Basic Subscriber", :integration => true do
       exchange = channel.topic("events")
       exchange.publish("Ohai Booked", :routing_key => "greg.basic_push.booked")
       exchange.publish("Ohai Cancelled", :routing_key => "basic.cancelled")
-      sleep 0.1
 
-      ::ActionSubscriber.auto_pop!
-
-      expect($messages).to eq(Set.new(["Ohai Booked", "Ohai Cancelled"]))
+      verify_expectation_within(2.0) do
+        ::ActionSubscriber.auto_pop!
+        expect($messages).to eq(Set.new(["Ohai Booked", "Ohai Cancelled"]))
+      end
     end
   end
 
@@ -40,9 +40,10 @@ describe "A Basic Subscriber", :integration => true do
       exchange = channel.topic("events")
       exchange.publish("Ohai Booked", :routing_key => "greg.basic_push.booked")
       exchange.publish("Ohai Cancelled", :routing_key => "basic.cancelled")
-      sleep 0.1
 
-      expect($messages).to eq(Set.new(["Ohai Booked", "Ohai Cancelled"]))
+      verify_expectation_within(2.0) do
+        expect($messages).to eq(Set.new(["Ohai Booked", "Ohai Cancelled"]))
+      end
     end
   end
 end

data/spec/integration/decoding_payloads_spec.rb

@@ -18,12 +18,13 @@ describe "Payload Decoding", :integration => true do
     channel = connection.create_channel
     exchange = channel.topic("events")
     exchange.publish(json_string, :routing_key => "twitter.tweet", :content_type => "application/json")
-    sleep 0.1
 
-    expect($messages).to eq Set.new([{
-      :decoded => JSON.parse(json_string),
-      :raw => json_string,
-    }])
+    verify_expectation_within(2.0) do
+      expect($messages).to eq Set.new([{
+        :decoded => JSON.parse(json_string),
+        :raw => json_string,
+      }])
+    end
   end
 
   context "Custom Decoder" do
@@ -37,12 +38,13 @@ describe "Payload Decoding", :integration => true do
       channel = connection.create_channel
       exchange = channel.topic("events")
       exchange.publish(json_string, :routing_key => "twitter.tweet", :content_type => content_type)
-      sleep 0.1
 
-      expect($messages).to eq Set.new([{
-        :decoded => :foo,
-        :raw => json_string,
-      }])
+      verify_expectation_within(2.0) do
+        expect($messages).to eq Set.new([{
+          :decoded => :foo,
+          :raw => json_string,
+        }])
+      end
     end
   end
 end

data/spec/integration/manual_acknowledgement_spec.rb

@@ -20,8 +20,9 @@ describe "Manual Message Acknowledgment", :integration => true do
     channel = connection.create_channel
     exchange = channel.topic("events")
     exchange.publish("BACON!", :routing_key => "bacon.served")
-    sleep 0.1
 
-    expect($messages).to eq(Set.new(["BACON!::0", "BACON!::1", "BACON!::2"]))
+    verify_expectation_within(2.0) do
+      expect($messages).to eq(Set.new(["BACON!::0", "BACON!::1", "BACON!::2"]))
+    end
   end
 end

data/spec/spec_helper.rb

@@ -20,13 +20,27 @@ RSpec.configure do |config|
 
   config.before(:each, :integration => true) do
     $messages = Set.new
-    ::ActionSubscriber::RabbitConnection.connect!
+    ::ActionSubscriber::RabbitConnection.subscriber_connection
     ::ActionSubscriber.setup_queues!
   end
   config.after(:each, :integration => true) do
-    ::ActionSubscriber::RabbitConnection.disconnect!
+    ::ActionSubscriber::RabbitConnection.subscriber_disconnect!
     ::ActionSubscriber::Base.inherited_classes.each do |klass|
       klass.instance_variable_set("@_queues", nil)
     end
   end
 end
+
+def verify_expectation_within(number_of_seconds, check_every = 0.02)
+  waiting_since = ::Time.now
+  begin
+    sleep check_every
+    yield
+  rescue RSpec::Expectations::ExpectationNotMetError => e
+    if ::Time.now - waiting_since > number_of_seconds
+      raise e
+    else
+      retry
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: action_subscriber
 version: !ruby/object:Gem::Version
-  version: 1.0.4
+  version: 1.0.5
 platform: java
 authors:
 - Brian Stien
@@ -12,7 +12,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2015-02-10 00:00:00.000000000 Z
+date: 2015-09-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -232,7 +232,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - '>='
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - '>='
@@ -240,7 +240,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project:
-rubygems_version: 2.1.9
+rubygems_version: 2.4.8
 signing_key:
 specification_version: 4
 summary: ActionSubscriber is a DSL that allows a rails app to consume messages from a RabbitMQ broker.