bunny-mock 1.2.2 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 236588fb9cea816f91284a397efdf607f564077e
-  data.tar.gz: 58a81920f3d8909260f4468f6f0fdab8a303c150
+  metadata.gz: 7ce7333898b5b71042b88715549e3aff6b578548
+  data.tar.gz: 576392f37adb5c7baf5c85eb3074b096bf75894d
 SHA512:
-  metadata.gz: d924c1e85439de0784bd9821343d6422b18a9692f829a785e4ac3a6298390c075c88371ab1a0df22db4e81648884457f8cda85df504146c2a90d652df71b8f95
-  data.tar.gz: 0c66093b0771d596a9e25724685e4c1846c9886c654897d8df55ef112d837fbc4df5ba4d831ef08bc4b068eb3a8b76afc28497338de45ce4e209249c2d1a6666
+  metadata.gz: 9aa0f6f2ad6557edbf70a984c0277396c69a820b2f8ed43f18e8e4be6ccdd654b1cc5993cff421a510713461a5755c599b6773ab28adfb1efb3ebcf76033953d
+  data.tar.gz: 94ea8ce5094702d17692bb653a9761313a6f6b9c0d02e9eabca040c9ca33e47bf447b8fd5cb97d02f04e1760f46cb5efd5ff8cb22395e322b90fc86b9d02bb0a

data/.rubocop.yml

@@ -6,4 +6,7 @@ AllCops:
   Exclude:
     - 'spec/**/*'
 
+Metrics/ClassLength:
+  Enabled: false
+
 inherit_from: .rubocop_todo.yml

data/.rubocop_todo.yml

@@ -24,22 +24,3 @@ Style/Documentation:
     - 'lib/bunny_mock/exchanges/headers.rb'
     - 'lib/bunny_mock/exchanges/topic.rb'
     - 'lib/bunny_mock/queue.rb'
-
-# Offense count: 9
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-# SupportedStyles: empty_lines, no_empty_lines
-Style/EmptyLinesAroundClassBody:
-  Enabled: false
-
-# Offense count: 36
-# Cop supports --auto-correct.
-Style/EmptyLinesAroundMethodBody:
-  Enabled: false
-
-# Offense count: 2
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-# SupportedStyles: empty_lines, no_empty_lines
-Style/EmptyLinesAroundModuleBody:
-  Enabled: false

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## v1.3.0
+
+* [#12](https://github.com/arempe93/bunny-mock/pull/12): Adds `basic_publish` functionality to `Channel` - [@podung](https://github.com/podung)
+* [#13](https://github.com/arempe93/bunny-mock/pull/13): Add `confirm_select` method stub - [@baelter](https://github.com/baelter)
+
 ## v1.2.2
 
 * [#6](https://github.com/arempe93/bunny-mock/pull/6): Adds more status methods to `Session` - [@syndbg](https://github.com/syndbg)

data/README.md

@@ -8,6 +8,10 @@ Bunny Mock
 
 A mock client for RabbitMQ, modeled after the popular [Bunny client](https://github.com/ruby-amqp/bunny). It currently supports basic usage of Bunny for managing exchanges and queues, with the goal of being able to handle and test all Bunny use cases.
 
+##### Upgrading
+
+This project does its best to follow [semantic versioning practices](http://semver.org/). Check the [CHANGELOG](CHANGELOG.md) to see detailed versioning notes, and [UPGRADING](UPGRADING.md) for notes about major changes or deprecations.
+
 ## Usage
 
 BunnyMock can be injected into your RabbitMQ application in place of Bunny for testing. For example, if you have a helper module named `AMQFactory`, some code similar to the following placed in `spec_helper` or `test_helper` or what have you is all you need to start using BunnyMock to test your RabbitMQ application
@@ -43,10 +47,10 @@ it 'should create queues and exchanges' do
   expect(session.queue_exists?('queue.test')).to be_falsey
 
   xchg = channel.exchange 'xchg.test'
-  expect(session.exchange_exists?('exchange.test')).to be_truthy
+  expect(session.exchange_exists?('xchg.test')).to be_truthy
 
   xchg.delete
-  expect(session.exchange_exists?('exchange.test')).to be_falsey
+  expect(session.exchange_exists?('xchg.test')).to be_falsey
 end
 ```
 
@@ -76,6 +80,7 @@ it 'should route messages from exchanges' do
   xchg = channel.topic 'xchg.topic'
   queue = channel.queue 'queue.test'
 
+  queue.bind xchg
   xchg.publish 'Routed message', routing_key: '*.test'
 
   expect(queue.message_count).to eq(1)
@@ -119,7 +124,7 @@ it 'should bind exchanges to exchanges' do
 
   receiver.unbind source
   expect(receiver.bound_to?(source)).to be_falsey
-  expect(xchg.routes_to?(receiver)).to be_falsey
+  expect(source.routes_to?(receiver)).to be_falsey
 
   receiver.bind 'xchg.source'
   expect(receiver.bound_to?(source)).to be_truthy

data/lib/bunny-mock.rb

@@ -20,12 +20,10 @@ require 'bunny_mock/exchanges/headers'
 # @see https://github.com/ruby-amq/bunny
 #
 module BunnyMock
-
   # AMQP protocol version
   PROTOCOL_VERSION = AMQ::Protocol::PROTOCOL_VERSION
 
   class << self
-
     #
     # API
     #
@@ -36,7 +34,6 @@ module BunnyMock
     # @return [BunnyMock::Session] Session instance
     # @api public
     def new(*)
-
       # return new mock session
       BunnyMock::Session.new
     end

data/lib/bunny_mock/channel.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 module BunnyMock
   class Channel
-
     #
     # API
     #
@@ -24,7 +23,6 @@ module BunnyMock
     # @api public
     #
     def initialize(connection = nil, id = nil)
-
       # store channel id
       @id = id
 
@@ -58,7 +56,6 @@ module BunnyMock
     # @api public
     #
     def open
-
       @status = :open
 
       self
@@ -71,7 +68,6 @@ module BunnyMock
     # @api public
     #
     def close
-
       @status = :closed
 
       self
@@ -100,10 +96,7 @@ module BunnyMock
     # @api public
     #
     def exchange(name, opts = {})
-
-      xchg = @connection.find_exchange(name) || Exchange.declare(self, name, opts)
-
-      @connection.register_exchange xchg
+      @connection.register_exchange xchg_find_or_create(name, opts)
     end
 
     ##
@@ -184,6 +177,23 @@ module BunnyMock
       direct '', no_declare: true
     end
 
+    ##
+    # Mocks Bunny::Channel#basic_publish
+    #
+    # @param [String] payload Message payload. It will never be modified by Bunny or RabbitMQ in any way.
+    # @param [String] exchange Exchange to publish to
+    # @param [String] routing_key Routing key
+    # @param [Hash] opts Publishing options
+
+    # @return [BunnyMock::Channel] Self
+    def basic_publish(payload, xchg, routing_key, opts = {})
+      xchg = xchg_find_or_create(xchg) unless xchg.respond_to? :name
+
+      xchg.publish payload, opts.merge(routing_key: routing_key)
+
+      self
+    end
+
     # @endgroup
 
     # @group Queue API
@@ -198,9 +208,7 @@ module BunnyMock
     # @api public
     #
     def queue(name = '', opts = {})
-
       queue = @connection.find_queue(name) || Queue.new(self, name, opts)
-
       @connection.register_queue queue
     end
 
@@ -214,10 +222,19 @@ module BunnyMock
     # @api public
     #
     def temporary_queue(opts = {})
-
       queue '', opts.merge(exclusive: true)
     end
 
+    ##
+    # Does nothing atm.
+    #
+    # @return nil
+    # @api public
+    #
+    def confirm_select(callback = nil)
+      # noop
+    end
+
     # @endgroup
 
     #
@@ -236,9 +253,7 @@ module BunnyMock
 
     # @private
     def queue_bind(queue, key, xchg)
-
       exchange = @connection.find_exchange xchg
-
       raise Bunny::NotFound.new("Exchange '#{xchg}' was not found", self, false) unless exchange
 
       exchange.add_route key, queue
@@ -246,9 +261,7 @@ module BunnyMock
 
     # @private
     def queue_unbind(key, xchg)
-
       exchange = @connection.find_exchange xchg
-
       raise Bunny::NotFound.new("Exchange '#{xchg}' was not found", self, false) unless exchange
 
       exchange.remove_route key
@@ -256,9 +269,7 @@ module BunnyMock
 
     # @private
     def xchg_bound_to?(receiver, key, name)
-
       source = @connection.find_exchange name
-
       raise Bunny::NotFound.new("Exchange '#{name}' was not found", self, false) unless source
 
       source.routes_to? receiver, routing_key: key
@@ -266,9 +277,7 @@ module BunnyMock
 
     # @private
     def xchg_routes_to?(key, xchg)
-
       exchange = @connection.find_exchange xchg
-
       raise Bunny::NotFound.new("Exchange '#{xchg}' was not found", self, false) unless exchange
 
       exchange.routes_to? key
@@ -276,9 +285,7 @@ module BunnyMock
 
     # @private
     def xchg_bind(receiver, routing_key, name)
-
       source = @connection.find_exchange name
-
       raise Bunny::NotFound.new("Exchange '#{name}' was not found", self, false) unless source
 
       source.add_route routing_key, receiver
@@ -286,12 +293,17 @@ module BunnyMock
 
     # @private
     def xchg_unbind(routing_key, name)
-
       source = @connection.find_exchange name
-
       raise Bunny::NotFound.new("Exchange '#{name}' was not found", self, false) unless source
 
       source.remove_route routing_key
     end
+
+    private
+
+    # @private
+    def xchg_find_or_create(name, opts = {})
+      @connection.find_exchange(name) || Exchange.declare(self, name, opts)
+    end
   end
 end

data/lib/bunny_mock/exchange.rb

@@ -1,9 +1,7 @@
 # frozen_string_literal: true
 module BunnyMock
   class Exchange
-
     class << self
-
       ##
       # Create a new {BunnyMock::Exchange} instance
       #
@@ -20,7 +18,6 @@ module BunnyMock
       # @api public
       #
       def declare(channel, name = '', opts = {})
-
         # get requested type
         type = opts.fetch :type, :direct
 
@@ -69,7 +66,6 @@ module BunnyMock
 
     # @private
     def initialize(channel, name, type, opts)
-
       # store creation information
       @channel  = channel
       @name     = name
@@ -117,7 +113,6 @@ module BunnyMock
     # @api public
     #
     def publish(payload, opts = {})
-
       # handle message sending, varies by type
       deliver payload, opts, opts.fetch(:routing_key, '')
 
@@ -130,7 +125,6 @@ module BunnyMock
     # @api public
     #
     def delete(*)
-
       @channel.deregister_exchange self
 
       @deleted = true
@@ -148,12 +142,10 @@ module BunnyMock
     # @api public
     #
     def bind(exchange, opts = {})
-
       if exchange.respond_to?(:add_route)
 
         # we can do the binding ourselves
         exchange.add_route opts.fetch(:routing_key, @name), self
-
       else
 
         # we need the channel to look up the exchange
@@ -175,12 +167,10 @@ module BunnyMock
     # @api public
     #
     def unbind(exchange, opts = {})
-
       if exchange.respond_to?(:remove_route)
 
         # we can do the unbinding ourselves
         exchange.remove_route opts.fetch(:routing_key, @name)
-
       else
 
         # we need the channel to look up the exchange
@@ -204,12 +194,10 @@ module BunnyMock
     # @api public
     #
     def bound_to?(exchange, opts = {})
-
       if exchange.respond_to?(:routes_to?)
 
         # we can find out on the exchange object
         exchange.routes_to? self, opts
-
       else
 
         # we need the channel to look up the exchange
@@ -229,9 +217,7 @@ module BunnyMock
     # @api public
     #
     def routes_to?(exchange_or_queue, opts = {})
-
       route = exchange_or_queue.respond_to?(:name) ? exchange_or_queue.name : exchange_or_queue
-
       @routes.key? opts.fetch(:routing_key, route)
     end
 

data/lib/bunny_mock/exchanges/direct.rb

@@ -2,7 +2,6 @@
 module BunnyMock
   module Exchanges
     class Direct < BunnyMock::Exchange
-
       #
       # API
       #
@@ -17,7 +16,6 @@ module BunnyMock
       # @api public
       #
       def deliver(payload, opts, key)
-
         @routes[key].publish payload, opts if @routes[key]
       end
     end

data/lib/bunny_mock/exchanges/fanout.rb

@@ -2,7 +2,6 @@
 module BunnyMock
   module Exchanges
     class Fanout < BunnyMock::Exchange
-
       #
       # API
       #
@@ -17,7 +16,6 @@ module BunnyMock
       # @api public
       #
       def deliver(payload, opts, _key)
-
         @routes.each_value { |destination| destination.publish(payload, opts) }
       end
     end

data/lib/bunny_mock/exchanges/headers.rb

@@ -2,7 +2,6 @@
 module BunnyMock
   module Exchanges
     class Header < BunnyMock::Exchange
-
       # @private
       # @return [Regexp] Any match
       ANY = /^any$/i
@@ -25,7 +24,6 @@ module BunnyMock
       # @api public
       #
       def deliver(payload, opts, key)
-
         # ~: proper headers exchange implementation
         @routes[key].publish payload, opts if @routes[key]
       end

data/lib/bunny_mock/exchanges/topic.rb

@@ -2,7 +2,6 @@
 module BunnyMock
   module Exchanges
     class Topic < BunnyMock::Exchange
-
       # @private
       # @return [String] Multiple subdomain wildcard
       MULTI_WILDCARD = '#'
@@ -25,7 +24,6 @@ module BunnyMock
       # @api public
       #
       def deliver(payload, opts, key)
-
         # escape periods with backslash for regex
         key = key.gsub('.', '\.')
 

data/lib/bunny_mock/queue.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 module BunnyMock
   class Queue
-
     #
     # API
     #
@@ -25,7 +24,6 @@ module BunnyMock
     # @see BunnyMock::Channel#queue
     #
     def initialize(channel, name = '', opts = {})
-
       # Store creation information
       @channel = channel
       @name    = name
@@ -63,7 +61,6 @@ module BunnyMock
     # @api public
     #
     def publish(payload, opts = {})
-
       check_queue_deleted!
 
       # add to messages
@@ -83,14 +80,12 @@ module BunnyMock
     # @api public
     #
     def bind(exchange, opts = {})
-
       check_queue_deleted!
 
       if exchange.respond_to?(:add_route)
 
         # we can do the binding ourselves
         exchange.add_route opts.fetch(:routing_key, @name), self
-
       else
 
         # we need the channel to lookup the exchange
@@ -109,14 +104,12 @@ module BunnyMock
     # @api public
     #
     def unbind(exchange, opts = {})
-
       check_queue_deleted!
 
       if exchange.respond_to?(:remove_route)
 
         # we can do the unbinding ourselves
         exchange.remove_route opts.fetch(:routing_key, @name)
-
       else
 
         # we need the channel to lookup the exchange
@@ -138,14 +131,12 @@ module BunnyMock
     # @api public
     #
     def bound_to?(exchange, opts = {})
-
       check_queue_deleted!
 
       if exchange.respond_to?(:routes_to?)
 
         # we can do the check ourselves
         exchange.routes_to? opts.fetch(:routing_key, @name)
-
       else
 
         # we need the channel to lookup the exchange
@@ -198,9 +189,7 @@ module BunnyMock
     # @api public
     #
     def delete
-
       @channel.deregister_queue self
-
       @deleted = true
     end
 

data/lib/bunny_mock/session.rb

@@ -2,7 +2,6 @@
 module BunnyMock
   # Mocks Bunny::Session
   class Session
-
     #
     # API
     #
@@ -21,7 +20,6 @@ module BunnyMock
     #
     # @api public
     def initialize(*)
-
       # not connected until {BunnyMock::Session#start} is called
       @status = :not_connected
 
@@ -39,9 +37,7 @@ module BunnyMock
     # @return [BunnyMock::Session] self
     # @api public
     def start
-
       @status = :connected
-
       self
     end
 
@@ -52,7 +48,6 @@ module BunnyMock
     # @api public
     def stop
       @status = :closed
-
       self
     end
     alias close stop
@@ -94,7 +89,6 @@ module BunnyMock
     # @return [BunnyMock::Channel] Channel instance
     # @api public
     def create_channel(n = nil, _pool_size = 1)
-
       # raise same error as {Bunny::Session#create_channel}
       raise ArgumentError, 'channel number 0 is reserved in the protocol and cannot be used' if n == 0
 

data/lib/bunny_mock/version.rb

@@ -2,7 +2,6 @@
 # frozen_string_literal: true
 
 module BunnyMock
-
   # @return [String] Version of the library
-  VERSION = '1.2.2'
+  VERSION = '1.3.0'
 end

data/spec/unit/bunny_mock/channel_spec.rb

@@ -114,6 +114,49 @@ describe BunnyMock::Channel do
 		end
 	end
 
+	context '#basic_publish' do
+		let(:xchg_name) { 'testing.xchg' }
+		let(:key) { 'routing.key' }
+		let(:data) { { some: 'data' } }
+
+		let(:xchg) { @channel.direct xchg_name }
+		let(:queue) { @channel.queue 'testing.queue' }
+
+		before do
+			queue.bind(xchg, routing_key: key)
+		end
+
+		it 'returns BunnyMock::Channel#self' do
+			result = @channel.basic_publish(data, xchg_name, key)
+
+			expect(result).to eq @channel
+		end
+
+		it 'should publish to the exchange' do
+			@channel.basic_publish(data, xchg_name, key)
+
+			expect(queue.pop[:message]).to eq data
+		end
+
+		it 'accepts exchange object for exchange param' do
+			@channel.basic_publish(data, xchg, key)
+
+			expect(queue.pop[:message]).to eq data
+		end
+
+		it 'passes opts down to exchange' do
+			@channel.basic_publish(data, xchg, key, extra: 'opts')
+
+			expect(queue.pop[:options]).to include(extra: 'opts')
+		end
+
+		it 'creates exchange if it does not exist' do
+			@channel.basic_publish(data, 'some.other.xchg', key)
+
+			expect(@channel.exchange('some.other.xchg')).to be_kind_of BunnyMock::Exchange
+		end
+	end
+
 	context '#queue' do
 
 		it 'should declare a new queue' do

data/spec/unit/bunny_mock_spec.rb

@@ -22,4 +22,14 @@ describe BunnyMock do
 			expect(BunnyMock.protocol_version).to eq('0.9.1')
 		end
 	end
+
+  it 'should route messages from exchanges' do
+    channel = BunnyMock.new.start.channel
+
+    xchg = channel.topic 'xchg.topic'
+    queue = channel.queue 'queue.test'
+
+    queue.bind xchg
+    xchg.publish('Routed message', routing_key: '*.test')
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bunny-mock
 version: !ruby/object:Gem::Version
-  version: 1.2.2
+  version: 1.3.0
 platform: ruby
 authors:
 - Andrew Rempe
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-03-28 00:00:00.000000000 Z
+date: 2016-05-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bunny