bunny-publisher 0.1.0 → 0.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3285bb8050efb8421bb572040231fa85c1b12ee7bcd3cda02af8649101559ce6
-  data.tar.gz: f00f996623c6d099cf405f9555b5cbf437b67ad65aeb34ea2722bc3235492388
+  metadata.gz: 9578d81f2d9e983145b25db99694b6600ceb72260af7400d53d222a4f0723d09
+  data.tar.gz: 88a145f8ea8e6859d383d9983093d467140f3fe3104a164aa0526125da1defc0
 SHA512:
-  metadata.gz: a0148e4c6275e4983b9853a8f12d1e38d1f9eb97c57ed4f4d0adbbb8a071638fa0998544d908ead27841d1b513bc11e284ddd83efe37f9d0d3f1302476f900c2
-  data.tar.gz: 0de5b4d2f9e9cc53c5da44582548b387b7939e8a23d45014bfe7a6868ec98c580c434bc65792790fdfe932ef671667acff2ec0eb2a5c7610e3d2eff014045c5f
+  metadata.gz: b256c60e4c18f7c033867a18077273246188aabc01091a9cd27c3927330daa6f461ed35c1bef7890f1817937884c920fd25807530b0ddbf03a436c9dacaf8727
+  data.tar.gz: f669cd0022bc139539232aebc467bdba3a0c373fc121ce6a91d2304731b23002e1d51401e4c418435cb753e9ffeaa7f926f5ab52634c84edcc20b3c40c7b49f7

data/CHANGELOG.md

@@ -1 +1,46 @@
-## Original Release: 0.1.0
+# Changelog
+
+All notable changes to this project will be documented in this file.
+
+The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
+
+## [Unreleased](https://github.com/veeqo/bunny-publisher/compare/v0.1.5...HEAD)
+
+
+## [0.1.5](https://github.com/veeqo/bunny-publisher/compare/v0.1.4...v0.1.5) - 2020-11-04
+
+### Fixed
+- [#7](https://github.com/veeqo/bunny-publisher/pull/7) Enforce single channel for threaded apps
+
+
+## [0.1.4](https://github.com/veeqo/bunny-publisher/compare/v0.1.3...v0.1.4) - 2020-11-03
+
+### Changed
+- [#5](https://github.com/veeqo/bunny-publisher/pull/5) Test against ruby 2.7.2
+
+### Fixed
+- [#6](https://github.com/veeqo/bunny-publisher/pull/6) Do not share channels between threads
+
+
+## [0.1.3](https://github.com/veeqo/bunny-publisher/compare/v0.1.2...v0.1.3) - 2020-09-18
+
+### Changed
+- [#2](https://github.com/veeqo/bunny-publisher/pull/2) Use appraisal gem to control gem versions of tests matrix
+- [#3](https://github.com/veeqo/bunny-publisher/pull/3) Require ruby 2.5
+
+### Fixed
+- [#4](https://github.com/veeqo/bunny-publisher/pull/4) Fix deprecation warnings of ruby 2.7
+
+
+## [0.1.2](https://github.com/veeqo/bunny-publisher/compare/v0.1.1...v0.1.2) - 2020-07-30
+
+### Added
+-  [846369a](https://github.com/veeqo/bunny-publisher/commit/846369a76a9ae1c38e0a08844a5c992412eac520) Test mode
+
+
+## [0.1.1](https://github.com/veeqo/bunny-publisher/compare/v0.1.0...v0.1.1) - 2020-05-21
+
+### Changed
+- [fec490c](https://github.com/veeqo/bunny-publisher/commit/fec490c91a7db9d053dba9afde0736a4ce2dde71) Imrove unrouted messages processing
+
+## [0.1.0](https://github.com/veeqo/bunny-publisher/releases/tag/v0.1.0) - 2020-05-20

data/README.md

@@ -111,6 +111,52 @@ end
 
 Not implemented yet
 
+## Test mode
+
+BunnyPublisher also has a test mode which replaces exchange and prevents real connection to RabbitMQ
+
+```ruby
+BunnyPublisher.configure do |c|
+  c.test = true # or Rails.env.test?
+
+  # ...
+end
+```
+
+Now publisher has `#messages`, `#flush!` methods:
+
+```ruby
+describe SomeServiceThatPublishesToRabbitMQ, '#call' do
+  subject { described_class.new(some: 'attributes').call }
+
+  before { BunnyPublisher.flush! }
+
+  it 'publishes a message' do
+    expect { subject }.to change(BunnyPublisher.messages).by(1)
+  end
+end
+```
+
+## Testing
+
+Run rabbitmq server (in separate console):
+```
+docker run --rm -p 5672:5672 -p 15672:15672 rabbitmq:3.8-management-alpine
+```
+
+Run tests with:
+
+```sh
+bundle
+rake
+```
+
+Use [wwtd](https://github.com/grosser/wwtd) to run test matrix:
+```sh
+gem install wwtd
+wwtd
+```
+
 ## Contributing
 
 Bug reports and pull requests are welcome on GitHub at https://github.com/veeqo/bunny-publisher.

data/lib/bunny_publisher.rb

@@ -6,6 +6,7 @@ require 'bunny_publisher/callbacks'
 require 'bunny_publisher/errors'
 require 'bunny_publisher/base'
 require 'bunny_publisher/mandatory'
+require 'bunny_publisher/test'
 
 module BunnyPublisher
   class << self
@@ -20,13 +21,28 @@ module BunnyPublisher
     def configure
       require 'ostruct'
 
-      config = OpenStruct.new({})
+      config = OpenStruct.new(mandatory: false, test: false)
 
       yield(config)
 
-      klass = Class.new(Base).tap { |k| k.include(::BunnyPublisher::Mandatory) if config.delete_field(:mandatory) }
+      klass = Class.new(Base) do
+        include ::BunnyPublisher::Mandatory if config.delete_field(:mandatory)
+        include ::BunnyPublisher::Test      if config.delete_field(:test)
+      end
 
-      @publisher = klass.new(config.to_h)
+      @publisher = klass.new(**config.to_h)
+    end
+
+    def method_missing(method_name, *args)
+      if publisher.respond_to?(method_name)
+        publisher.send(method_name, *args)
+      else
+        super
+      end
+    end
+
+    def respond_to_missing?(method_name, *args)
+      publisher.respond_to?(method_name) || super
     end
   end
 end

data/lib/bunny_publisher/base.rb

@@ -25,13 +25,15 @@ module BunnyPublisher
     end
 
     def publish(message, options = {})
-      ensure_connection!
+      @mutex.synchronize do
+        ensure_connection!
 
-      run_callback(:before_publish, message, options)
-      result = run_callback(:around_publish, message, options) { exchange.publish(message, options) }
-      run_callback(:after_publish, message, options)
+        run_callback(:before_publish, message, options)
+        result = run_callback(:around_publish, message, options) { exchange.publish(message, options) }
+        run_callback(:after_publish, message, options)
 
-      result
+        result
+      end
     end
 
     def close
@@ -43,7 +45,7 @@ module BunnyPublisher
     private
 
     def ensure_connection!
-      @mutex.synchronize { connect! unless connected? }
+      connect! unless connected?
     end
 
     def connect!

data/lib/bunny_publisher/mandatory.rb

@@ -16,7 +16,7 @@ module BunnyPublisher
     attr_reader :queue_name, :queue_options
 
     def initialize(republish_connection: nil, queue: nil, queue_options: {}, timeout_at_exit: 5, **options)
-      super(options)
+      super(**options)
 
       @queue_name = queue
       @queue_options = queue_options
@@ -65,7 +65,7 @@ module BunnyPublisher
     end
 
     def ensure_republish_connection!
-      @republish_mutex.synchronize { connect_for_republish! unless connected_for_republish? }
+      connect_for_republish! unless connected_for_republish?
     end
 
     def connected_for_republish?
@@ -91,7 +91,7 @@ module BunnyPublisher
     end
 
     def on_message_return(return_info, properties, message)
-      @unrouted_message = true
+      @unrouted_message_processing = true
 
       ensure_message_is_unrouted!(return_info, properties, message)
 
@@ -103,19 +103,22 @@ module BunnyPublisher
       end
       run_callback(:after_republish, return_info, properties, message)
 
-      @unrouted_message = false
       result
+    ensure
+      @unrouted_message_processing = false
     end
 
     def setup_queue_for_republish(return_info, properties, message)
-      ensure_republish_connection!
+      @republish_mutex.synchronize do
+        ensure_republish_connection!
 
-      queue = declare_republish_queue(return_info, properties, message)
+        queue = declare_republish_queue(return_info, properties, message)
 
-      # default exchange already has bindings with queues
-      declare_republish_queue_binding(queue, return_info, properties, message) unless republish_exchange.name == ''
+        # default exchange already has bindings with queues
+        declare_republish_queue_binding(queue, return_info, properties, message) unless republish_exchange.name == ''
 
-      republish_channel.deregister_queue(queue) # we are not going to work with this queue in this channel
+        republish_channel.deregister_queue(queue) # we are not going to work with this queue in this channel
+      end
     end
 
     def ensure_message_is_unrouted!(return_info, properties, message)
@@ -130,13 +133,13 @@ module BunnyPublisher
     def wait_for_unrouted_messages_processing(timeout:)
       sleep(0.05) # gives exchange some time to receive retuned message
 
-      return unless @unrouted_message
+      return unless @unrouted_message_processing
 
       puts("Waiting up to #{timeout} seconds for unrouted messages handling")
 
-      Timeout.timeout(timeout) { sleep 0.01 while @unrouted_message }
+      Timeout.timeout(timeout) { sleep 0.01 while @unrouted_message_processing }
     rescue Timeout::Error
-      puts('Some unrouted messages are lost on process exit!')
+      warn('Some unrouted messages are lost on process exit!')
     end
   end
 end

data/lib/bunny_publisher/test.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+module BunnyPublisher
+  # Test module prevents real connections to be made and replaces exchange with array-based TestExchange
+  module Test
+    class TestExchange < Array
+      def publish(message, options = {})
+        self << [message, options]
+        true
+      end
+
+      # in case if used with Mandatory module included
+      def on_return(&block); end
+    end
+
+    def exchange
+      @exchange ||= TestExchange.new
+    end
+
+    alias messages exchange
+
+    def flush!
+      exchange.clear
+    end
+
+    alias close flush!
+    alias stop flush!
+
+    private
+
+    def ensure_connection!; end
+  end
+end

data/lib/bunny_publisher/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module BunnyPublisher
-  VERSION = '0.1.0'
+  VERSION = '0.1.5'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bunny-publisher
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.5
 platform: ruby
 authors:
 - Rustam Sharshenov
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-05-20 00:00:00.000000000 Z
+date: 2020-11-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bunny
@@ -24,20 +24,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.10'
+- !ruby/object:Gem::Dependency
+  name: appraisal
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.3.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.3.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
@@ -66,34 +80,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.13'
-- !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.0'
 description: AMQP publisher for RabbitMQ based on Bunny
 email:
 - rustam@sharshenov.com
@@ -101,22 +101,16 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gitignore"
-- ".rspec"
-- ".rubocop.yml"
-- ".travis.yml"
 - CHANGELOG.md
-- Gemfile
 - LICENSE.txt
 - README.md
-- Rakefile
-- bunny-publisher.gemspec
 - lib/bunny/publisher.rb
 - lib/bunny_publisher.rb
 - lib/bunny_publisher/base.rb
 - lib/bunny_publisher/callbacks.rb
 - lib/bunny_publisher/errors.rb
 - lib/bunny_publisher/mandatory.rb
+- lib/bunny_publisher/test.rb
 - lib/bunny_publisher/version.rb
 homepage: https://github.com/veeqo/bunny-publisher
 licenses:
@@ -133,14 +127,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.8
+rubygems_version: 3.0.6
 signing_key: 
 specification_version: 4
 summary: AMQP publisher for RabbitMQ based on Bunny

data/.gitignore

@@ -1,14 +0,0 @@
-/.bundle/
-/.yardoc
-/_yardoc/
-/coverage/
-/doc/
-/pkg/
-/spec/reports/
-/tmp/
-/Gemfile.lock
-/.ruby-gemset
-/.ruby-version
-
-# rspec failure tracking
-.rspec_status

data/.rspec

@@ -1,3 +0,0 @@
---format documentation
---color
---require spec_helper

data/.rubocop.yml

@@ -1,16 +0,0 @@
-Layout/LineLength:
-  Max: 120
-  Exclude:
-    - "spec/**/*"
-
-Lint/AmbiguousBlockAssociation:
-  Exclude:
-    - "spec/**/*"
-
-Metrics/BlockLength:
-  Exclude:
-    - "spec/**/*"
-
-Style/EvalWithLocation:
-  Exclude:
-    - "spec/**/*"

data/.travis.yml

@@ -1,16 +0,0 @@
----
-dist: xenial
-language: ruby
-cache: bundler
-rvm:
-  - 2.5.8
-  - 2.6.6
-  - 2.7.1
-env:
-  - BUNNY_VERSION="~> 2.15"
-  - BUNNY_VERSION="2.10.0"
-
-before_install: gem install bundler
-before_script:
-  - ".ci/install_rabbitmq"
-bundler_args: --jobs 3 --retry 3

data/Gemfile

@@ -1,10 +0,0 @@
-# frozen_string_literal: true
-
-source 'https://rubygems.org'
-
-git_source(:github) { |repo_name| "https://github.com/#{repo_name}" }
-
-# Specify your gem's dependencies in bunny-publisher.gemspec
-gemspec
-
-gem 'bunny', ENV.fetch('BUNNY_VERSION', '~> 2.15')

data/Rakefile

@@ -1,8 +0,0 @@
-# frozen_string_literal: true
-
-require 'bundler/gem_tasks'
-require 'rspec/core/rake_task'
-
-RSpec::Core::RakeTask.new(:spec)
-
-task default: :spec

data/bunny-publisher.gemspec

@@ -1,36 +0,0 @@
-# frozen_string_literal: true
-
-lib = File.expand_path('lib', __dir__)
-$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'bunny_publisher/version'
-
-Gem::Specification.new do |spec|
-  spec.name          = 'bunny-publisher'
-  spec.version       = BunnyPublisher::VERSION
-  spec.authors       = ['Rustam Sharshenov']
-  spec.email         = ['rustam@sharshenov.com']
-
-  spec.summary       = 'AMQP publisher for RabbitMQ based on Bunny'
-  spec.description   = 'AMQP publisher for RabbitMQ based on Bunny'
-  spec.homepage      = 'https://github.com/veeqo/bunny-publisher'
-  spec.license       = 'MIT'
-
-  if spec.respond_to?(:metadata)
-    spec.metadata['homepage_uri'] = spec.homepage
-    spec.metadata['source_code_uri'] = spec.homepage
-    spec.metadata['changelog_uri'] = 'https://github.com/veeqo/bunny-publisher/blob/master/CHANGELOG.md'
-  end
-
-  spec.files = Dir.chdir(File.expand_path(__dir__)) do
-    `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(spec|\.ci)/}) }
-  end
-  spec.require_paths = ['lib']
-
-  spec.add_dependency 'bunny', '~> 2.10'
-
-  spec.add_development_dependency 'bundler'
-  spec.add_development_dependency 'pry-byebug'
-  spec.add_development_dependency 'rabbitmq_http_api_client', '~> 1.13'
-  spec.add_development_dependency 'rake'
-  spec.add_development_dependency 'rspec'
-end