sidekiq-apriori 0.0.1 → 0.0.2

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 3203f8525a25c20fa3a2809bbc431ee64b27ee81
+  data.tar.gz: 8dc78be48780cf4eb5c9c010d73fae05f0a24f31
+SHA512:
+  metadata.gz: 3227506019efd51481e045ebb7fc613a624687b5c64e351d609847445522babf56956add7474f4e017bb042d5750196a335b504001f11ce170b46affa93e65af
+  data.tar.gz: 4b924a010def90fa31bd661c54366e12aee79c58f67a0a6e747b37913f6164b00aed989155713895f55cc68f6ea3a68148dc644a86bea95239b0735e02c3f385

data/.ruby-version

@@ -0,0 +1 @@
+2.0.0-p247

data/README.md

@@ -1,4 +1,119 @@
 sidekiq-apriori
 ===============
-
 Prioritization Middleware for Sidekiq
+
+[![Build Status](https://travis-ci.org/enova/sidekiq-apriori.png)](https://travis-ci.org/enova/sidekiq-apriori)
+
+Overview
+--------
+
+sidekiq-apriori simplifies dynamic prioritization for sidekiq by supplying a
+simple sidekiq middleware, related active record hooks, & some additional
+argument handling (for ruby 2 users).
+
+Installation
+------------
+
+Manual installation is always an option. From the command line:
+
+    $ gem install sidekiq-priority
+
+Or, if you're using bundler, simply include it in your Gemfile:
+
+    gem 'sidekiq-priority'
+
+Priorities
+----------
+
+By default, sidekiq-apriori supports four priorities: immediate, high, nil
+(default), and low. If you would like to use different priorities you can
+add something along these lines to (as an example for you railsy folk) your
+sidekiq initializer:
+
+```ruby
+## config/initializers/sidekiq.rb
+
+Sidekiq::Apriori::PRIORITIES = ['wut', 'huh', 'ok', nil, 'not_even_a_little']
+```
+
+The nil is meaningful insofar as it represents the default priority. Unless you
+want to disallow unset priorities, leave the nil in.
+
+sidekiq-apriori is inspired by (a response to?) [sidekiq-priority](https://github.com/socialpandas/sidekiq-priority), in which the
+order of the priorities is important. Contrary to the approach taken by
+sidekiq-priority, sidekiq-apriori uses sidekiq's built in mechanism for
+configuring the order of processing. So, for example, if your sidekiq.yml
+currently looks like this:
+
+```yaml
+## sidekiq.yml
+
+verbose: false
+:pidfile: /tmp/sidekiq.pid
+:concurrency: 5
+:queues:
+  - postback
+  - background
+```
+
+you might want to change the 'queues' entry to look more like this:
+
+```yaml
+:queues:
+  - postback_wut
+  - postback_huh
+  - postback_ok
+  - postback
+  - postback_not_even_a_little
+  - background
+```
+
+Use
+---
+
+In addition to the use described in the PRIORITIES section, some tooling is
+provided for active record classes with priority as an attribute:
+
+```ruby
+## app/models/prioritized.rb
+
+class Prioritized < ActiveRecord::Base
+  include Sidekiq::Apriori::Arb
+
+  prioritize do
+    self.priority = nil unless
+      Sidekiq::Apriori::PRIORITIES.include?(self.priority)
+
+      self.priority = (...)
+  end
+end
+```
+
+Alternatively, you can pass a method name to prioritize:
+
+```ruby
+## app/models/prioritized.rb
+
+class Prioritized < ActiveRecord::Base
+  include Sidekiq::Apriori::Arb
+
+  prioritize using: 'some_method'
+
+  def some_method
+    (...)
+  end
+end
+```
+
+If you're lucky enough to be using ruby 2, you can save yourself some work by
+including ```Sidekiq::Apriori::Worker``` instead of ```Sidekiq::Worker``` in your
+worker classes. This will have the nifty side effect of saving you the effort of
+changing the definition of the classes' perform method & its invocation.
+```Sidekiq::Apriori::Worker``` uses ```prepend``` to define a perform which will
+take an optional hash containing a priority designation.
+
+License
+-------
+
+sidekiq-apriori is released under the MIT License. Please see the [LICENSE](LICENSE)
+file for details.

data/lib/sidekiq-apriori/arb.rb

@@ -1,23 +1,28 @@
 module Sidekiq::Apriori
-  def self.included(base)
-    return unless defined?(ActiveRecord::Base) && base < ActiveRecord::Base
+  module Arb
+    def self.included(base)
 
-    base.extend ClassMethods
+      if defined?(ActiveRecord::Base) && base < ActiveRecord::Base
+        require 'sidekiq-apriori/arb'
+        base.extend ClassMethods
 
-    ## Add validation for priority attribute
-    if base.attribute_names.include?('priority')
-      base.validates_inclusion_of :priority, :in => PRIORITIES
+        ## Add validation for priority attribute
+        if base.attribute_names.include?('priority')
+          base.validates_inclusion_of :priority, :in => PRIORITIES
+        end
+      end
     end
-  end
 
-  module ClassMethods
-    ## Declarative hook to prioritize instances
-    def prioritize(options = {}, &block)
-      method = ( block_given? && [-1, 0].include?(block.arity) ) ?
-        block : ( options[:using] || options[:with] )
+    module ClassMethods
+
+      ## Declarative hook to prioritize instances
+      def prioritize(options = {}, &block)
+        method = ( block_given? && [-1, 0].include?(block.arity) ) ?
+          block : ( options[:using] || options[:with] )
 
-      before_validation method, :on => :create
+        before_validation method, :on => :create
+      end
     end
-  end
 
+  end
 end

data/lib/sidekiq-apriori/middleware/client.rb

@@ -0,0 +1,7 @@
+require 'sidekiq-apriori/middleware/prioritizer'
+
+Sidekiq.configure_client do |config|
+  config.client_middleware do |chain|
+    chain.add Sidekiq::Apriori::Prioritizer
+  end
+end

data/lib/sidekiq-apriori/{prioritizer.rb → middleware/prioritizer.rb}

@@ -1,7 +1,11 @@
+require 'sidekiq-apriori/priorities'
+
 module Sidekiq::Apriori
   class Prioritizer
     def call(worker, msg, queue)
-      priority = msg["args"].try(:last)
+      options   = msg["args"].last if msg["args"].respond_to?(:last)
+      priority  = options[:priority] if options.is_a?(Hash) && options.has_key?(:priority)
+
       if priorities.include?(priority)
         msg["queue"] = queue.to_s.sub(priority_regexp,"_#{priority}")
       end

data/lib/sidekiq-apriori/middleware/server.rb

@@ -0,0 +1,7 @@
+require 'sidekiq-apriori/middleware/prioritizer'
+
+Sidekiq.configure_server do |config|
+  config.client_middleware do |chain|
+    chain.add Sidekiq::Apriori::Prioritizer
+  end
+end

data/lib/sidekiq-apriori/middleware.rb

@@ -1,14 +1,3 @@
-require 'sidekiq'
-require 'sidekiq-apriori/prioritizer'
-
-Sidekiq.configure_client do |config|
-  config.client_middleware do |chain|
-    chain.add Sidekiq::Apriori::Prioritizer
-  end
-end
-
-Sidekiq.configure_server do |config|
-  config.client_middleware do |chain|
-    chain.add Sidekiq::Apriori::Prioritizer
-  end
-end
+require 'sidekiq-apriori/middleware/prioritizer'
+require 'sidekiq-apriori/middleware/client'
+require 'sidekiq-apriori/middleware/server'

data/lib/sidekiq-apriori/priorities.rb

@@ -0,0 +1,10 @@
+module Sidekiq
+  module Apriori
+    PRIORITIES = [
+      'immediate',
+      'high',
+      nil,
+      'low'
+    ].freeze unless defined?(PRIORITIES)
+  end
+end

data/lib/sidekiq-apriori/version.rb

@@ -1,5 +1,5 @@
 module Sidekiq
   module Apriori
-    VERSION = "0.0.1"
+    VERSION = "0.0.2"
   end
 end

data/lib/sidekiq-apriori/worker.rb

@@ -0,0 +1,28 @@
+module Sidekiq::Apriori
+  module Worker
+    def self.included(base)
+      base.class_eval do
+        include Sidekiq::Worker if defined?(Sidekiq::Worker)
+
+        version = RUBY_VERSION.split(/\./).map(&:to_i) rescue []
+        prepend ClassMethods if version.first > 1
+      end
+    end
+  end
+
+  module ClassMethods
+    def perform(*args)
+      retried = false
+
+      begin
+        super(*args)
+      rescue ArgumentError => err
+        raise err unless
+          args.last.is_a?(Hash) && args.last.has_key?(:priority)
+
+        args = args[0..-2]
+        (retried = true ) && retry
+      end
+    end
+  end
+end

data/lib/sidekiq-apriori.rb

@@ -1,12 +1,4 @@
 require 'sidekiq-apriori/version'
-require 'sidekiq-apriori/middleware'
-require 'sidekiq-apriori/arb.rb'
-
-module Sidekiq::Apriori
-  PRIORITIES = [
-    'immediate',
-    'high',
-    nil,
-    'low'
-  ].freeze unless defined?(PRIORITIES)
-end
+require 'sidekiq-apriori/priorities'
+require 'sidekiq-apriori/middleware/prioritizer'
+require 'sidekiq-apriori/arb'

data/sidekiq-apriori.gemspec

@@ -20,10 +20,14 @@ Gem::Specification.new do |s|
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
 
+  s.add_development_dependency "activerecord"
+  s.add_development_dependency "fakeredis"
   s.add_development_dependency "pry"
   s.add_development_dependency "rake"
   s.add_development_dependency "rspec"
   s.add_development_dependency "simplecov"
+  s.add_development_dependency "simplecov-rcov"
+  s.add_development_dependency "sqlite3"
 
   s.add_dependency "sidekiq"
 end

data/spec/sidekiq-apriori/arb_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+require 'sidekiq-apriori/arb'
+
+describe Sidekiq::Apriori::Arb do
+
+  ## See spec/support/arb.rb for Arb class definition
+  require 'support/arb'
+
+  let(:arb)             { Arb.new  }
+  let(:using_method)    { PrioritizedUsingMethod.new }
+  let(:using_callable)  { PrioritizedUsingCallable.new }
+
+  it "should validate priorities" do
+    arb.priority = "none"
+    arb.should be_invalid
+
+    Sidekiq::Apriori::PRIORITIES.each do |priority|
+      arb.priority = priority
+      arb.should be_valid
+    end
+  end
+
+  it "should attempt prioritization on creation with named prioritizer" do
+    using_method.should_receive(:some_method)
+    using_method.save
+  end
+
+  it "should attempt prioritization on creation with using a block" do
+    using_callable.should_receive(:some_other_method)
+    using_callable.save
+  end
+end

data/spec/sidekiq-apriori/middleware/client_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+require 'sidekiq-apriori/middleware/prioritizer'
+
+describe Sidekiq::Apriori, 'client middleware' do
+  before do
+    Sidekiq.stub(:server? => false)
+
+    Sidekiq.configure_client do |config|
+      config.client_middleware do |chain|
+        chain.remove Sidekiq::Apriori::Prioritizer
+      end
+    end
+
+  end
+
+  it "should include Sidekiq::Apriori::Prioritizer in client middleware" do
+    Sidekiq.client_middleware.entries.should be_empty
+
+    (require 'sidekiq-apriori/middleware/client').should be_true
+
+    Sidekiq.client_middleware.entries.should_not be_empty
+    Sidekiq.client_middleware.entries.map(&:klass).
+      should be_include(Sidekiq::Apriori::Prioritizer)
+  end
+end

data/spec/sidekiq-apriori/middleware/prioritizer_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+require 'sidekiq-apriori/middleware/prioritizer'
+
+describe Sidekiq::Apriori::Prioritizer do
+  subject(:middleware)  { Sidekiq::Apriori::Prioritizer.new }
+
+  let(:queue)   { 'foo' }
+  let(:message) { {'args' => [1, nil, priority: 'high'], 'queue' => queue } }
+
+  describe '#call' do
+    it 'should respond' do
+      middleware.respond_to?(:call).should be_true
+    end
+
+    it 'should require three arguments' do
+      expect { middleware.call }.
+        to raise_error( ArgumentError,'wrong number of arguments (0 for 3)')
+    end
+
+    it 'should set priority queue' do
+      middleware.call(nil, message, queue) {}
+      message['queue'].should eql('foo_high')
+    end
+
+    it 'should allow only one priority suffix' do
+      message['queue'] = 'foo_low_high_immediate'
+      middleware.call(nil, message, queue) {}
+      message['queue'].should eql('foo_high')
+
+      middleware.call(nil, message, queue) {}
+      message['queue'].should eql('foo_high')
+    end
+  end
+end

data/spec/sidekiq-apriori/middleware/server_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+require 'sidekiq-apriori/middleware/prioritizer'
+
+describe Sidekiq::Apriori, 'server middleware' do
+  before do
+    Sidekiq.stub(:server? => true)
+
+    Sidekiq.configure_server do |config|
+      config.client_middleware do |chain|
+        chain.remove Sidekiq::Apriori::Prioritizer
+      end
+    end
+
+  end
+
+  it "should include Sidekiq::Apriori::Prioritizer in client middleware" do
+    Sidekiq.client_middleware.entries.should be_empty
+
+    (require 'sidekiq-apriori/middleware/server').should be_true
+
+    Sidekiq.client_middleware.entries.should_not be_empty
+    Sidekiq.client_middleware.entries.map(&:klass).
+      should be_include(Sidekiq::Apriori::Prioritizer)
+  end
+end

data/spec/sidekiq-apriori/worker_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+require 'sidekiq-apriori/worker'
+
+describe Sidekiq::Apriori::Worker do
+  before(:all) do
+    class Job
+      def perform; end
+      include Sidekiq::Apriori::Worker
+    end
+  end
+
+  let(:job) { Job.new }
+
+  ## Checking for ruby 2
+  #
+  if ( RUBY_VERSION.split(/\./).map(&:to_i) rescue [] ).first > 1
+    it "redefines 'perform' to handle an extra argument when that argument has priority information" do
+      job.should be_an_instance_of(Job)
+      expect { job.perform(priority: "high") }.not_to raise_error
+    end
+
+    it "does not rescue errors with incorrectly formatted priority information" do
+      expect { job.perform("high") }.to raise_error(ArgumentError)
+      expect { job.perform({}) }.to raise_error(ArgumentError)
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,3 +1,7 @@
+require 'fakeredis/rspec'
+require 'sidekiq'
+require 'sidekiq/testing'
+
 require 'simplecov'
 
 SimpleCov.start do
@@ -5,6 +9,35 @@ SimpleCov.start do
   add_filter "spec"
 end
 
-require File.expand_path("../../lib/sidekiq-apriori", __FILE__)
+require 'sidekiq-apriori/priorities'
+
+redis = { :url => "redis://localhost:6379/0",
+          :driver => Redis::Connection::Memory  }
 
-require 'support/arb'
+Sidekiq.configure_client { |config| config.redis = redis }
+Sidekiq.configure_server do |config|
+  config.redis = redis
+
+  # require 'support/tracked_fetch'
+  # Sidekiq.options[:fetch] = TrackedFetch
+end
+
+RSpec.configure do |config|
+  config.before(:each) do
+    ## Use metadata to determine testing behavior
+    ## for queuing.
+    Sidekiq::Worker.clear_all
+
+    case example.metadata[:queuing].to_s
+    when 'enable', 'enabled', 'on', 'true'
+      Sidekiq::Testing.disable!
+    when 'fake', 'mock'
+      Sidekiq::Testing.fake!
+    when 'inline'
+      Sidekiq::Testing.inline!
+    else
+      defined?(Redis::Connection::Memory) ?
+        Sidekiq::Testing.disable! : Sidekiq::Testing.inline!
+    end
+  end
+end

data/spec/support/arb.rb

@@ -1,19 +1,27 @@
 require 'active_record'
+require 'sidekiq-apriori/arb'
 require 'sqlite3'
 
 ActiveRecord::Base.establish_connection(
-  :adapter => :sqlite3,
-  :database => 'spec/support/test.db'
+  :adapter   => :sqlite3,
+  :database  => 'spec/support/test.db'
 )
 
 ActiveRecord::Schema.define do
-  drop_table :arbs
-
-  create_table :arbs do |t|
-    t.column :priority, :string
-  end
+  drop_table(:arbs) rescue nil
+  create_table(:arbs) { |t| t.column(:priority, :string) }
 end
 
 class Arb < ActiveRecord::Base
-  include Sidekiq::Apriori
+  include Sidekiq::Apriori::Arb
+end
+
+class PrioritizedUsingMethod < Arb
+  prioritize using: :some_method
+end
+
+class PrioritizedUsingCallable < Arb
+  prioritize do
+    self.some_other_method
+  end
 end

metadata

@@ -1,8 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sidekiq-apriori
 version: !ruby/object:Gem::Version
-  version: 0.0.1
-  prerelease: 
+  version: 0.0.2
 platform: ruby
 authors:
 - Blake Thomas
@@ -10,86 +9,132 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-11-28 00:00:00.000000000 Z
+date: 2013-12-01 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: activerecord
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: fakeredis
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: simplecov-rcov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: sidekiq
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 description: Prioritization middleware for Sidekiq
@@ -101,6 +146,7 @@ extra_rdoc_files: []
 files:
 - .gitignore
 - .rspec
+- .ruby-version
 - .simplecov
 - .travis.yml
 - Gemfile
@@ -110,39 +156,49 @@ files:
 - lib/sidekiq-apriori.rb
 - lib/sidekiq-apriori/arb.rb
 - lib/sidekiq-apriori/middleware.rb
-- lib/sidekiq-apriori/prioritizer.rb
+- lib/sidekiq-apriori/middleware/client.rb
+- lib/sidekiq-apriori/middleware/prioritizer.rb
+- lib/sidekiq-apriori/middleware/server.rb
+- lib/sidekiq-apriori/priorities.rb
 - lib/sidekiq-apriori/version.rb
+- lib/sidekiq-apriori/worker.rb
 - sidekiq-apriori.gemspec
-- spec/sidekiq-apriori_spec.rb
+- spec/sidekiq-apriori/arb_spec.rb
+- spec/sidekiq-apriori/middleware/client_spec.rb
+- spec/sidekiq-apriori/middleware/prioritizer_spec.rb
+- spec/sidekiq-apriori/middleware/server_spec.rb
+- spec/sidekiq-apriori/worker_spec.rb
 - spec/spec_helper.rb
 - spec/support/arb.rb
 homepage: https://github.com/bwthomas/sidekiq-apriori
 licenses:
 - MIT
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: nowarning
-rubygems_version: 1.8.23
+rubygems_version: 2.0.3
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Prioritization Middleware for Sidekiq
 test_files:
-- spec/sidekiq-apriori_spec.rb
+- spec/sidekiq-apriori/arb_spec.rb
+- spec/sidekiq-apriori/middleware/client_spec.rb
+- spec/sidekiq-apriori/middleware/prioritizer_spec.rb
+- spec/sidekiq-apriori/middleware/server_spec.rb
+- spec/sidekiq-apriori/worker_spec.rb
 - spec/spec_helper.rb
 - spec/support/arb.rb
-has_rdoc: 

data/spec/sidekiq-apriori_spec.rb

@@ -1,60 +0,0 @@
-require 'spec_helper'
-
-describe Sidekiq::Apriori do
-  before(:all) do
-    ## See spec/support/arb.rb for Arb class definition
-    class Prioritized < Arb
-      prioritize using: :some_method
-    end
-  end
-
-  let(:arb)           { Arb.new  }
-  let(:prioritized)   { Prioritized.new }
-
-  it "should validate priorities" do
-    arb.priority = "none"
-    arb.should be_invalid
-
-    Sidekiq::Apriori::PRIORITIES.each do |priority|
-      arb.priority = priority
-      arb.should be_valid
-    end
-  end
-
-  it "should attempt prioritization on creation with named prioritizer" do
-    prioritized.should_receive(:some_method)
-    prioritized.save
-  end
-
-  describe Sidekiq::Apriori::Prioritizer do
-    subject(:middleware)  { Sidekiq::Apriori::Prioritizer.new }
-
-    let(:message)         { {'args' => [1, nil, 'high'], 'queue' => 'foo'} }
-    let(:queue)           { 'foo' }
-
-    describe '#call' do
-      it 'should respond' do
-        middleware.respond_to?(:call).should be_true
-      end
-
-      it 'should require three arguments' do
-        expect { middleware.call }.
-          to raise_error( ArgumentError,'wrong number of arguments (0 for 3)')
-      end
-
-      it 'should set priority queue' do
-        middleware.call(nil, message, queue) {}
-        message['queue'].should eql('foo_high')
-      end
-
-      it 'should allow only one priority suffix' do
-        message['queue'] = 'foo_low_high_immediate'
-        middleware.call(nil, message, queue) {}
-        message['queue'].should eql('foo_high')
-
-        middleware.call(nil, message, queue) {}
-        message['queue'].should eql('foo_high')
-      end
-    end
-  end
-end