amqp 0.7.0.pre → 0.7.0

data/spec/unit/mq/misc_spec.rb

@@ -0,0 +1,228 @@
+require 'spec_helper'
+require 'mq_helper'
+
+describe 'MQ', 'class object' do
+#  after{AMQP.cleanup_state} # Tips off Thread.current[:mq] call
+  subject { MQ }
+
+  its(:logging) { should be_false }
+
+
+  describe '.logging=' do
+    it 'is independent from AMQP.logging' do
+      #
+      AMQP.logging = true
+      MQ.logging.should be_false
+      MQ.logging = false
+      AMQP.logging.should == true
+
+      #
+      AMQP.logging = false
+      MQ.logging = true
+      AMQP.logging.should be_false
+      MQ.logging = false
+    end
+  end # .logging=
+
+
+  describe '.default' do
+    it 'creates MQ object and stashes it in a Thread-local Hash' do
+      MQ.should_receive(:new).with(no_args).and_return('MQ mock')
+      Thread.current.should_receive(:[]).with(:mq)
+      Thread.current.should_receive(:[]=).with(:mq, 'MQ mock')
+      MQ.default.should == 'MQ mock'
+    end
+  end # .default
+
+
+
+  describe '.id' do
+    it 'returns Thread-local uuid for mq' do
+      uuid_pattern = /.*-[0-9]*-[0-9]*/
+      Thread.current.should_receive(:[]).with(:mq_id)
+      Thread.current.should_receive(:[]=).with(:mq_id, uuid_pattern)
+      MQ.id.should =~ uuid_pattern
+    end
+  end # .id
+
+
+
+  describe '.error' do
+    after(:all) { MQ.instance_variable_set(:@error_callback, nil) } # clear error callback
+
+    it 'is noop unless @error_callback was previously set' do
+      MQ.instance_variable_get(:@error_callback).should be_nil
+      MQ.error("Msg").should be_nil
+    end
+
+    context 'when @error_callback is set' do
+      before { MQ.error { |message| message.should == "Msg"; @callback_fired = true } }
+
+      it 'is setting @error_callback to given block' do
+        MQ.instance_variable_get(:@error_callback).should_not be_nil
+        @callback_fired.should be_false
+      end
+
+      it 'does not fire @error_callback immediately' do
+        @callback_fired.should be_false
+      end
+
+      it 'fires @error_callback on subsequent .error calls' do
+        MQ.error("Msg")
+        @callback_fired.should be_true
+      end
+    end
+  end # .error
+
+
+
+  describe '.method_missing' do
+    it 'routes all unrecognized methods to MQ.default' do
+      MQ.should_receive(:new).with(no_args).and_return(mock('channel'))
+      MQ.default.should_receive(:foo).with("Msg")
+      MQ.foo("Msg")
+    end
+  end # .method_missing
+end # describe 'MQ as a class'
+
+
+
+
+describe 'Channel object' do
+  context 'when initialized with a mock connection' do
+    before { @client = MockConnection.new }
+    after { AMQP.cleanup_state }
+    subject { MQ.new(@client).tap { |mq| mq.succeed } } # Indicates that channel is connected
+
+    it 'should have a channel' do
+      subject.channel.should be_kind_of Fixnum
+      subject.channel.should == 1 # Essentially, this channel (mq) number
+    end
+
+    it 'has publicly accessible collections' do
+      subject.exchanges.should be_empty
+      subject.queues.should be_empty
+      subject.consumers.should be_empty
+      subject.rpcs.should be_empty
+    end
+
+    describe '#send' do
+      it 'sends given data through its connection' do
+        args = [ 'data1', 'data2', 'data3']
+        subject.send *args
+        @client.messages[-3..-1].map { |message| message[:data] }.should == args
+      end #send
+
+      it 'sets data`s ticket property if @ticket is set for MQ object' do
+        ticket = subject_mock(:@ticket)
+        data = mock('data')
+        data.should_receive(:ticket=).with(ticket)
+        subject.send data
+      end
+    end
+
+
+
+    describe '#reset' do
+      it 'resets and reinitializes the channel, clears and resets its queues/exchanges' do
+        subject.queue('test').should_receive(:reset)
+        subject.fanout('fanout').should_receive(:reset)
+        subject.should_receive(:initialize).with(@client)
+
+        subject.reset
+        subject.queues.should be_empty
+        subject.exchanges.should be_empty
+        subject.consumers.should be_empty
+      end
+    end #reset
+
+
+
+    describe '#prefetch', 'setting :prefetch_count for broker' do
+      it 'sends Protocol::Basic::Qos' do
+        subject.prefetch 13
+        @client.messages.last[:data].should be_an AMQP::Protocol::Basic::Qos
+        @client.messages.last[:data].instance_variable_get(:@prefetch_count).should == 13
+      end
+
+      it 'returns MQ object itself, allowing for method chains' do
+        subject.prefetch(1).should == subject
+      end
+    end #prefetch
+
+
+
+    describe '#recover', "asks broker to redeliver unack'ed messages on this channel" do
+      it "sends Basic::Recover through @connection" do
+        subject.recover
+        @client.messages.last[:data].should be_an AMQP::Protocol::Basic::Recover
+      end
+
+      it 'defaults to :requeue=nil, redeliver messages to the original recipient' do
+        subject.recover
+        @client.messages.last[:data].instance_variable_get(:@requeue).should == nil
+      end #prefetch
+
+      it 'prompts broker to requeue messages (to other subs) if :requeue set to true' do
+        subject.recover true
+        @client.messages.last[:data].instance_variable_get(:@requeue).should == true
+      end
+
+      it 'returns MQ object itself, allowing for method chains' do
+        subject.recover.should == subject
+      end
+    end #recover
+
+
+
+    describe '#close', 'closes channel' do
+      it 'can be simplified, getting rid of @closing ivar?'
+      # TODO: Just set a callback sending Protocol::Channel::Close...'
+
+      it 'sends Channel::Close through @connection' do
+        subject.close
+        @client.messages.last[:data].should be_an AMQP::Protocol::Channel::Close
+      end
+
+      it 'does not actually delete the channel or close connection' do
+        @client.channels.should_not_receive(:delete)
+        @client.should_not_receive(:close)
+        subject.close
+      end
+
+      it 'actual channel closing happens ONLY when Channel::CloseOk is received' do
+        @client.channels.should_receive(:delete) do |key|
+          key.should == 1
+          @client.channels.clear
+        end
+        @client.should_receive(:close)
+        subject.process_frame AMQP::Frame::Method.new(AMQP::Protocol::Channel::CloseOk.new)
+      end
+    end #close
+
+
+
+
+    describe '#get_queue', 'supports #pop (Basic::Get) operation on queues' do
+      it 'yields a FIFO queue to a given block' do
+        subject.get_queue do |fifo|
+          @block_called = true
+          fifo.should == []
+        end
+        @block_called.should be_true
+      end
+
+      it 'FIFO queue contains consumers that called Queue#pop' do
+        queue1 = subject.queue('test1')
+        queue2 = subject.queue('test2')
+        queue1.pop
+        queue2.pop
+        subject.get_queue do |fifo|
+          fifo.should have(2).consumers
+          fifo.first.should == queue1
+          fifo.last.should == queue2
+        end
+      end
+    end #get_queue
+  end #  context 'when initialized with a mock connection'
+end # describe MQ object, also vaguely known as "channel"

data/spec/unit/mq/mq_basic_spec.rb

@@ -0,0 +1,39 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe MQ do
+
+  #
+  # Environment
+  #
+
+  include AMQP::Spec
+
+  default_timeout 5
+
+  amqp_before do
+    @channel = MQ.new
+  end
+
+
+  #
+  # Examples
+  #
+
+
+  describe ".channel" do
+    it 'gives each thread a separate channel' do
+      pending 'This is not implemented in current lib'
+      class MQ
+        @@cur_channel = 0
+      end
+
+      described_class.channel.should == 1
+
+      Thread.new { described_class.channel }.value.should == 2
+      Thread.new { described_class.channel }.value.should == 3
+      done
+    end
+  end
+end # describe MQ

data/spec/unit/mq/queue_declaration_spec.rb

@@ -0,0 +1,97 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe MQ do
+
+  #
+  # Environment
+  #
+
+  include AMQP::Spec
+
+  default_timeout 5
+
+  amqp_before do
+    @channel = MQ.new
+  end
+
+
+  #
+  # Examples
+  #
+
+  describe "#queue" do
+    context "when queue name is specified" do
+      let(:name) { "a queue declared at #{Time.now.to_i}" }
+
+      it "declares a new queue with that name" do
+        queue = @channel.queue(name)
+        queue.name.should == name
+
+        done
+      end
+
+      it "caches that queue" do
+        queue = @channel.queue(name)
+        @channel.queue(name).object_id.should == queue.object_id
+
+        done
+      end
+    end # context
+
+    context "when queue name is passed on as an empty string" do
+      it "uses server-assigned queue name" do
+        @channel.queue("") do |queue, *args|
+          queue.name.should_not be_empty
+          done
+        end
+      end
+    end # context
+
+
+
+
+    context "when passive option is used" do
+      context "and exchange with given name already exists" do
+        it "silently returns" do
+          name = "a_new_queue declared at #{Time.now.to_i}"
+
+          original_exchange = @channel.queue(name)
+          exchange          = @channel.queue(name, :passive => true)
+
+          exchange.should == original_exchange
+
+          done
+        end # it
+      end
+
+      context "and exchange with given name DOES NOT exist" do
+        it "raises an exception" do
+          pending "Not yet supported"
+
+          expect {
+            exchange = @channel.queue("queue declared at #{Time.now.to_i}", :passive => true)
+          }.to raise_error
+
+          done
+        end # it
+      end # context
+    end # context
+
+
+
+
+    context "when queue is re-declared with parameters different from original declaration" do
+      it "raises an exception" do
+        @channel.queue("previously.declared.durable.queue", :durable => true)
+
+        expect {
+          @channel.queue("previously.declared.durable.queue", :durable => false)
+        }.to raise_error(MQ::IncompatibleOptionsError)
+
+        done
+      end # it
+    end # context
+  end # describe
+end # describe MQ

data/spec/unit/mq/queue_spec.rb

@@ -0,0 +1,71 @@
+require 'spec_helper'
+require 'mq_helper'
+
+describe MQ::Queue, :broker => true do
+
+  #
+  # Environment
+  #
+
+  include AMQP::Spec
+
+  default_options AMQP_OPTS
+  default_timeout 3
+
+  amqp_before do
+    @channel = MQ.new
+    @queue   = MQ::Queue.new(@channel, 'test_queue', :option => 'useless')
+  end
+
+  amqp_after { @queue.purge; AMQP.cleanup_state }
+
+
+  #
+  # Examples
+  #
+
+  context 'declared with name of "test_queue"' do
+    amqp_after {@queue.unsubscribe; @queue.delete}
+    # TODO: Fix amqp_after: it is NOT run in case of raised exceptions, it seems
+
+    it 'supports asynchronous subscription to broker-predefined amq.direct exchange' do
+      data = "data sent via amq.direct exchange"
+
+      @queue.subscribe do |header, message|
+        header.should be_an MQ::Header
+        message.should == data
+        done
+      end
+      @queue.publish(data)
+    end # it
+
+    it 'supports synchronous message fetching via #pop (Basic.Get)' do
+      @queue.publish('send some data down the pipe')
+
+      @queue.pop do |header, message|
+        header.should be_an MQ::Header
+        message.should == 'send some data down the pipe'
+        done
+      end
+    end # it
+
+    xit 'asynchronously receives own status from the broker' do
+      pending "TODO: this example currently fails; do investigate why. MK."
+      total_number_of_messages = 123
+
+      total_number_of_messages.times do |n|
+        @queue.publish('message # #{n}')
+      end
+
+      @on_status_fired = false
+      @queue.status do |messages, consumers|
+        @on_status_fired = true
+        messages.should == total_number_of_messages
+        consumers.should == 0
+      end # it
+
+      @on_status_fired.should be_true
+      done(1)
+    end # it
+  end # context
+end # MQ::Queue, 'with real connection

metadata

@@ -1,13 +1,12 @@
 --- !ruby/object:Gem::Specification 
 name: amqp
 version: !ruby/object:Gem::Version 
-  prerelease: true
+  prerelease: false
   segments: 
   - 0
   - 7
   - 0
-  - pre
-  version: 0.7.0.pre
+  version: 0.7.0
 platform: ruby
 authors: 
 - Aman Gupta
@@ -15,7 +14,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: 
-date: 2011-01-05 00:00:00 +00:00
+date: 2011-01-18 00:00:00 +00:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -50,14 +49,16 @@ extra_rdoc_files:
 - doc/EXAMPLE_06_HASHTABLE
 files: 
 - .gitignore
+- .rspec
 - CHANGELOG
+- CONTRIBUTORS
 - Gemfile
-- Gemfile.lock
 - README.md
 - Rakefile
 - amqp.gemspec
 - amqp.pre.gemspec
 - amqp.todo
+- bin/cleanify.rb
 - bin/irb
 - doc/EXAMPLE_01_PINGPONG
 - doc/EXAMPLE_02_CLOCK
@@ -68,6 +69,7 @@ files:
 - doc/EXAMPLE_06_HASHTABLE
 - examples/amqp/simple.rb
 - examples/mq/ack.rb
+- examples/mq/automatic_binding_for_default_direct_exchange.rb
 - examples/mq/callbacks.rb
 - examples/mq/clock.rb
 - examples/mq/hashtable.rb
@@ -97,12 +99,6 @@ files:
 - lib/mq/logger.rb
 - lib/mq/queue.rb
 - lib/mq/rpc.rb
-- old/README
-- old/Rakefile
-- old/amqp-0.8.json
-- old/amqp_spec.rb
-- old/amqpc.rb
-- old/codegen.rb
 - protocol/amqp-0.8.json
 - protocol/amqp-0.8.xml
 - protocol/codegen.rb
@@ -111,17 +107,35 @@ files:
 - research/primes-forked.rb
 - research/primes-processes.rb
 - research/primes-threaded.rb
-- spec/channel_close_spec.rb
+- spec/integration/automatic_binding_for_default_direct_exchange_spec.rb
+- spec/mq_helper.rb
 - spec/spec_helper.rb
-- spec/sync_async_spec.rb
+- spec/unit/amqp/buffer_spec.rb
+- spec/unit/amqp/client_spec.rb
+- spec/unit/amqp/frame_spec.rb
+- spec/unit/amqp/misc_spec.rb
+- spec/unit/amqp/protocol_spec.rb
+- spec/unit/mq/channel_close_spec.rb
+- spec/unit/mq/collection_spec.rb
+- spec/unit/mq/exchange_declaration_spec.rb
+- spec/unit/mq/misc_spec.rb
+- spec/unit/mq/mq_basic_spec.rb
+- spec/unit/mq/queue_declaration_spec.rb
+- spec/unit/mq/queue_spec.rb
 has_rdoc: true
-homepage: http://github.com/tmm1/amqp
+homepage: http://github.com/ruby-amqp/amqp
 licenses: []
 
 post_install_message: "[\e[32mVersion 0.7\e[0m] [BUG] Sync API for queues and exchanges, support for server-generated queues & exchange names (via semi-lazy collection).\n\
   [\e[32mVersion 0.7\e[0m] [BUG] Sync API for MQ#close (Channel.Close) [issue #34].\n\
-  [\e[32mVersion 0.7\e[0m] [FEATURE] From majek's fork, with some fixes. Example: AMQP.start(\"amqps://\")\n\
-  [\e[32mVersion 0.7\e[0m] [DEVELOP] some em-spec-based specs, bin/irb, Gemfile.\n"
+  [\e[32mVersion 0.7\e[0m] [FEATURE] AMQP URL from majek's fork, with some fixes. Example: AMQP.start(\"amqps://\")\n\
+  [\e[32mVersion 0.7\e[0m] [DEVELOP] Added some em-spec-based specs, bin/irb, Gemfile.\n\
+  [\e[32mVersion 0.7\e[0m] [FEATURE] Added MQ::Exchange.default for the default exchange.\n\
+  [\e[32mVersion 0.7\e[0m] [FEATURE] Raise an exception if we're trying to use Basic.Reject with RabbitMQ.\n\
+  [\e[32mVersion 0.7\e[0m] [FEATURE] Fail if an entity is re-declared with different options.\n\
+  [\e[32mVersion 0.7\e[0m] [BUG] Don't reconnect if the credentials are wrong.\n\
+  [\e[32mVersion 0.7\e[0m] [BUG] Fixed an exception which occurred when Queue#bind was called synchronously with a callback.\n\
+  [\e[32mVersion 0.7\e[0m] [DEVELOPMENT] Added a lot of specs (Bacon replaced by rSpec 2).\n"
 rdoc_options: 
 - --include=examples
 require_paths: 
@@ -137,13 +151,11 @@ required_ruby_version: !ruby/object:Gem::Requirement
 required_rubygems_version: !ruby/object:Gem::Requirement 
   none: false
   requirements: 
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version 
       segments: 
-      - 1
-      - 3
-      - 1
-      version: 1.3.1
+      - 0
+      version: "0"
 requirements: []
 
 rubyforge_project: amqp