amqp 0.8.0.rc13 → 0.8.0.rc14

data/spec/integration/{extensions/basic_return_spec.rb → basic_return_spec.rb}

@@ -1,4 +1,5 @@
-# -*- coding: utf-8 -*-
+# encoding: utf-8
+
 require "spec_helper"
 
 

data/spec/integration/channel_level_exception_handling_spec.rb

@@ -0,0 +1,53 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe "Channel-level exception" do
+
+  #
+  # Environment
+  #
+
+  include EventedSpec::AMQPSpec
+
+  default_timeout 2
+
+  amqp_before do
+    @connection = AMQP.connect
+    @channel    = AMQP::Channel.new(@connection)
+  end
+
+  after(:all) do
+    AMQP.cleanup_state
+    done
+  end
+
+
+  #
+  # Examples
+  #
+
+  it "can be handled with Channel#on_error" do
+    @channel.on_error do |ch, channel_close|
+      channel_close.method_id.should == 10
+      channel_close.class_id.should == 50
+      channel_close.reply_code.should == 406
+      channel_close.reply_text.should_not be_nil
+      channel_close.reply_text.should_not be_empty
+
+      done
+    end
+
+    EventMachine.add_timer(0.4) do
+      # these two definitions result in a race condition. For sake of this example,
+      # however, it does not matter. Whatever definition succeeds first, 2nd one will
+      # cause a channel-level exception (because attributes are not identical)
+      #
+      # 'a' * 80 makes queue name long enough for complete error message to be longer than 127 charachters.
+      # makes it possible to detect signed vs unsigned integer issues in amq-protocol. MK.
+      AMQP::Queue.new(@channel, "amqpgem.examples.channel_exception.#{'a' * 80}", :auto_delete => true, :durable => false)
+
+      AMQP::Queue.new(@channel, "amqpgem.examples.channel_exception.#{'a' * 80}", :auto_delete => true, :durable => true)
+    end
+  end
+end

data/spec/integration/connection_level_exception_handling_spec.rb

@@ -0,0 +1,49 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe "Connection-level exception" do
+
+  #
+  # Environment
+  #
+
+  include EventedSpec::AMQPSpec
+
+  default_timeout 2
+
+  amqp_before do
+    @connection = AMQP.connect
+    @channel    = AMQP::Channel.new(@connection)
+  end
+
+  after(:all) do
+    AMQP.cleanup_state
+    done
+  end
+
+
+  #
+  # Examples
+  #
+
+  it "can be handled with Session#on_error" do
+    @connection.on_error do |conn, connection_close|
+      conn.should == @connection
+
+      connection_close.method_id.should == 31
+      connection_close.class_id.should == 10
+      connection_close.reply_code.should == 504
+      connection_close.reply_text.should_not be_nil
+      connection_close.reply_text.should_not be_empty
+
+      done
+    end
+
+    EventMachine.add_timer(0.3) do
+      # send_frame is NOT part of the public API, but it is public for entities like AMQ::Client::Channel
+      # and we use it here to trigger a connection-level exception. MK.
+      @connection.send_frame(AMQ::Protocol::Connection::TuneOk.encode(1000, 1024 * 128 * 1024, 10))
+    end
+  end
+end

data/spec/integration/declare_and_immediately_bind_a_server_named_queue_spec.rb

@@ -10,11 +10,7 @@ describe "Server-named", AMQP::Queue do
 
   include EventedSpec::AMQPSpec
 
-  default_timeout 5
-
-  amqp_before do
-    @channel = AMQP::Channel.new
-  end
+  default_timeout 2
 
 
   #
@@ -22,21 +18,46 @@ describe "Server-named", AMQP::Queue do
   #
 
 
-  it "delays binding until after queue.declare-ok arrives" do
-    mailbox  = []
-    exchange = @channel.fanout("amq.fanout")
-    input    = "Independencia de resolución, ¿una realidad en Mac OS X Lion?"
+  context "bound to a pre-defined exchange" do
+    it "delays binding until after queue.declare-ok arrives" do
+      mailbox  = []
+      channel  = AMQP::Channel.new
+      exchange = channel.fanout("amq.fanout")
+      input    = "Independencia de resolución, ¿una realidad en Mac OS X Lion?"
+
+      channel.queue("", :auto_delete => true).bind(exchange).subscribe do |header, body|
+        mailbox << body
+      end
 
-    @channel.queue("", :auto_delete => true).bind(exchange).subscribe do |header, body|
-      mailbox << body
+      delayed(0.5) {
+        exchange.publish(input)
+      }
+
+      done(1.0) {
+        mailbox.size.should == 1
+      }
     end
+  end
 
-    delayed(0.3) {
-      exchange.publish(input)
-    }
 
-    done(0.5) {
-      mailbox.size.should == 1
-    }
+  context "bound to a pre-defined exchange" do
+    it "delays binding until after queue.declare-ok arrives" do
+      mailbox  = []
+      channel  = AMQP::Channel.new
+      exchange = channel.fanout("amqpgem.examples.exchanges.fanout", :durable => false)
+      input    = "Just send me already"
+
+      channel.queue("", :auto_delete => true).bind(exchange).subscribe do |header, body|
+        mailbox << body
+      end
+
+      delayed(0.5) {
+        exchange.publish(input)
+      }
+
+      done(1.0) {
+        mailbox.size.should == 1
+      }
+    end
   end
 end

data/spec/integration/declare_one_hundred_server_named_queues_spec.rb

@@ -0,0 +1,44 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe "Server-named", AMQP::Queue do
+
+  #
+  # Environment
+  #
+
+  include EventedSpec::AMQPSpec
+
+  default_timeout 5
+
+  amqp_before do
+    @channel = AMQP::Channel.new
+  end
+
+
+  #
+  # Examples
+  #
+
+
+  it "can be declared en masse" do
+    n       = 100
+    queues  = []
+
+    n.times do
+      queues << @channel.queue("", :auto_delete => true)
+    end
+
+    done(2.5) {
+      queues.size.should == n
+      # this is RabbitMQ-specific. But it is OK for now. MK.
+      queues.all? { |q| q.name =~ /^amq.*/ }.should be_true
+
+      # no duplicates. MK.
+      names = queues.map { |q| q.name }
+      names.uniq.size.should == n
+      names.uniq.should == names
+    }
+  end
+end

data/spec/integration/direct_exchange_routing_spec.rb

@@ -0,0 +1,125 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe "Multiple consumers bound to a queue with the same routing key" do
+
+  #
+  # Environment
+  #
+
+  include EventedSpec::AMQPSpec
+  default_options AMQP_OPTS
+  default_timeout 5
+
+  amqp_before do
+    @channel   = AMQP::Channel.new
+    @channel.on_error do |ch, close|
+      raise "Channel-level error!: #{close.inspect}"
+    end
+
+    @queue    = @channel.queue("", :auto_delete => true)
+    @exchange = @channel.direct("amqpgem.tests.integration.direct.exchange", :auto_delete => true)
+
+    @queue.bind(@exchange, :routing_key => "builds.all")
+  end
+
+
+
+  it "get messages distributed to them in a round-robin manner" do
+    mailbox1  = Array.new
+    mailbox2  = Array.new
+
+    consumer1 = AMQP::Consumer.new(@channel, @queue).consume
+    consumer2 = AMQP::Consumer.new(@channel, @queue).consume
+
+
+    consumer1.on_delivery do |metadata, payload|
+      mailbox1 << payload
+    end
+    consumer2.on_delivery do |metadata, payload|
+      mailbox2 << payload
+    end
+
+
+    EventMachine.add_timer(0.5) do
+      12.times { @exchange.publish(".", :routing_key => "builds.all") }
+      12.times { @exchange.publish(".", :routing_key => "all.builds") }
+    end
+
+    done(2.5) {
+      mailbox1.size.should == 6
+      mailbox2.size.should == 6
+    }
+  end
+end
+
+
+
+
+describe "Multiple queues bound to a direct exchange with the same routing key" do
+
+  #
+  # Environment
+  #
+
+  include EventedSpec::AMQPSpec
+  default_options AMQP_OPTS
+  default_timeout 5
+
+
+  amqp_before do
+    @channel   = AMQP::Channel.new
+    @channel.on_error do |ch, close|
+      raise "Channel-level error!: #{close.inspect}"
+    end
+
+    @queue1   = @channel.queue("", :auto_delete => true)
+    @queue2   = @channel.queue("", :auto_delete => true)
+    @exchange = @channel.direct("amqpgem.tests.integration.direct.exchange", :auto_delete => true)
+
+    @queue1.bind(@exchange, :routing_key => "builds.all")
+    @queue2.bind(@exchange, :routing_key => "builds.all")
+  end
+
+
+  it "all get a copy of messages with that routing key" do
+    mailbox1  = Array.new
+    mailbox2  = Array.new
+    mailbox3  = Array.new
+    mailbox4  = Array.new
+
+
+    consumer1 = AMQP::Consumer.new(@channel, @queue1).consume
+    consumer2 = AMQP::Consumer.new(@channel, @queue1).consume
+    consumer3 = AMQP::Consumer.new(@channel, @queue2).consume
+    consumer4 = AMQP::Consumer.new(@channel, @queue2).consume
+
+
+    consumer1.on_delivery do |metadata, payload|
+      mailbox1 << payload
+    end
+    consumer2.on_delivery do |metadata, payload|
+      mailbox2 << payload
+    end
+    consumer3.on_delivery do |metadata, payload|
+      mailbox3 << payload
+    end
+    consumer4.on_delivery do |metadata, payload|
+      mailbox4 << payload
+    end
+
+
+    EventMachine.add_timer(0.5) do
+      13.times { @exchange.publish(".", :routing_key => "builds.all") }
+      13.times { @exchange.publish(".", :routing_key => "all.builds") }
+    end
+
+    done(3.5) {
+      mailbox1.size.should == 7
+      mailbox2.size.should == 6
+      mailbox3.size.should == 7
+      mailbox4.size.should == 6
+    }
+  end
+end

data/spec/integration/exchange_declaration_spec.rb

@@ -28,7 +28,7 @@ describe AMQP::Channel do
       it 'declares a new direct exchange with that name' do
         @channel.direct('name').name.should == 'name'
 
-        done(0.3)
+        done
       end
 
       it "declares direct exchange as transient (non-durable)" do
@@ -37,7 +37,7 @@ describe AMQP::Channel do
         exchange.should_not be_durable
         exchange.should be_transient
 
-        done(0.3)
+        done
       end
 
       it "declares direct exchange as non-auto-deleted" do
@@ -45,7 +45,7 @@ describe AMQP::Channel do
 
         exchange.should_not be_auto_deleted
 
-        done(0.3)
+        done
       end
     end
 
@@ -53,7 +53,7 @@ describe AMQP::Channel do
     context "when exchange name is omitted" do
       it 'uses amq.direct' do
         @channel.direct.name.should == 'amq.direct'
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -66,7 +66,7 @@ describe AMQP::Channel do
     EOF
         @channel.direct("") do |exchange|
           exchange.name.should_not be_empty
-          done(0.3)
+          done
         end
       end
     end # context
@@ -82,7 +82,7 @@ describe AMQP::Channel do
 
           exchange.should == original_exchange
 
-          done(0.3)
+          done
         end # it
       end
 
@@ -94,7 +94,7 @@ describe AMQP::Channel do
             exchange = @channel.direct("direct exchange declared at #{Time.now.to_i}", :passive => true)
           }.to raise_error
 
-          done(0.3)
+          done
         end # it
       end # context
     end # context
@@ -106,7 +106,7 @@ describe AMQP::Channel do
         exchange.should be_durable
         exchange.should_not be_transient
 
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -117,7 +117,7 @@ describe AMQP::Channel do
         exchange.should_not be_durable
         exchange.should be_transient
 
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -127,7 +127,7 @@ describe AMQP::Channel do
         exchange = @channel.direct("a new auto-deleted direct exchange", :auto_delete => true)
 
         exchange.should be_auto_deleted
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -137,7 +137,7 @@ describe AMQP::Channel do
         exchange = @channel.direct("a new non-auto-deleted direct exchange", :auto_delete => false)
 
         exchange.should_not be_auto_deleted
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -147,7 +147,7 @@ describe AMQP::Channel do
         exchange = @channel.direct("a new non-auto-deleted direct exchange", :auto_delete => false)
 
         exchange.should_not be_auto_deleted
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -160,7 +160,7 @@ describe AMQP::Channel do
           @channel.direct("previously.declared.durable.direct.exchange", :durable => false)
         }.to raise_error(AMQP::IncompatibleOptionsError)
 
-        done(0.3)
+        done
       end # it
     end # context
   end # describe
@@ -177,7 +177,7 @@ describe AMQP::Channel do
 
         exchange.name.should == name
 
-        done(0.3)
+        done
       end
     end # context
 
@@ -187,7 +187,7 @@ describe AMQP::Channel do
         exchange.name.should == "amq.fanout"
         exchange.name.should_not == "amq.fanout2"
 
-        done(0.3)
+        done
       end
     end # context
 
@@ -201,7 +201,7 @@ describe AMQP::Channel do
 
           exchange.should == original_exchange
 
-          done(0.3)
+          done
         end # it
       end
 
@@ -213,7 +213,7 @@ describe AMQP::Channel do
             exchange = @channel.fanout("fanout exchange declared at #{Time.now.to_i}", :passive => true)
           }.to raise_error
 
-          done(0.3)
+          done
         end # it
       end # context
     end # context
@@ -225,7 +225,7 @@ describe AMQP::Channel do
         exchange.should be_durable
         exchange.should_not be_transient
 
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -236,7 +236,7 @@ describe AMQP::Channel do
         exchange.should_not be_durable
         exchange.should be_transient
 
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -246,7 +246,7 @@ describe AMQP::Channel do
         exchange = @channel.fanout("a new auto-deleted fanout exchange", :auto_delete => true)
 
         exchange.should be_auto_deleted
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -256,7 +256,7 @@ describe AMQP::Channel do
         exchange = @channel.fanout("a new non-auto-deleted fanout exchange", :auto_delete => false)
 
         exchange.should_not be_auto_deleted
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -266,7 +266,7 @@ describe AMQP::Channel do
         exchange = @channel.fanout("a new non-auto-deleted fanout exchange", :auto_delete => false)
 
         exchange.should_not be_auto_deleted
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -279,7 +279,7 @@ describe AMQP::Channel do
           @channel.fanout("previously.declared.durable.topic.exchange", :durable => false)
         }.to raise_error(AMQP::IncompatibleOptionsError)
 
-        done(0.3)
+        done
       end # it
     end # context
   end # describe
@@ -295,7 +295,7 @@ describe AMQP::Channel do
         exchange = @channel.topic(name)
         exchange.name.should == name
 
-        done(0.3)
+        done
       end
     end # context
 
@@ -305,7 +305,7 @@ describe AMQP::Channel do
         exchange.name.should == "amq.topic"
         exchange.name.should_not == "amq.topic2"
 
-        done(0.3)
+        done
       end
     end # context
 
@@ -319,7 +319,7 @@ describe AMQP::Channel do
 
           exchange.should == original_exchange
 
-          done(0.3)
+          done
         end # it
       end
 
@@ -331,7 +331,7 @@ describe AMQP::Channel do
             exchange = @channel.topic("topic exchange declared at #{Time.now.to_i}", :passive => true)
           }.to raise_error
 
-          done(0.3)
+          done
         end # it
       end # context
     end # context
@@ -343,7 +343,7 @@ describe AMQP::Channel do
         exchange.should be_durable
         exchange.should_not be_transient
 
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -354,7 +354,7 @@ describe AMQP::Channel do
         exchange.should_not be_durable
         exchange.should be_transient
 
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -364,7 +364,7 @@ describe AMQP::Channel do
         exchange = @channel.topic("a new auto-deleted topic exchange", :auto_delete => true)
 
         exchange.should be_auto_deleted
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -374,7 +374,7 @@ describe AMQP::Channel do
         exchange = @channel.topic("a new non-auto-deleted topic exchange", :auto_delete => false)
 
         exchange.should_not be_auto_deleted
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -384,7 +384,7 @@ describe AMQP::Channel do
         exchange = @channel.topic("a new non-auto-deleted topic exchange", :auto_delete => false)
 
         exchange.should_not be_auto_deleted
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -404,7 +404,7 @@ describe AMQP::Channel do
           channel.topic("previously.declared.durable.topic.exchange", :durable => false)
         }.to raise_error(AMQP::IncompatibleOptionsError)
 
-        done(0.3)
+        done
       end # it
     end # context
   end # describe
@@ -421,7 +421,7 @@ describe AMQP::Channel do
 
         exchange.name.should == name
 
-        done(0.3)
+        done
       end
     end # context
 
@@ -435,7 +435,7 @@ describe AMQP::Channel do
         exchange.name.should == "amq.match"
         exchange.name.should_not == "amq.headers"
 
-        done(0.3)
+        done
       end
     end # context
 
@@ -449,7 +449,7 @@ describe AMQP::Channel do
 
           exchange.should == original_exchange
 
-          done(0.3)
+          done
         end # it
       end
 
@@ -461,7 +461,7 @@ describe AMQP::Channel do
             exchange = @channel.headers("headers exchange declared at #{Time.now.to_i}", :passive => true)
           }.to raise_error
 
-          done(0.3)
+          done
         end # it
       end # context
     end # context
@@ -473,7 +473,7 @@ describe AMQP::Channel do
         exchange.should be_durable
         exchange.should_not be_transient
 
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -484,7 +484,7 @@ describe AMQP::Channel do
         exchange.should_not be_durable
         exchange.should be_transient
 
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -494,7 +494,7 @@ describe AMQP::Channel do
         exchange = @channel.headers("a new auto-deleted headers exchange", :auto_delete => true)
 
         exchange.should be_auto_deleted
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -504,7 +504,7 @@ describe AMQP::Channel do
         exchange = @channel.headers("a new non-auto-deleted headers exchange", :auto_delete => false)
 
         exchange.should_not be_auto_deleted
-        done(0.3)
+        done
       end # it
     end # context
 
@@ -514,24 +514,53 @@ describe AMQP::Channel do
         exchange = @channel.headers("a new non-auto-deleted headers exchange", :auto_delete => false)
 
         exchange.should_not be_auto_deleted
-        done(0.3)
+        done
       end # it
     end # context
 
 
-    context "when exchange is re-declared with parameters different from original declaration" do
+    context "when exchange is re-declared with parameters different from original declaration on the same channel" do
       amqp_after do
         done
       end
 
       it "raises an exception" do
-        @channel.headers("previously.declared.durable.topic.exchange", :durable => true)
+        @channel.headers("previously.declared.durable.headers.exchange", :durable => true)
 
         expect {
-          @channel.headers("previously.declared.durable.topic.exchange", :durable => false)
+          @channel.headers("previously.declared.durable.headers.exchange", :durable => false)
         }.to raise_error(AMQP::IncompatibleOptionsError)
 
-        done(0.3)
+        done
+      end # it
+    end # context
+
+
+    context "when exchange is re-declared with parameters different from original declaration on two separate channels" do
+      it "raises an exception" do
+        channel2 = AMQP::Channel.new
+        @channel.headers("previously.declared.durable.headers.exchange", :durable => true)
+
+        channel2.on_error do |ch, channel_close|
+          puts "reply_text: #{channel_close.reply_text}, reply_code: #{channel_close.reply_code}"
+          done
+        end
+        channel2.headers("previously.declared.durable.headers.exchange", :durable => false)
+      end # it
+    end # context
+
+
+
+    context "when exchange is re-declared with type different from original declaration on two separate channels" do
+      it "raises an exception" do
+        channel2 = AMQP::Channel.new
+        @channel.topic("previously.declared.durable.topic.exchange", :durable => true)
+
+        channel2.on_error do |ch, channel_close|
+          puts "reply_text: #{channel_close.reply_text}, reply_code: #{channel_close.reply_code}"
+          done
+        end
+        channel2.headers("previously.declared.durable.topic.exchange", :durable => true)
       end # it
     end # context
   end # describe