action_subscriber 2.0.0.pre0-java → 2.0.1-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 704909d89faa6c4cb11c8b6627b68338a95f42f6
-  data.tar.gz: e1b21845d9959e21cf6945ab3520d7c5ed44f7c4
+  metadata.gz: 939204017ecb3b17925a0ceca1442798b98ddcdc
+  data.tar.gz: 1780324c82d3dac0a2f0d36d00bfaa724296782f
 SHA512:
-  metadata.gz: 9520afb5c807074447480240201f81e38784a30da8466266acb22dd434e7b69998674f1aea37f94691702d6c61f0e86c54477c8ca853da1f39328564aa56d11b
-  data.tar.gz: 3380283dd6b6325d8bcdeda10f7f9314a2a5ba73dd6be8c3b25c2a63781dc4c13a110d41eb6a19aef03ac59974078b55ddf1f73de9e9f317d1692cd0f33b67f3
+  metadata.gz: 258b3dcf397203650f2fe7c5b6590ddc3cf5ce0c9e81d1352d3a50505fc58ffa9b0dbcb5ec16b929d3ae9bb41580e6c15ba993da71194873eb4104ab58f44ba1
+  data.tar.gz: 9235bf4d8f54c8616ff77f958da778d04cd4a3f6271968a74f35245ad9fbf9a52c9b7cf1cf642e04dfd71a5e3a7ea8d586d7986f06a3fd6f6e1fc94ac2ab8e86

data/lib/action_subscriber/babou.rb

@@ -96,8 +96,7 @@ module ActionSubscriber
       wait_loops = 0
       ::ActionSubscriber::Babou.stop_receving_messages!
 
-      # Going to wait until the thread pool drains or we wait for 1000 seconds
-      while ::ActionSubscriber::Threadpool.pool.busy_size > 0 && wait_loops < 1000
+      while ::ActionSubscriber::Threadpool.pool.busy_size > 0 && wait_loops < ::ActionSubscriber.configuration.seconds_to_wait_for_graceful_shutdown
         puts "waiting for threadpool to empty (#{::ActionSubscriber::Threadpool.pool.busy_size})"
         Thread.pass
         wait_loops = wait_loops + 1

data/lib/action_subscriber/bunny/subscriber.rb

@@ -21,6 +21,7 @@ module ActionSubscriber
             next unless encoded_payload # empty queue
             ::ActiveSupport::Notifications.instrument "popped_event.action_subscriber", :payload_size => encoded_payload.bytesize, :queue => queue.name
             properties = {
+              :action => route.action,
               :channel => queue.channel,
               :content_type => properties[:content_type],
               :delivery_tag => delivery_info.delivery_tag,
@@ -44,6 +45,7 @@ module ActionSubscriber
           consumer.on_delivery do |delivery_info, properties, encoded_payload|
             ::ActiveSupport::Notifications.instrument "received_event.action_subscriber", :payload_size => encoded_payload.bytesize, :queue => queue.name
             properties = {
+              :action => route.action,
               :channel => queue.channel,
               :content_type => properties.content_type,
               :delivery_tag => delivery_info.delivery_tag,

data/lib/action_subscriber/configuration.rb

@@ -12,6 +12,7 @@ module ActionSubscriber
                   :port,
                   :prefetch,
                   :publisher_confirms,
+                  :seconds_to_wait_for_graceful_shutdown,
                   :threadpool_size,
                   :timeout,
                   :times_to_pop
@@ -25,8 +26,9 @@ module ActionSubscriber
       :mode => 'subscribe',
       :pop_interval => 100, # in milliseconds
       :port => 5672,
-      :prefetch => 200,
+      :prefetch => 5,
       :publisher_confirms => false,
+      :seconds_to_wait_for_graceful_shutdown => 30,
       :threadpool_size => 8,
       :timeout => 1,
       :times_to_pop => 8

data/lib/action_subscriber/march_hare/subscriber.rb

@@ -17,6 +17,7 @@ module ActionSubscriber
             next unless encoded_payload
             ::ActiveSupport::Notifications.instrument "popped_event.action_subscriber", :payload_size => encoded_payload.bytesize, :queue => queue.name
             properties = {
+              :action => route.action,
               :channel => queue.channel,
               :content_type => metadata.content_type,
               :delivery_tag => metadata.delivery_tag,
@@ -41,6 +42,7 @@ module ActionSubscriber
           consumer = queue.subscribe(route.queue_subscription_options) do |metadata, encoded_payload|
             ::ActiveSupport::Notifications.instrument "received_event.action_subscriber", :payload_size => encoded_payload.bytesize, :queue => queue.name
             properties = {
+              :action => route.action,
               :channel => queue.channel,
               :content_type => metadata.content_type,
               :delivery_tag => metadata.delivery_tag,

data/lib/action_subscriber/middleware/env.rb

@@ -5,7 +5,8 @@ module ActionSubscriber
     class Env
       attr_accessor :payload
 
-      attr_reader :content_type,
+      attr_reader :action,
+                  :content_type,
                   :encoded_payload,
                   :exchange,
                   :headers,
@@ -26,6 +27,7 @@ module ActionSubscriber
       #         :message_id => String
       #         :routing_key => String
       def initialize(subscriber, encoded_payload, properties)
+        @action = properties.fetch(:action)
         @channel = properties.fetch(:channel)
         @content_type = properties.fetch(:content_type)
         @delivery_tag = properties.fetch(:delivery_tag)
@@ -43,13 +45,6 @@ module ActionSubscriber
         @channel.ack(@delivery_tag, acknowledge_multiple_messages)
       end
 
-      # Return the last element of the routing key to indicate which action
-      # to route the payload to
-      #
-      def action
-        routing_key.split('.').last.to_s
-      end
-
       def reject
         requeue_message = true
         @channel.reject(@delivery_tag, requeue_message)

data/lib/action_subscriber/rspec.rb

@@ -13,6 +13,7 @@ module ActionSubscriber
     end
 
     PROPERTIES_DEFAULTS = {
+      :action => :created,
       :channel => FakeChannel.new,
       :content_type => "text/plain",
       :delivery_tag => "XYZ",
@@ -71,6 +72,7 @@ end
     let(:app) { Proc.new { |inner_env| inner_env } }
     let(:env) { ActionSubscriber::Middleware::Env.new(UserSubscriber, 'encoded payload', message_properties) }
     let(:message_properties) {{
+      :action => :created,
       :channel => ::ActionSubscriber::RSpec::FakeChannel.new,
       :content_type => "text/plain",
       :delivery_tag => "XYZ",

data/lib/action_subscriber/version.rb

@@ -1,3 +1,3 @@
 module ActionSubscriber
-  VERSION = "2.0.0.pre0"
+  VERSION = "2.0.1"
 end

data/spec/integration/custom_actions_spec.rb

@@ -0,0 +1,24 @@
+class CustomActionSubscriber < ActionSubscriber::Base
+  def wat
+    $messages << payload
+  end
+end
+
+describe "A subscriber with a custom action", :integration => true do
+  let(:draw_routes) do
+    ::ActionSubscriber.draw_routes do
+      route ::CustomActionSubscriber, :wat,
+        :queue => "unrelated_to_the_action",
+        :routing_key => "*.javascript_framework"
+    end
+  end
+
+  it "routes the message to the selected action" do
+    ::ActionSubscriber.auto_subscribe!
+    ::ActionSubscriber::Publisher.publish("react.javascript_framework", "Another?!?!", "events")
+
+    verify_expectation_within(2.0) do
+      expect($messages).to eq(Set.new(["Another?!?!"]))
+    end
+  end
+end

data/spec/lib/action_subscriber/configuration_spec.rb

@@ -7,7 +7,8 @@ describe ::ActionSubscriber::Configuration do
     specify { expect(subject.mode).to eq('subscribe') }
     specify { expect(subject.pop_interval).to eq(100) }
     specify { expect(subject.port).to eq(5672) }
-    specify { expect(subject.prefetch).to eq(200) }
+    specify { expect(subject.prefetch).to eq(5) }
+    specify { expect(subject.seconds_to_wait_for_graceful_shutdown).to eq(30) }
     specify { expect(subject.threadpool_size).to eq(8) }
     specify { expect(subject.timeout).to eq(1) }
     specify { expect(subject.times_to_pop).to eq(8) }

data/spec/lib/action_subscriber/middleware/env_spec.rb

@@ -2,6 +2,7 @@ describe ActionSubscriber::Middleware::Env do
   let(:channel) { double("channel") }
   let(:encoded_payload) { 'encoded_payload' }
   let(:properties){ {
+    :action => :created,
     :channel => channel,
     :content_type => "application/json",
     :delivery_tag => "XYZ",
@@ -16,7 +17,7 @@ describe ActionSubscriber::Middleware::Env do
 
   subject { described_class.new(subscriber, encoded_payload, properties) }
 
-  specify { expect(subject.action).to eq("created") }
+  specify { expect(subject.action).to eq(:created) }
   specify { expect(subject.content_type).to eq(properties[:content_type]) }
   specify { expect(subject.exchange).to eq(properties[:exchange]) }
   specify { expect(subject.headers).to eq(properties[:headers]) }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: action_subscriber
 version: !ruby/object:Gem::Version
-  version: 2.0.0.pre0
+  version: 2.0.1
 platform: java
 authors:
 - Brian Stien
@@ -12,7 +12,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2016-01-05 00:00:00.000000000 Z
+date: 2016-01-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -228,6 +228,7 @@ files:
 - spec/integration/at_most_once_spec.rb
 - spec/integration/automatic_reconnect_spec.rb
 - spec/integration/basic_subscriber_spec.rb
+- spec/integration/custom_actions_spec.rb
 - spec/integration/custom_headers_spec.rb
 - spec/integration/decoding_payloads_spec.rb
 - spec/integration/inferred_routes_spec.rb
@@ -264,9 +265,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project:
 rubygems_version: 2.5.0
@@ -279,6 +280,7 @@ test_files:
 - spec/integration/at_most_once_spec.rb
 - spec/integration/automatic_reconnect_spec.rb
 - spec/integration/basic_subscriber_spec.rb
+- spec/integration/custom_actions_spec.rb
 - spec/integration/custom_headers_spec.rb
 - spec/integration/decoding_payloads_spec.rb
 - spec/integration/inferred_routes_spec.rb