bunny-mock 1.2.0 → 1.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c674fde478dce165ef8edd3f4b5679487d1a77e9
-  data.tar.gz: 9b61556fb514f0f12e3be7f72efc01a0f11a820c
+  metadata.gz: 3808cb4a1e923e7b48519a07ecb6d2acaa235da2
+  data.tar.gz: 93726320854a4f15bd18f5b5f485d24d4c901293
 SHA512:
-  metadata.gz: 06040eeb88b2446118f40a8ff9ef61d27caaa0d7173bafb6d27a8f9410b7ba9fa47e106557adb728e11d3bce8ba5d2193382944473b551fe42805cd939bcd501
-  data.tar.gz: 4fa7fd68ca0c53948622f0bc45ab230f9646864534a62ab896d47a87b3a0b1efc8c9faa52ea0b9d8907e81009924761fc2634101ed370c61f3a65bd47bab88e9
+  metadata.gz: 883f34380ad7b9720c1cbc8ae0f41ca71676cf82e6d9ec716778bc91710a79c3fb940a6631bf4a169182c75a74a46c255cebff97df7c30fef404f79a4981f5e4
+  data.tar.gz: 20406048e9c1f232892e64825fb3d7119a884623ac8457da1824d0b161614e74cc280ad17a3eb8908e9e684eae287c47384caa8175ea6a1fb9547bde7bb49599

data/.rubocop_todo.yml

@@ -1,16 +1,16 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2016-02-18 00:29:42 -0500 using RuboCop version 0.37.2.
+# on 2016-02-24 14:28:43 -0500 using RuboCop version 0.37.2.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
 # versions of RuboCop, may require this file to be generated again.
 
-# Offense count: 33
+# Offense count: 38
 # Configuration parameters: AllowHeredoc, AllowURI, URISchemes.
 # URISchemes: http, https
 Metrics/LineLength:
-  Max: 126
+  Max: 128
 
 # Offense count: 7
 Style/Documentation:
@@ -25,41 +25,21 @@ Style/Documentation:
     - 'lib/bunny_mock/exchanges/topic.rb'
     - 'lib/bunny_mock/queue.rb'
 
-# Offense count: 8
+# Offense count: 9
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, SupportedStyles.
 # SupportedStyles: empty_lines, no_empty_lines
 Style/EmptyLinesAroundClassBody:
-  Exclude:
-    - 'lib/bunny_mock/channel.rb'
-    - 'lib/bunny_mock/exchange.rb'
-    - 'lib/bunny_mock/exchanges/direct.rb'
-    - 'lib/bunny_mock/exchanges/fanout.rb'
-    - 'lib/bunny_mock/exchanges/headers.rb'
-    - 'lib/bunny_mock/exchanges/topic.rb'
-    - 'lib/bunny_mock/queue.rb'
-    - 'lib/bunny_mock/session.rb'
+  Enabled: false
 
 # Offense count: 36
 # Cop supports --auto-correct.
 Style/EmptyLinesAroundMethodBody:
-  Exclude:
-    - 'lib/bunny-mock.rb'
-    - 'lib/bunny_mock/channel.rb'
-    - 'lib/bunny_mock/exchange.rb'
-    - 'lib/bunny_mock/exchanges/direct.rb'
-    - 'lib/bunny_mock/exchanges/fanout.rb'
-    - 'lib/bunny_mock/exchanges/headers.rb'
-    - 'lib/bunny_mock/exchanges/topic.rb'
-    - 'lib/bunny_mock/queue.rb'
-    - 'lib/bunny_mock/session.rb'
+  Enabled: false
 
-# Offense count: 3
+# Offense count: 2
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle, SupportedStyles.
 # SupportedStyles: empty_lines, no_empty_lines
 Style/EmptyLinesAroundModuleBody:
-  Exclude:
-    - 'lib/bunny-mock.rb'
-    - 'lib/bunny_mock/exceptions.rb'
-    - 'lib/bunny_mock/version.rb'
+  Enabled: false

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## v1.2.1
+
+* [#5](https://github.com/arempe93/bunny-mock/pull/5): Fixes `routing_key` being modified when delivering in a topic exchange - [@austinmoore](https://github.com/austinmoore)
+
 ## v1.2.0
 
 * Adds `with_channel` functionality to `BunnyMock::Session`

data/README.md

@@ -127,6 +127,11 @@ it 'should bind exchanges to exchanges' do
 end
 ```
 
+## Other features
+
+This gem was made based on my own use of Bunny in a project. If there are other uses for Bunny that this library does not cover (eg. missing methods, functionality), feel free to open an issue or pull request!
+
+
 ## Installation
 
 #### With RubyGems

data/lib/bunny-mock.rb

@@ -24,28 +24,31 @@ module BunnyMock
   # AMQP protocol version
   PROTOCOL_VERSION = AMQ::Protocol::PROTOCOL_VERSION
 
-  #
-  # API
-  #
-
-  ##
-  # Instantiate a new mock Bunny session
-  #
-  # @return [BunnyMock::Session] Session instance
-  # @api public
-  def self.new(*)
-
-    # return new mock session
-    BunnyMock::Session.new
-  end
-
-  # @return [String] Bunny mock version
-  def self.version
-    VERSION
-  end
-
-  # @return [String] AMQP protocol version
-  def self.protocol_version
-    AMQ::Protocol::PROTOCOL_VERSION
+  class << self
+
+    #
+    # API
+    #
+
+    ##
+    # Instantiate a new mock Bunny session
+    #
+    # @return [BunnyMock::Session] Session instance
+    # @api public
+    def new(*)
+
+      # return new mock session
+      BunnyMock::Session.new
+    end
+
+    # @return [String] Bunny mock version
+    def version
+      VERSION
+    end
+
+    # @return [String] AMQP protocol version
+    def protocol_version
+      AMQ::Protocol::PROTOCOL_VERSION
+    end
   end
 end

data/lib/bunny_mock/exchange.rb

@@ -2,31 +2,34 @@
 module BunnyMock
   class Exchange
 
-    ##
-    # Create a new {BunnyMock::Exchange} instance
-    #
-    # @param [BunnyMock::Channel] channel Channel this exchange will use
-    # @param [String] name Name of exchange
-    # @param [Hash] opts Creation options
-    #
-    # @option opts [Boolean] :durable (false) Should this exchange be durable?
-    # @option opts [Boolean] :auto_delete (false) Should this exchange be automatically deleted when it is no longer used?
-    # @option opts [Boolean] :arguments ({}) Additional optional arguments (typically used by RabbitMQ extensions and plugins)
-    #
-    # @return [BunnyMock::Exchange] A new exchange
-    # @see BunnyMock::Channel#exchange
-    # @api public
-    #
-    def self.declare(channel, name = '', opts = {})
-
-      # get requested type
-      type = opts.fetch :type, :direct
-
-      # get needed class type
-      klazz = BunnyMock::Exchanges.const_get type.to_s.capitalize
-
-      # create exchange of desired type
-      klazz.new channel, name, type, opts
+    class << self
+
+      ##
+      # Create a new {BunnyMock::Exchange} instance
+      #
+      # @param [BunnyMock::Channel] channel Channel this exchange will use
+      # @param [String] name Name of exchange
+      # @param [Hash] opts Creation options
+      #
+      # @option opts [Boolean] :durable (false) Should this exchange be durable?
+      # @option opts [Boolean] :auto_delete (false) Should this exchange be automatically deleted when it is no longer used?
+      # @option opts [Boolean] :arguments ({}) Additional optional arguments (typically used by RabbitMQ extensions and plugins)
+      #
+      # @return [BunnyMock::Exchange] A new exchange
+      # @see BunnyMock::Channel#exchange
+      # @api public
+      #
+      def declare(channel, name = '', opts = {})
+
+        # get requested type
+        type = opts.fetch :type, :direct
+
+        # get needed class type
+        klazz = BunnyMock::Exchanges.const_get type.to_s.capitalize
+
+        # create exchange of desired type
+        klazz.new channel, name, type, opts
+      end
     end
 
     #
@@ -228,7 +231,11 @@ module BunnyMock
 
       @routes.key? opts.fetch(:routing_key, route)
     end
-    alias has_binding? routes_to?
+
+    def has_binding?(exchange_or_queue, opts = {}) # rubocop:disable Style/PredicateName
+      warn '[DEPRECATED] `has_binding?` is deprecated. Please use `routes_to?` instead.'
+      routes_to?(exchange_or_queue, opts)
+    end
 
     ##
     # Deliver a message to routes

data/lib/bunny_mock/exchanges/topic.rb

@@ -27,13 +27,13 @@ module BunnyMock
       def deliver(payload, opts, key)
 
         # escape periods with backslash for regex
-        key.gsub!('.', '\.')
+        key = key.gsub('.', '\.')
 
         # replace single wildcards with regex for a single domain
-        key.gsub!(SINGLE_WILDCARD, '(?:\w+)')
+        key = key.gsub(SINGLE_WILDCARD, '(?:\w+)')
 
         # replace multi wildcards with regex for many domains separated by '.'
-        key.gsub!(MULTI_WILDCARD, '\w+\.?')
+        key = key.gsub(MULTI_WILDCARD, '\w+\.?')
 
         # turn key into regex
         key = Regexp.new(key)

data/lib/bunny_mock/version.rb

@@ -4,5 +4,5 @@
 module BunnyMock
 
   # @return [String] Version of the library
-  VERSION = '1.2.0'
+  VERSION = '1.2.1'
 end

data/spec/unit/bunny_mock/channel_spec.rb

@@ -7,17 +7,14 @@ describe BunnyMock::Channel do
 		end
 
 		it 'should store connection' do
-
 			expect(@channel.connection).to eq(@session)
 		end
 
 		it 'should store channel identifier' do
-
 			expect(@channel.id).to eq(1)
 		end
 
 		it 'should set status to opening' do
-
 			expect(@channel.status).to eq(:opening)
 		end
 	end
@@ -25,7 +22,6 @@ describe BunnyMock::Channel do
 	context '#open' do
 
 		it 'should set status to open' do
-
 			expect(@channel.open.status).to eq(:open)
 		end
 	end
@@ -33,9 +29,7 @@ describe BunnyMock::Channel do
 	context '#close' do
 
 		it 'should set status to open' do
-
 			@channel.open
-
 			expect(@channel.close.status).to eq(:closed)
 		end
 	end
@@ -43,16 +37,11 @@ describe BunnyMock::Channel do
 	context '#open?' do
 
 		it 'should return true if status is open' do
-
 			expect(@channel.open?).to be_truthy
 		end
 
 		it 'should return false otherwise' do
-
-			# opening
 			expect(BunnyMock::Channel.new.open?).to be_falsey
-
-			# closed
 			expect(@channel.close.open?).to be_falsey
 		end
 	end
@@ -60,16 +49,11 @@ describe BunnyMock::Channel do
 	context '#closed?' do
 
 		it 'should return true if status is closed' do
-
 			expect(@channel.close.closed?).to be_truthy
 		end
 
 		it 'should return false otherwise' do
-
-			# opening
 			expect(BunnyMock::Channel.new.closed?).to be_falsey
-
-			# open
 			expect(@channel.closed?).to be_falsey
 		end
 	end
@@ -77,23 +61,17 @@ describe BunnyMock::Channel do
 	context '#exchange' do
 
 		it 'should declare a new exchange' do
-
 			xchg = @channel.exchange 'testing.xchg', type: :fanout
-
 			expect(xchg.class).to eq(BunnyMock::Exchanges::Fanout)
 		end
 
 		it 'should return a cached exchange with the same name' do
-
 			xchg = @channel.exchange 'testing.xchg', type: :fanout
-
 			expect(@channel.exchange('testing.xchg', type: :fanout)).to eq(xchg)
 		end
 
 		it 'should register the exchange in cache' do
-
 			xchg = @channel.exchange 'testing.xchg', type: :fanout
-
 			expect(@session.exchange_exists?('testing.xchg')).to be_truthy
 		end
 	end
@@ -101,7 +79,6 @@ describe BunnyMock::Channel do
 	context '#direct' do
 
 		it 'should declare a new direct exchange' do
-
 			expect(@channel.direct('testing.xchg').class).to eq(BunnyMock::Exchanges::Direct)
 		end
 	end
@@ -109,7 +86,6 @@ describe BunnyMock::Channel do
 	context '#topic' do
 
 		it 'should declare a new topic exchange' do
-
 			expect(@channel.topic('testing.xchg').class).to eq(BunnyMock::Exchanges::Topic)
 		end
 	end
@@ -117,7 +93,6 @@ describe BunnyMock::Channel do
 	context '#fanout' do
 
 		it 'should declare a new fanout exchange' do
-
 			expect(@channel.fanout('testing.xchg').class).to eq(BunnyMock::Exchanges::Fanout)
 		end
 	end
@@ -125,7 +100,6 @@ describe BunnyMock::Channel do
 	context '#header' do
 
 		it 'should declare a new headers exchange' do
-
 			expect(@channel.header('testing.xchg').class).to eq(BunnyMock::Exchanges::Header)
 		end
 	end
@@ -133,7 +107,6 @@ describe BunnyMock::Channel do
 	context '#default_exchange' do
 
 		it 'should return a nameless direct exchange' do
-
 			xchg = @channel.default_exchange
 
 			expect(xchg.class).to eq(BunnyMock::Exchanges::Direct)
@@ -144,23 +117,17 @@ describe BunnyMock::Channel do
 	context '#queue' do
 
 		it 'should declare a new queue' do
-
 			q = @channel.queue 'testing.q'
-
 			expect(q.class).to eq(BunnyMock::Queue)
 		end
 
 		it 'should return a cached queue with the same name' do
-
 			q = @channel.queue 'testing.q'
-
 			expect(@channel.queue('testing.q')).to eq(q)
 		end
 
 		it 'should register the queue in cache' do
-
 			q = @channel.queue 'testing.q'
-
 			expect(@session.queue_exists?('testing.q')).to be_truthy
 		end
 	end
@@ -168,7 +135,6 @@ describe BunnyMock::Channel do
 	context '#temporary_queue' do
 
 		it 'should declare a nameless queue' do
-
 			expect(@channel.temporary_queue.class).to eq(BunnyMock::Queue)
 		end
 	end

data/spec/unit/bunny_mock/exchange_spec.rb

@@ -1,172 +1,155 @@
 describe BunnyMock::Exchange do
 
-	context '::declare' do
+  context '::declare' do
 
-		it 'should create a direct exchange' do
+    it 'should create a direct exchange' do
+      expect(BunnyMock::Exchange.declare(@channel, 'testing.xchg', type: :direct).class).to eq(BunnyMock::Exchanges::Direct)
+    end
 
-			expect(BunnyMock::Exchange.declare(@channel, 'testing.xchg', type: :direct).class).to eq(BunnyMock::Exchanges::Direct)
-		end
+    it 'should create a topic exchange' do
+      expect(BunnyMock::Exchange.declare(@channel, 'testing.xchg', type: :topic).class).to eq(BunnyMock::Exchanges::Topic)
+    end
 
-		it 'should create a topic exchange' do
+    it 'should create a fanout exchange' do
+      expect(BunnyMock::Exchange.declare(@channel, 'testing.xchg', type: :fanout).class).to eq(BunnyMock::Exchanges::Fanout)
+    end
 
-			expect(BunnyMock::Exchange.declare(@channel, 'testing.xchg', type: :topic).class).to eq(BunnyMock::Exchanges::Topic)
-		end
+    it 'should create a header exchange' do
+      expect(BunnyMock::Exchange.declare(@channel, 'testing.xchg', type: :header).class).to eq(BunnyMock::Exchanges::Header)
+    end
 
-		it 'should create a fanout exchange' do
+    it 'should default to a direct exchange' do
+      expect(BunnyMock::Exchange.declare(@channel, 'testing.xchg').class).to eq(BunnyMock::Exchanges::Direct)
+    end
+  end
 
-			expect(BunnyMock::Exchange.declare(@channel, 'testing.xchg', type: :fanout).class).to eq(BunnyMock::Exchanges::Fanout)
-		end
+  context '#bind' do
 
-		it 'should create a header exchange' do
+    before do
+      @source = @channel.exchange 'xchg.source'
+      @receiver = @channel.exchange 'xchg.receiver'
+    end
 
-			expect(BunnyMock::Exchange.declare(@channel, 'testing.xchg', type: :header).class).to eq(BunnyMock::Exchanges::Header)
-		end
+    it 'should bind by exchange instance' do
+      @receiver.bind @source
 
-		it 'should default to a direct exchange' do
+      expect(@receiver.bound_to?(@source)).to be_truthy
+      expect(@source.routes_to?(@receiver)).to be_truthy
+    end
 
-			expect(BunnyMock::Exchange.declare(@channel, 'testing.xchg').class).to eq(BunnyMock::Exchanges::Direct)
-		end
-	end
+    it 'should bind by exchange name' do
+      @receiver.bind @source.name
 
-	context '#bind' do
+      expect(@receiver.bound_to?(@source)).to be_truthy
+      expect(@source.routes_to?(@receiver)).to be_truthy
+    end
 
-		before do
-			@source = @channel.exchange 'xchg.source'
-			@receiver = @channel.exchange 'xchg.receiver'
-		end
+    it 'should raise error when exchange does not exists' do
+      expect { @receiver.bind('this.xchg.does.not.exist') }.to raise_exception(Bunny::NotFound)
+    end
+  end
 
-		it 'should bind by exchange instance' do
+  context '#unbind' do
 
-			@receiver.bind @source
+    before do
+      @source = @channel.exchange 'xchg.source'
+      @receiver = @channel.exchange 'xchg.receiver'
 
-			expect(@receiver.bound_to?(@source)).to be_truthy
-			expect(@source.routes_to?(@receiver)).to be_truthy
-		end
+      @receiver.bind @source
+    end
 
-		it 'should bind by exchange name' do
+    it 'should unbind by exchange instance' do
+      @receiver.unbind @source
 
-			@receiver.bind @source.name
+      expect(@receiver.bound_to?(@source)).to be_falsey
+      expect(@source.routes_to?(@receiver)).to be_falsey
+    end
 
-			expect(@receiver.bound_to?(@source)).to be_truthy
-			expect(@source.routes_to?(@receiver)).to be_truthy
-		end
+    it 'should unbind by exchange name' do
+      @receiver.unbind @source.name
 
-		it 'should raise error when exchange does not exists' do
+      expect(@receiver.bound_to?(@source)).to be_falsey
+      expect(@source.routes_to?(@receiver)).to be_falsey
+    end
 
-			expect { @receiver.bind('this.xchg.does.not.exist') }.to raise_exception(Bunny::NotFound)
-		end
-	end
+    it 'should raise error when exchange does not exists' do
+      expect { @receiver.unbind('this.xchg.does.not.exist') }.to raise_exception(Bunny::NotFound)
+    end
+  end
 
-	context '#unbind' do
+  context '#bound_to?' do
 
-		before do
-			@source = @channel.exchange 'xchg.source'
-			@receiver = @channel.exchange 'xchg.receiver'
+    before do
+      @source = @channel.exchange 'xchg.source'
+      @receiver = @channel.exchange 'xchg.receiver'
+    end
 
-			@receiver.bind @source
-		end
+    context 'should return true if bound' do
 
-		it 'should unbind by exchange instance' do
+      it 'by instance' do
+        @receiver.bind @source
+        expect(@receiver.bound_to?(@source)).to be_truthy
+      end
 
-			@receiver.unbind @source
+      it 'by name' do
+        @receiver.bind @source
+        expect(@receiver.bound_to?(@source.name)).to be_truthy
+      end
 
-			expect(@receiver.bound_to?(@source)).to be_falsey
-			expect(@source.routes_to?(@receiver)).to be_falsey
-		end
+      it 'by routing key' do
+        @receiver.bind @source, routing_key: 'xchg.route'
 
-		it 'should unbind by exchange name' do
+        expect(@receiver.bound_to?(@source)).to be_falsey
+        expect(@receiver.bound_to?(@source, routing_key: 'xchg.route')).to be_truthy
+      end
+    end
 
-			@receiver.unbind @source.name
+    it 'return false otherwise' do
+      expect(@receiver.bound_to?(@source)).to be_falsey
+    end
 
-			expect(@receiver.bound_to?(@source)).to be_falsey
-			expect(@source.routes_to?(@receiver)).to be_falsey
-		end
+    it 'should raise error when exchange does not exists' do
+      expect { @receiver.bound_to?('this.xchg.does.not.exist') }.to raise_exception(Bunny::NotFound)
+    end
+  end
 
-		it 'should raise error when exchange does not exists' do
+  context '#routes_to?' do
 
-			expect { @receiver.unbind('this.xchg.does.not.exist') }.to raise_exception(Bunny::NotFound)
-		end
-	end
+    before do
+      @source = @channel.exchange 'xchg.source'
+      @receiver = @channel.exchange 'xchg.receiver'
 
-	context '#bound_to?' do
+      @receiver.bind @source
+    end
 
-		before do
-			@source = @channel.exchange 'xchg.source'
-			@receiver = @channel.exchange 'xchg.receiver'
-		end
+    it 'should return true if bound' do
+      expect(@receiver.bound_to?(@source)).to be_truthy
+      expect(@source.routes_to?(@receiver)).to be_truthy
+    end
 
-		context 'should return true if bound' do
+    it 'should return false if unbound' do
+      @receiver.unbind @source
 
-			it 'by instance' do
+      expect(@receiver.bound_to?(@source)).to be_falsey
+      expect(@source.routes_to?(@receiver)).to be_falsey
+    end
 
-				@receiver.bind @source
+    context 'when using #has_binding?' do
+      it 'should output a deprecation warning' do
+        expect { @source.has_binding?(@receiver) }.to output(/DEPRECATED/).to_stderr
+      end
+    end
+  end
 
-				expect(@receiver.bound_to?(@source)).to be_truthy
-			end
+  context '#delete' do
 
-			it 'by name' do
+    before do
+      @exchange = @channel.direct 'xchg.direct'
+      @exchange.delete
+    end
 
-				@receiver.bind @source
-
-				expect(@receiver.bound_to?(@source.name)).to be_truthy
-			end
-
-			it 'by routing key' do
-
-				@receiver.bind @source, routing_key: 'xchg.route'
-
-				expect(@receiver.bound_to?(@source)).to be_falsey
-				expect(@receiver.bound_to?(@source, routing_key: 'xchg.route')).to be_truthy
-			end
-		end
-
-		it 'return false otherwise' do
-
-			expect(@receiver.bound_to?(@source)).to be_falsey
-		end
-
-		it 'should raise error when exchange does not exists' do
-
-			expect { @receiver.bound_to?('this.xchg.does.not.exist') }.to raise_exception(Bunny::NotFound)
-		end
-	end
-
-	context '#routes_to?' do
-
-		before do
-			@source = @channel.exchange 'xchg.source'
-			@receiver = @channel.exchange 'xchg.receiver'
-
-			@receiver.bind @source
-		end
-
-		it 'should unbind by exchange instance' do
-
-			@receiver.unbind @source
-
-			expect(@receiver.bound_to?(@source)).to be_falsey
-			expect(@source.routes_to?(@receiver)).to be_falsey
-		end
-
-		it 'should unbind by exchange name' do
-
-			@receiver.unbind @source.name
-
-			expect(@receiver.bound_to?(@source)).to be_falsey
-			expect(@source.routes_to?(@receiver)).to be_falsey
-		end
-	end
-
-	context '#delete' do
-
-		before do
-			@exchange = @channel.direct 'xchg.direct'
-			@exchange.delete
-		end
-
-		it 'should remove exchange from session' do
-
-			expect(@session.exchange_exists?(@exchange.name)).to be_falsey
-		end
-	end
+    it 'should remove exchange from session' do
+      expect(@session.exchange_exists?(@exchange.name)).to be_falsey
+    end
+  end
 end

data/spec/unit/bunny_mock/exchanges/direct_spec.rb

@@ -15,7 +15,6 @@ describe BunnyMock::Exchanges::Direct do
 		end
 
 		it 'should only deliver to direct route match' do
-
 			@source.publish 'Testing message', routing_key: 'queue.second'
 
 			expect(@first.message_count).to eq(0)

data/spec/unit/bunny_mock/exchanges/fanout_spec.rb

@@ -15,7 +15,6 @@ describe BunnyMock::Exchanges::Fanout do
 		end
 
 		it 'should only deliver to all routes' do
-
 			@source.publish 'Testing message', routing_key: 'queue.second'
 
 			expect(@first.message_count).to eq(1)

data/spec/unit/bunny_mock/exchanges/topic_spec.rb

@@ -15,7 +15,6 @@ describe BunnyMock::Exchanges::Topic do
 		end
 
 		it 'should deliver with no wildcards' do
-
 			@source.publish 'Testing message', routing_key: 'queue.category.second'
 
 			expect(@first.message_count).to eq(0)
@@ -23,12 +22,17 @@ describe BunnyMock::Exchanges::Topic do
 
 			expect(@second.message_count).to eq(1)
 			expect(@second.pop[:message]).to eq('Testing message')
-		end
+    end
 
-		context 'should deliver with wildcards' do
+    it 'does not modify the routing key' do
+      @source.publish 'Testing message',
+                      routing_key: 'queue.category.sub.first'.freeze
+      expect(message = @first.pop).to_not be_nil
+      expect(message[:options][:routing_key]).to eq('queue.category.sub.first')
+    end
 
+		context 'should deliver with wildcards' do
 			it 'for single wildcards' do
-
 				@source.publish 'Testing message', routing_key: 'queue.*.sub.*'
 
 				expect(@second.message_count).to eq(0)
@@ -41,7 +45,6 @@ describe BunnyMock::Exchanges::Topic do
 			end
 
 			it 'for multiple wildcards' do
-
 				@source.publish 'Testing message', routing_key: 'queue.category.#'
 
 				expect(@third.message_count).to eq(0)
@@ -54,7 +57,6 @@ describe BunnyMock::Exchanges::Topic do
 			end
 
 			it 'for a mixed wildcards' do
-
 				@source.publish 'Testing message', routing_key: '#.sub.*'
 
 				expect(@second.message_count).to eq(0)

data/spec/unit/bunny_mock/queue_spec.rb

@@ -1,172 +1,154 @@
 describe BunnyMock::Queue do
 
-	before do
-		@queue = @channel.queue 'testing.q'
-	end
+  before do
+    @queue = @channel.queue 'testing.q'
+  end
 
-	context '#publish' do
+  context '#publish' do
 
-		it 'should add message' do
+    it 'should add message' do
+      @queue.publish 'This is a test message'
 
-			@queue.publish 'This is a test message'
+      expect(@queue.message_count).to eq(1)
+      expect(@queue.pop[:message]).to eq('This is a test message')
+      expect(@queue.message_count).to eq(0)
+    end
+  end
 
-			expect(@queue.message_count).to eq(1)
-			expect(@queue.pop[:message]).to eq('This is a test message')
-			expect(@queue.message_count).to eq(0)
-		end
-	end
+  context '#bind' do
 
-	context '#bind' do
+    before do
+      @source = @channel.exchange 'xchg.source'
+      @receiver = @channel.queue 'queue.receiver'
+    end
 
-		before do
-			@source = @channel.exchange 'xchg.source'
-			@receiver = @channel.queue 'queue.receiver'
-		end
+    it 'should bind by exchange instance' do
+      @receiver.bind @source
 
-		it 'should bind by exchange instance' do
+      expect(@receiver.bound_to?(@source)).to be_truthy
+      expect(@source.routes_to?(@receiver)).to be_truthy
+    end
 
-			@receiver.bind @source
+    it 'should bind by exchange name' do
+      @receiver.bind @source.name
 
-			expect(@receiver.bound_to?(@source)).to be_truthy
-			expect(@source.routes_to?(@receiver)).to be_truthy
-		end
+      expect(@receiver.bound_to?(@source)).to be_truthy
+      expect(@source.routes_to?(@receiver)).to be_truthy
+    end
 
-		it 'should bind by exchange name' do
+    it 'should raise error when exchange does not exists' do
+      expect { @receiver.bind('this.xchg.does.not.exist') }.to raise_exception(Bunny::NotFound)
+    end
+  end
 
-			@receiver.bind @source.name
+  context '#unbind' do
 
-			expect(@receiver.bound_to?(@source)).to be_truthy
-			expect(@source.routes_to?(@receiver)).to be_truthy
-		end
+    before do
+      @source = @channel.exchange 'xchg.source'
+      @receiver = @channel.queue 'queue.receiver'
 
-		it 'should raise error when exchange does not exists' do
+      @receiver.bind @source
+    end
 
-			expect { @receiver.bind('this.xchg.does.not.exist') }.to raise_exception(Bunny::NotFound)
-		end
-	end
+    it 'should unbind by exchange instance' do
+      @receiver.unbind @source
 
-	context '#unbind' do
+      expect(@receiver.bound_to?(@source)).to be_falsey
+      expect(@source.routes_to?(@receiver)).to be_falsey
+    end
 
-		before do
-			@source = @channel.exchange 'xchg.source'
-			@receiver = @channel.queue 'queue.receiver'
+    it 'should unbind by exchange name' do
+      @receiver.unbind @source.name
 
-			@receiver.bind @source
-		end
+      expect(@receiver.bound_to?(@source)).to be_falsey
+      expect(@source.routes_to?(@receiver)).to be_falsey
+    end
 
-		it 'should unbind by exchange instance' do
+    it 'should raise error when exchange does not exists' do
+      expect { @receiver.unbind('this.xchg.does.not.exist') }.to raise_exception(Bunny::NotFound)
+    end
+  end
 
-			@receiver.unbind @source
+  context '#bound_to?' do
 
-			expect(@receiver.bound_to?(@source)).to be_falsey
-			expect(@source.routes_to?(@receiver)).to be_falsey
-		end
+    before do
+      @source = @channel.exchange 'xchg.source'
+      @receiver = @channel.queue 'queue.receiver'
+    end
 
-		it 'should unbind by exchange name' do
+    context 'should return true if bound' do
 
-			@receiver.unbind @source.name
+      it 'by instance' do
+        @receiver.bind @source
+        expect(@receiver.bound_to?(@source)).to be_truthy
+      end
 
-			expect(@receiver.bound_to?(@source)).to be_falsey
-			expect(@source.routes_to?(@receiver)).to be_falsey
-		end
+      it 'by name' do
+        @receiver.bind @source
+        expect(@receiver.bound_to?(@source.name)).to be_truthy
+      end
 
-		it 'should raise error when exchange does not exists' do
+      it 'by routing key' do
+        @receiver.bind @source, routing_key: 'queue.route'
 
-			expect { @receiver.unbind('this.xchg.does.not.exist') }.to raise_exception(Bunny::NotFound)
-		end
-	end
+        expect(@receiver.bound_to?(@source)).to be_falsey
+        expect(@receiver.bound_to?(@source, routing_key: 'queue.route')).to be_truthy
+      end
+    end
 
-	context '#bound_to?' do
+    it 'return false otherwise' do
+      expect(@receiver.bound_to?(@source)).to be_falsey
+    end
 
-		before do
-			@source = @channel.exchange 'xchg.source'
-			@receiver = @channel.queue 'queue.receiver'
-		end
+    it 'should raise error when exchange does not exists' do
+      expect { @receiver.bound_to?('this.xchg.does.not.exist') }.to raise_exception(Bunny::NotFound)
+    end
+  end
 
-		context 'should return true if bound' do
+  context '#message_count' do
 
-			it 'by instance' do
+    it 'should return number of messages in queue' do
+      @queue.publish 'First'
+      @queue.publish 'Second'
 
-				@receiver.bind @source
+      expect(@queue.message_count).to eq(2)
 
-				expect(@receiver.bound_to?(@source)).to be_truthy
-			end
+      @queue.pop
 
-			it 'by name' do
+      expect(@queue.message_count).to eq(1)
+    end
+  end
 
-				@receiver.bind @source
+  context '#pop' do
 
-				expect(@receiver.bound_to?(@source.name)).to be_truthy
-			end
+    it 'should return oldest message in queue' do
+      @queue.publish 'First'
+      @queue.publish 'Second'
 
-			it 'by routing key' do
+      expect(@queue.pop[:message]).to eq('First')
+    end
+  end
 
-				@receiver.bind @source, routing_key: 'queue.route'
+  context '#purge' do
 
-				expect(@receiver.bound_to?(@source)).to be_falsey
-				expect(@receiver.bound_to?(@source, routing_key: 'queue.route')).to be_truthy
-			end
-		end
+    it 'should clear all messages' do
+      @queue.publish 'First'
+      @queue.publish 'Second'
 
-		it 'return false otherwise' do
+      @queue.purge
 
-			expect(@receiver.bound_to?(@source)).to be_falsey
-		end
+      expect(@queue.message_count).to eq(0)
+    end
+  end
 
-		it 'should raise error when exchange does not exists' do
+  context '#delete' do
 
-			expect { @receiver.bound_to?('this.xchg.does.not.exist') }.to raise_exception(Bunny::NotFound)
-		end
-	end
+    before do
+      @queue.delete
+    end
 
-	context '#message_count' do
-
-		it 'should return number of messages in queue' do
-
-			@queue.publish 'First'
-			@queue.publish 'Second'
-
-			expect(@queue.message_count).to eq(2)
-
-			@queue.pop
-
-			expect(@queue.message_count).to eq(1)
-		end
-	end
-
-	context '#pop' do
-
-		it 'should return oldest message in queue' do
-
-			@queue.publish 'First'
-			@queue.publish 'Second'
-
-			expect(@queue.pop[:message]).to eq('First')
-		end
-	end
-
-	context '#purge' do
-
-		it 'should clear all messages' do
-
-			@queue.publish 'First'
-			@queue.publish 'Second'
-
-			@queue.purge
-
-			expect(@queue.message_count).to eq(0)
-		end
-	end
-
-	context '#delete' do
-
-		before do
-			@queue.delete
-		end
-
-		it 'should remove queue from session' do
-
-			expect(@session.queue_exists?(@queue.name)).to be_falsey
-		end
-	end
+    it 'should remove queue from session' do
+      expect(@session.queue_exists?(@queue.name)).to be_falsey
+    end
+  end
 end

data/spec/unit/bunny_mock/session_spec.rb

@@ -7,7 +7,6 @@ describe BunnyMock::Session do
   context '::new' do
 
     it 'should start as not connected' do
-
       expect(@session.status).to eq(:not_connected)
     end
   end
@@ -15,7 +14,6 @@ describe BunnyMock::Session do
   context '#start' do
 
     it 'should set status to connected' do
-
       expect(@session.start.status).to eq(:connected)
     end
   end
@@ -23,9 +21,7 @@ describe BunnyMock::Session do
   context '#stop (close)' do
 
     it 'should set status to closed' do
-
       @session.start
-
       expect(@session.stop.status).to eq(:closed)
     end
   end
@@ -33,14 +29,11 @@ describe BunnyMock::Session do
   context '#open?' do
 
     it 'should return true if status is open' do
-
       @session.start
-
       expect(@session.open?).to be_truthy
     end
 
     it 'should return false otherwise' do
-
       expect(@session.status).to eq(:not_connected)
       expect(@session.open?).to be_falsey
 
@@ -55,7 +48,6 @@ describe BunnyMock::Session do
   context '#create_channel (channel)' do
 
     it 'should create a new channel with no arguments' do
-
       first = @session.create_channel
       second = @session.create_channel
 
@@ -66,7 +58,6 @@ describe BunnyMock::Session do
     end
 
     it 'should return cached channel with same identifier' do
-
       first = @session.create_channel 1
       second = @session.create_channel 1
 
@@ -74,7 +65,6 @@ describe BunnyMock::Session do
     end
 
     it 'should return an ArgumentError for reserved channel' do
-
       expect { @session.create_channel(0) }.to raise_error(ArgumentError)
     end
   end
@@ -83,7 +73,6 @@ describe BunnyMock::Session do
 
     it 'should close the channel after the block ends' do
       channel = nil
-
       @session.with_channel { |c| channel = c }
 
       expect(channel.closed?).to be_truthy

data/spec/unit/bunny_mock_spec.rb

@@ -3,7 +3,6 @@ describe BunnyMock do
 	context '::new' do
 
 		it 'should return a new session' do
-
 			expect(BunnyMock.new.class).to eq(BunnyMock::Session)
 		end
 	end
@@ -11,7 +10,6 @@ describe BunnyMock do
 	context '::version' do
 
 		it 'should return the current version' do
-
 			expect(BunnyMock::VERSION).to_not be_nil
 			expect(BunnyMock.version).to_not be_nil
 		end
@@ -20,7 +18,6 @@ describe BunnyMock do
 	context '::protocol_version' do
 
 		it 'should return the current amq protocol version' do
-
 			expect(BunnyMock::PROTOCOL_VERSION).to eq('0.9.1')
 			expect(BunnyMock.protocol_version).to eq('0.9.1')
 		end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bunny-mock
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.2.1
 platform: ruby
 authors:
 - Andrew Rempe
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-23 00:00:00.000000000 Z
+date: 2016-03-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bunny