queueing_rabbit 0.2.1 → 0.3.0

data/lib/queueing_rabbit.rb

@@ -1,4 +1,5 @@
 require "queueing_rabbit/version"
+require "queueing_rabbit/misc/inheritable_class_variables"
 require "queueing_rabbit/callbacks"
 require "queueing_rabbit/configuration"
 require "queueing_rabbit/logging"
@@ -9,6 +10,9 @@ require "queueing_rabbit/client/bunny"
 require "queueing_rabbit/extensions/new_relic"
 require "queueing_rabbit/extensions/retryable"
 require "queueing_rabbit/extensions/direct_exchange"
+require "queueing_rabbit/bus"
+require "queueing_rabbit/buses/abstract_bus"
+require "queueing_rabbit/buses/json_bus"
 require "queueing_rabbit/job"
 require "queueing_rabbit/jobs/abstract_job"
 require "queueing_rabbit/jobs/json_job"
@@ -49,15 +53,29 @@ module QueueingRabbit
 
     true
   end
-  alias_method :publish, :enqueue
+
+  def publish(bus, payload = nil, options = {})
+    info "publishing to event bus #{bus}"
+
+    follow_bus_requirements(bus) do |ch, ex|
+      conn.enqueue(ex, payload, options)
+      ch.close
+    end
+  end
 
   def follow_job_requirements(job)
-    conn.open_channel(job.channel_options) do |ch, _|
-      conn.define_exchange(ch, job.exchange_name, job.exchange_options) do |ex|
-        conn.define_queue(ch, job.queue_name, job.queue_options) do |q|
-          conn.bind_queue(q, ex, job.binding_options) if job.bind_queue?
-          yield ch, ex, q if block_given?
-        end
+    follow_bus_requirements(job) do |ch, ex|
+      conn.define_queue(ch, job.queue_name, job.queue_options) do |q|
+        conn.bind_queue(q, ex, job.binding_options) if job.bind_queue?
+        yield ch, ex, q
+      end
+    end
+  end
+
+  def follow_bus_requirements(bus)
+    conn.open_channel(bus.channel_options) do |ch, _|
+      conn.define_exchange(ch, bus.exchange_name, bus.exchange_options) do |ex|
+        yield ch, ex
       end
     end
   end

data/lib/queueing_rabbit/bus.rb

@@ -0,0 +1,65 @@
+module QueueingRabbit
+
+  module Bus
+    def self.extended(othermod)
+      othermod.extend(QueueingRabbit::InheritableClassVariables)
+
+      othermod.class_eval do
+        inheritable_variables :channel_options, :exchange_name,
+                              :exchange_options, :publishing_defaults
+      end
+    end
+
+    def channel(options = {})
+      @channel_options ||= {}
+      @channel_options.update(options)
+    end
+
+    def channel_options
+      @channel_options ||= {}
+    end
+
+    def exchange(*args)
+      @exchange_options ||= {}
+      name, options = extract_name_and_options(*args)
+      @exchange_name = name if name
+      @exchange_options.update(options) if options
+    end
+
+    def exchange_name
+      @exchange_name || ''
+    end
+
+    def exchange_options
+      @exchange_options || {}
+    end
+
+    def publish_with(options = {})
+      @publishing_defaults ||= {}
+      @publishing_defaults.update(options)
+    end
+
+    def publishing_defaults
+      @publishing_defaults || {}
+    end
+
+    def publish(payload, options = {})
+      QueueingRabbit.publish(self, payload, publishing_defaults.merge(options))
+    end
+
+    protected
+
+    def extract_name_and_options(*args)
+      name = options = nil
+      if args.first.kind_of?(Hash)
+        options = args.first
+      elsif args.count > 1
+        name, options = args
+      else
+        name = args.first
+      end
+      [name, options]
+    end
+  end
+
+end

data/lib/queueing_rabbit/buses/abstract_bus.rb

@@ -0,0 +1,9 @@
+module QueueingRabbit
+
+  class AbstractBus
+
+    extend Bus
+
+  end
+
+end

data/lib/queueing_rabbit/buses/json_bus.rb

@@ -0,0 +1,13 @@
+module QueueingRabbit
+
+  class JSONBus < AbstractBus
+
+    extend QueueingRabbit::Serializer
+
+    def self.publish(payload, metadata = {})
+      super serialize(payload), metadata
+    end
+
+  end
+
+end

data/lib/queueing_rabbit/configuration.rb

@@ -1,7 +1,7 @@
 module QueueingRabbit
 
   module Configuration
-    attr_accessor :amqp_uri, :amqp_exchange_name, :amqp_exchange_options
+    attr_accessor :amqp_uri
     attr_writer :tcp_timeout, :heartbeat
 
     def configure

data/lib/queueing_rabbit/job.rb

@@ -1,23 +1,8 @@
 module QueueingRabbit
-  module InheritableClassVariables
-    def inheritable_variables(*args)
-      @inheritable_variables ||= [:inheritable_variables]
-      @inheritable_variables += args
-    end
-
-    def inherited(subclass)
-      @inheritable_variables ||= []
-      @inheritable_variables.each do |var|
-        if !subclass.instance_variable_get("@#{var}") ||
-           subclass.instance_variable_get("@#{var}").empty?
-          subclass.instance_variable_set("@#{var}",
-                                         instance_variable_get("@#{var}"))
-        end
-      end
-    end
-  end
 
   module Job
+    include Bus
+
     def self.extended(othermod)
       othermod.extend(QueueingRabbit::InheritableClassVariables)
 
@@ -31,15 +16,9 @@ module QueueingRabbit
 
     def queue(*args)
       @queue_options ||= {}
-      if args.first.kind_of?(Hash)
-        @queue_options.update(args.first)
-      elsif args.count > 1
-        name, options = args
-        @queue_name = name
-        @queue_options.update(options)
-      else
-        @queue_name = args.first
-      end
+      name, options = extract_name_and_options(*args)
+      @queue_name = name if name
+      @queue_options.update(options) if options
     end
 
     def queue_name
@@ -54,36 +33,6 @@ module QueueingRabbit
       QueueingRabbit.queue_size(self)
     end
 
-    def channel(options = {})
-      @channel_options ||= {}
-      @channel_options.update(options)
-    end
-
-    def channel_options
-      @channel_options ||= {}
-    end
-
-    def exchange(*args)
-      @exchange_options ||= {}
-      if args.first.kind_of?(Hash)
-        @exchange_options.update(args.first)
-      elsif args.count > 1
-        name, options = args
-        @exchange_name = name
-        @exchange_options.update(options)
-      else
-        @exchange_name = args.first
-      end
-    end
-
-    def exchange_name
-      @exchange_name || ''
-    end
-
-    def exchange_options
-      @exchange_options || {}
-    end
-
     def bind(options = {})
       @binding_options ||= {}
       @binding_options.update(options)
@@ -107,11 +56,6 @@ module QueueingRabbit
     end
     alias_method :subscribe, :listen
 
-    def publish(options = {})
-      @publishing_defaults ||= {}
-      @publishing_defaults.update(options)
-    end
-
     def publishing_defaults
       @publishing_defaults ||= {}
       {:routing_key => queue_name.to_s}.merge(@publishing_defaults)

data/lib/queueing_rabbit/misc/inheritable_class_variables.rb

@@ -0,0 +1,23 @@
+module QueueingRabbit
+
+  module InheritableClassVariables
+    
+    def inheritable_variables(*args)
+      @inheritable_variables ||= [:inheritable_variables]
+      @inheritable_variables += args
+    end
+
+    def inherited(subclass)
+      @inheritable_variables ||= []
+      @inheritable_variables.each do |var|
+        if !subclass.instance_variable_get("@#{var}") ||
+           subclass.instance_variable_get("@#{var}").empty?
+          subclass.instance_variable_set("@#{var}",
+                                         instance_variable_get("@#{var}"))
+        end
+      end
+    end
+
+  end
+
+end

data/lib/queueing_rabbit/version.rb

@@ -1,3 +1,3 @@
 module QueueingRabbit
-  VERSION = "0.2.1"
+  VERSION = "0.3.0"
 end

data/spec/integration/binary_synchronous_publishing_via_bus_and_asynchronous_consuming_via_job_spec.rb

@@ -0,0 +1,33 @@
+require 'spec_helper'
+require 'integration/jobs/print_line_job'
+
+describe 'Binary synchronous publishing via bus and asynchronous consuming via job' do
+  include_context "StringIO logger"
+  include_context "Evented spec"
+  
+  let(:bus) {
+    Class.new(QueueingRabbit::AbstractBus) do
+      publish_with :routing_key => 'print_line_job'
+    end
+  }
+  let(:job) { PrintLineJob }
+  let(:worker) { QueueingRabbit::Worker.new(job.to_s) }
+  let(:line) { "Hello, world!" }
+  let(:io) { StringIO.new }
+
+  before do
+    job.io = io
+    bus.publish(line)
+    QueueingRabbit.drop_connection
+  end
+
+  it "works" do
+    em {
+      worker.work
+
+      done(1.0) {
+        io.string.should include(line)
+      }
+    }
+  end
+end

data/spec/integration/configuration_spec.rb

@@ -8,7 +8,7 @@ describe 'Configuring jobs' do
         channel :prefetch => 15
         queue :durable => true
         bind :ack => true
-        publish :persistent => true
+        publish_with :persistent => true
       end
     }
 
@@ -35,7 +35,7 @@ describe 'Configuring jobs' do
         channel :prefetch => 15
         queue :durable => true
         bind :ack => true
-        publish :persistent => true
+        publish_with :persistent => true
       end
     }
 

data/spec/integration/json_synchronous_publishing_via_bus_and_asynchronous_consuming_via_job_spec.rb

@@ -0,0 +1,50 @@
+require 'spec_helper'
+require 'integration/jobs/print_line_job'
+
+describe 'JSON synchronous publishing via bus and asynchronous consuming via job' do
+  include_context "StringIO logger"
+  include_context "Evented spec"
+  
+  let(:bus) {
+    Class.new(QueueingRabbit::JSONBus) do
+      publish_with :routing_key => 'print_line_job'
+    end
+  }
+  let(:job) {
+    Class.new(QueueingRabbit::JSONJob) do
+      class << self
+        attr_accessor :io
+      end
+
+      queue 'print_line_job'
+
+      def perform
+        self.class.io.puts arguments[:line]
+      end
+    end
+  }
+  let(:job_name) { 'PrintLineJob' }
+  let(:worker) { QueueingRabbit::Worker.new(job_name) }
+  let(:line) { "Hello, world!" }
+  let(:io) { StringIO.new }
+
+  before do
+    stub_const(job_name, job)
+  end
+
+  before do
+    job.io = io
+    bus.publish(:line => line)
+    QueueingRabbit.drop_connection
+  end
+
+  it "works" do
+    em {
+      worker.work
+
+      done(1.0) {
+        io.string.should include(line)
+      }
+    }
+  end
+end

data/spec/integration/persistent_asynchronous_publishing_and_consuming_spec.rb

@@ -15,7 +15,7 @@ describe "Persistent asynchronous publishing and consuming" do
       Class.new(PrintLineJob) do
         queue 'persistent_print_line_job'
         listen :ack => true
-        publish :persistent => true
+        publish_with :persistent => true
         channel :use_publisher_confirms => true
 
         def perform

data/spec/spec_helper.rb

@@ -20,10 +20,4 @@ RSpec.configure do |config|
   config.exclusion_filter = {
     :ruby => RUBY_VERSION
   }
-
-  QueueingRabbit.configure do |qr|
-    qr.amqp_uri = "amqp://guest:guest@localhost:5672"
-    qr.amqp_exchange_name = "queueing_rabbit_test"
-    qr.amqp_exchange_options = {:durable => true}
-  end
 end

data/spec/unit/queueing_rabbit/buses/abstract_bus_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe QueueingRabbit::AbstractBus do
+  let(:bus_class) {
+    Class.new(QueueingRabbit::AbstractBus) do
+      exchange 'test_exchange', :durable => false
+      publish_with :routing_key => 'test_queue'
+    end
+  }
+
+  subject { bus_class }
+
+  it { should respond_to(:exchange).with(1).argument }
+  it { should respond_to(:exchange).with(2).arguments }
+  it { should respond_to(:exchange_name) }
+  it { should respond_to(:exchange_options) }
+  it { should respond_to(:channel_options) }
+  it { should respond_to(:channel).with(1).argument }
+  it { should respond_to(:publishing_defaults) }
+
+  its(:exchange_name) { should == 'test_exchange' }
+  its(:exchange_options) { should include(:durable => false) }
+  its(:publishing_defaults) { should include(:routing_key => 'test_queue') }
+  
+end

data/spec/unit/queueing_rabbit/buses/json_bus_spec.rb

@@ -0,0 +1,16 @@
+require 'spec_helper'
+
+describe QueueingRabbit::JSONBus do
+  let(:json_bus) { QueueingRabbit::JSONBus }
+  let(:payload) { JSON.dump(:foo => 'bar') }
+
+  describe '.publish' do
+    let(:options) { {:persistent => true} }
+
+    it 'dumps payload to JSON' do
+      QueueingRabbit.should_receive(:publish).
+                     with(json_bus, payload, options)
+      json_bus.publish({:foo => 'bar'}, options)
+    end
+  end
+end

data/spec/unit/queueing_rabbit/client/amqp_spec.rb

@@ -7,9 +7,7 @@ describe QueueingRabbit::Client::AMQP do
   let(:connection) { mock :on_tcp_connection_loss => nil, :on_recovery => nil }
 
   before do
-    QueueingRabbit.stub(:amqp_uri => 'amqp://localhost:5672',
-                        :amqp_exchange_name => 'queueing_rabbit_test',
-                        :amqp_exchange_options => {:durable => true})
+    QueueingRabbit.stub(:amqp_uri => 'amqp://localhost:5672')
   end
 
   context "class" do

data/spec/unit/queueing_rabbit/client/bunny_spec.rb

@@ -4,9 +4,7 @@ describe QueueingRabbit::Client::Bunny do
   let(:connection) { stub(:start => true) }
 
   before do
-    QueueingRabbit.stub(:amqp_uri => 'amqp://localhost:5672',
-                        :amqp_exchange_name => 'queueing_rabbit_test',
-                        :amqp_exchange_options => {:durable => true})
+    QueueingRabbit.stub(:amqp_uri => 'amqp://localhost:5672')
   end
 
   context 'class' do

data/spec/unit/queueing_rabbit/configuration_spec.rb

@@ -4,9 +4,6 @@ describe QueueingRabbit::Configuration do
   subject { Class.new { extend QueueingRabbit::Configuration} }
 
   it { should respond_to(:amqp_uri) }
-  it { should respond_to(:amqp_exchange_name) }
-  it { should respond_to(:amqp_exchange_options) }
-
   its(:tcp_timeout) { should == 1 }
   its(:heartbeat) { should == 10 }
   its(:default_client) { should == QueueingRabbit::Client::Bunny }

data/spec/unit/queueing_rabbit_spec.rb

@@ -74,13 +74,11 @@ describe QueueingRabbit do
       subject.instance_variable_set(:@connection, connection)
     end
 
-    it 'opens a channel, defines an exchange and a queue, binds the queue to ' \
-       'the exchange, yields' do
-      connection.should_receive(:open_channel).with(job.channel_options).
-                                        and_yield(channel, nil)
-      connection.should_receive(:define_exchange).
-                 with(channel, job.exchange_name, job.exchange_options).
-                 and_yield(exchange)
+    it 'follows bus requirements, creates a queue, binds the queue to ' \
+       'the exchange and yields' do
+      subject.should_receive(:follow_bus_requirements).
+              with(job).
+              and_yield(channel, exchange)
       connection.should_receive(:define_queue).
                  with(channel, job.queue_name, job.queue_options).
                  and_yield(queue)
@@ -95,6 +93,33 @@ describe QueueingRabbit do
     end
   end
 
+  describe '.follow_bus_requirements' do
+    let(:channel) { mock }
+    let(:exchange) { mock }
+    let(:bus) {
+      stub(:channel_options => channel_options,
+           :exchange_name => exchange_name,
+           :exchange_options => exchange_options)
+    }
+
+    before do
+      subject.instance_variable_set(:@connection, connection)
+    end
+
+    it 'opens a channel, defines an exchange and yields' do
+      connection.should_receive(:open_channel).with(bus.channel_options).
+                                        and_yield(channel, nil)
+      connection.should_receive(:define_exchange).
+                 with(channel, bus.exchange_name, bus.exchange_options).
+                 and_yield(exchange)
+
+      subject.follow_bus_requirements(bus) do |ch, ex|
+        ch.should == channel
+        ex.should == exchange
+      end
+    end
+  end
+
   describe ".queue_size" do
     let(:size) { mock }
     let(:queue) { mock }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: queueing_rabbit
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-02 00:00:00.000000000 Z
+date: 2013-09-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: amqp
@@ -101,6 +101,9 @@ files:
 - README.md
 - Rakefile
 - lib/queueing_rabbit.rb
+- lib/queueing_rabbit/bus.rb
+- lib/queueing_rabbit/buses/abstract_bus.rb
+- lib/queueing_rabbit/buses/json_bus.rb
 - lib/queueing_rabbit/callbacks.rb
 - lib/queueing_rabbit/client/amqp.rb
 - lib/queueing_rabbit/client/bunny.rb
@@ -113,6 +116,7 @@ files:
 - lib/queueing_rabbit/jobs/abstract_job.rb
 - lib/queueing_rabbit/jobs/json_job.rb
 - lib/queueing_rabbit/logging.rb
+- lib/queueing_rabbit/misc/inheritable_class_variables.rb
 - lib/queueing_rabbit/serializer.rb
 - lib/queueing_rabbit/tasks.rb
 - lib/queueing_rabbit/version.rb
@@ -121,15 +125,19 @@ files:
 - queueing_rabbit.gemspec
 - spec/integration/asynchronous_publishing_and_consuming_spec.rb
 - spec/integration/asynchronous_publishing_and_consuming_with_retries_spec.rb
+- spec/integration/binary_synchronous_publishing_via_bus_and_asynchronous_consuming_via_job_spec.rb
 - spec/integration/configuration_spec.rb
 - spec/integration/direct_exchange_asynchronous_publishing_and_consuming_spec.rb
 - spec/integration/jobs/print_line_job.rb
 - spec/integration/json_job_asynchronous_publishing_and_consuming_spec.rb
+- spec/integration/json_synchronous_publishing_via_bus_and_asynchronous_consuming_via_job_spec.rb
 - spec/integration/persistent_asynchronous_publishing_and_consuming_spec.rb
 - spec/integration/synchronous_publishing_and_asynchronous_consuming_spec.rb
 - spec/integration/synchronous_publishing_spec.rb
 - spec/spec_helper.rb
 - spec/support/shared_contexts.rb
+- spec/unit/queueing_rabbit/buses/abstract_bus_spec.rb
+- spec/unit/queueing_rabbit/buses/json_bus_spec.rb
 - spec/unit/queueing_rabbit/callbacks_spec.rb
 - spec/unit/queueing_rabbit/client/amqp_spec.rb
 - spec/unit/queueing_rabbit/client/bunny_spec.rb
@@ -138,7 +146,6 @@ files:
 - spec/unit/queueing_rabbit/extensions/direct_exchange_spec.rb
 - spec/unit/queueing_rabbit/extensions/new_relic_spec.rb
 - spec/unit/queueing_rabbit/extensions/retryable_spec.rb
-- spec/unit/queueing_rabbit/job_spec.rb
 - spec/unit/queueing_rabbit/jobs/abstract_job_spec.rb
 - spec/unit/queueing_rabbit/jobs/json_job_spec.rb
 - spec/unit/queueing_rabbit/logging_spec.rb
@@ -160,7 +167,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -615049155772679483
+      hash: 1557293148620146716
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -169,7 +176,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -615049155772679483
+      hash: 1557293148620146716
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.25
@@ -179,15 +186,19 @@ summary: QueueingRabbit is an AMQP-based queueing system
 test_files:
 - spec/integration/asynchronous_publishing_and_consuming_spec.rb
 - spec/integration/asynchronous_publishing_and_consuming_with_retries_spec.rb
+- spec/integration/binary_synchronous_publishing_via_bus_and_asynchronous_consuming_via_job_spec.rb
 - spec/integration/configuration_spec.rb
 - spec/integration/direct_exchange_asynchronous_publishing_and_consuming_spec.rb
 - spec/integration/jobs/print_line_job.rb
 - spec/integration/json_job_asynchronous_publishing_and_consuming_spec.rb
+- spec/integration/json_synchronous_publishing_via_bus_and_asynchronous_consuming_via_job_spec.rb
 - spec/integration/persistent_asynchronous_publishing_and_consuming_spec.rb
 - spec/integration/synchronous_publishing_and_asynchronous_consuming_spec.rb
 - spec/integration/synchronous_publishing_spec.rb
 - spec/spec_helper.rb
 - spec/support/shared_contexts.rb
+- spec/unit/queueing_rabbit/buses/abstract_bus_spec.rb
+- spec/unit/queueing_rabbit/buses/json_bus_spec.rb
 - spec/unit/queueing_rabbit/callbacks_spec.rb
 - spec/unit/queueing_rabbit/client/amqp_spec.rb
 - spec/unit/queueing_rabbit/client/bunny_spec.rb
@@ -196,7 +207,6 @@ test_files:
 - spec/unit/queueing_rabbit/extensions/direct_exchange_spec.rb
 - spec/unit/queueing_rabbit/extensions/new_relic_spec.rb
 - spec/unit/queueing_rabbit/extensions/retryable_spec.rb
-- spec/unit/queueing_rabbit/job_spec.rb
 - spec/unit/queueing_rabbit/jobs/abstract_job_spec.rb
 - spec/unit/queueing_rabbit/jobs/json_job_spec.rb
 - spec/unit/queueing_rabbit/logging_spec.rb

data/spec/unit/queueing_rabbit/job_spec.rb

@@ -1,94 +0,0 @@
-require 'spec_helper'
-
-describe QueueingRabbit::AbstractJob do
-  let(:job_class) {
-    Class.new(QueueingRabbit::AbstractJob) do
-      queue 'test_queue', :durable => true
-      exchange 'test_exchange', :durable => false
-      bind :routing_key => 'test.*'
-    end
-  }
-
-  subject { job_class }
-
-  it { should respond_to(:exchange).with(1).argument }
-  it { should respond_to(:exchange).with(2).arguments }
-  it { should respond_to(:exchange_name) }
-  it { should respond_to(:exchange_options) }
-  it { should respond_to(:queue).with(1).argument }
-  it { should respond_to(:queue).with(2).arguments }
-  it { should respond_to(:queue_name) }
-  it { should respond_to(:queue_options) }
-  it { should respond_to(:channel_options) }
-  it { should respond_to(:channel).with(1).argument }
-  it { should respond_to(:enqueue).with(1).argument }
-  it { should respond_to(:enqueue).with(2).arguments }
-  it { should respond_to(:listening_options) }
-  it { should respond_to(:listen) }
-  it { should respond_to(:listen).with(1).argument }
-  it { should respond_to(:publishing_defaults) }
-
-  its(:queue_name) { should == 'test_queue' }
-  its(:queue_options) { should include(:durable => true) }
-  its(:exchange_options) { should include(:durable => false) }
-  its(:binding_options) { should include(:routing_key => 'test.*') }
-  its(:publishing_defaults) { should include(:routing_key => 'test_queue') }
-
-  describe ".queue_size" do
-    let(:size) { mock }
-
-    before do
-      QueueingRabbit.should_receive(:queue_size).with(subject).and_return(size)
-    end
-
-    its(:queue_size) { should == size }
-  end
-
-  describe '.enqueue' do
-    let(:payload) { mock }
-    let(:options) { {:persistent => true} }
-    let(:result_options) { options.merge(job_class.publishing_defaults) }
-
-    it 'enqueues a job of its own type with given argument' do
-      QueueingRabbit.should_receive(:enqueue).
-                     with(subject, payload, result_options)
-      subject.enqueue(payload, options)
-    end
-  end
-
-  context 'instance methods' do
-    let(:payload) { mock }
-    let(:headers) { mock }
-    let(:metadata) { stub(:headers => headers) }
-
-    subject { job_class.new(payload, metadata) }
-
-    its(:payload) { should == payload }
-    its(:metadata) { should == metadata }
-    its(:headers) { should == headers }
-
-    it { should respond_to(:perform) }
-  end
-end
-
-describe QueueingRabbit::JSONJob do
-  let(:json_job) { QueueingRabbit::JSONJob }
-  let(:payload) { JSON.dump(:foo => 'bar') }
-  let(:metadata) { mock }
-
-  subject { json_job.new(payload, metadata) }
-
-  its(:payload) { should include(:foo => 'bar') }
-
-  describe '.enqueue' do
-    let(:options) { {:persistent => true} }
-    let(:result_options) { options.merge(:routing_key => 'JSONJob') }
-
-    it 'dumps payload to JSON' do
-      QueueingRabbit.should_receive(:enqueue).
-                     with(json_job, payload, result_options)
-      json_job.enqueue({:foo => 'bar'}, options)
-    end
-  end
-end
-