rubykiq 1.0.1 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fe8a63e0620e49a3abee5a4b793f6a47de9d2134
-  data.tar.gz: 538763a88753b76a2d0647b0a66efba79da5db4f
+  metadata.gz: f16a7dd593de391c8588314bf0aaaba87b387308
+  data.tar.gz: 5c91ee1222eebf07a49e30dc695064f515cc31e6
 SHA512:
-  metadata.gz: 61192efb893a6bbe6d0d094e3c672a56ae4cea8e1e6d65dbcdc7c21762423f49fcfd6f6e54d311c7852844af4bafeee5a069342a2e9fe4188bae1109a72bedfd
-  data.tar.gz: 000c33af26dfd8edb0342f35265f2e85284affbd2e49338af480af75fee90db40b38abb08618981510d761136443da1d25513cb0ce8e559ec2fb391b56b9f79e
+  metadata.gz: 4b784fe4ec2dc3d0c972793170b0cd58b740dc1a9c251c344f32ad709062d42399d38b83154ebd185e90b97141014c503e7367e909e655b9d970660ec91a0a0c
+  data.tar.gz: 2de3671bf79f3f07d5b2c554766eaae6d20520a8c4de3047c81208c06e91c9221e1a83407bdeb9afd4a175d4cbc9dfdf980793f871c0380f0350c21f3937073e

data/.gitignore

@@ -2,6 +2,8 @@
 *.rbc
 .bundle
 .config
+.devnotes
+.greenbar
 .yardoc
 Gemfile.lock
 InstalledFiles

data/.travis.yml

@@ -21,4 +21,6 @@ matrix:
 notifications:
   email: false
 services:
-  - redis-server
+  - redis-server
+env:
+  - CODECLIMATE_REPO_TOKEN=04a9fef09063cdef2f62cf46531329ad01a9383e3de8270e112480052774f676

data/CHANGELOG.md

@@ -0,0 +1,16 @@
+Next Release
+============
+* Your contribution here.
+
+1.0.2 (02/08/2015)
+==================
+* [Include `enqueued_at` in payload](https://github.com/karlfreeman/rubykiq/commit/0a68c9dc670f94efe8a344869db0b7ba4a97d1d7) - [@amiel](https://github.com/karlfreeman).
+
+
+1.0.1 (18/03/2014)
+==================
+* [Ensure loading a client is Thread.exclusive](https://github.com/karlfreeman/rubykiq/commit/0a68c9dc670f94efe8a344869db0b7ba4a97d1d7) - [@karlfreeman](https://github.com/karlfreeman).
+
+1.0.0 (03/03/2014)
+==================
+* Initial public release - [@karlfreeman](https://github.com/karlfreeman).

data/Gemfile

@@ -8,5 +8,7 @@ gem 'em-synchrony'
 group :test do
   gem 'rake', '~> 10.0'
   gem 'rspec'
+  gem 'rspec-its'
   gem 'timecop'
+  gem 'codeclimate-test-reporter'
 end

data/README.md

@@ -69,6 +69,7 @@ bar_client = Rubykiq::Client.new(namespace: 'bar')
 [![Gem Version](http://img.shields.io/gem/v/rubykiq.svg)][gem]
 [![Build Status](http://img.shields.io/travis/karlfreeman/rubykiq.svg)][travis]
 [![Code Quality](http://img.shields.io/codeclimate/github/karlfreeman/rubykiq.svg)][codeclimate]
+[![Code Coverage](http://img.shields.io/codeclimate/coverage/github/karlfreeman/rubykiq.svg)][codeclimate]
 [![Gittip](http://img.shields.io/gittip/karlfreeman.svg)][gittip]
 
 ## Supported Redis Drivers

data/Rakefile

@@ -15,7 +15,7 @@ end
 begin
   require 'rubocop/rake_task'
   desc 'Run rubocop'
-  Rubocop::RakeTask.new(:rubocop)
+  RuboCop::RakeTask.new(:rubocop)
 rescue LoadError
 end
 

data/lib/rubykiq.rb

@@ -12,7 +12,7 @@ module Rubykiq
   def_delegators :client, *Rubykiq::Client::VALID_OPTIONS_KEYS
 
   # delegate all VALID_OPTIONS_KEYS setters to the client ( hacky I know... )
-  def_delegators :client, *(Rubykiq::Client::VALID_OPTIONS_KEYS.dup.map! do |key| "#{key}=".to_sym; end)
+  def_delegators :client, *(Rubykiq::Client::VALID_OPTIONS_KEYS.dup.map! { |key| "#{key}=".to_sym; })
 
   # Fetch the Rubykiq::Client
   #

data/lib/rubykiq/client.rb

@@ -176,6 +176,9 @@ module Rubykiq
       # provide a job ID
       pre_normalized_item[:jid] = ::SecureRandom.hex(12)
 
+      # Sidekiq::Queue#latency (used in sidekiq web), requires `enqueued_at`
+      pre_normalized_item[:enqueued_at] = Time.now.to_f
+
       pre_normalized_item
     end
 

data/lib/rubykiq/version.rb

@@ -1,3 +1,3 @@
 module Rubykiq
-  VERSION = '1.0.1' unless defined?(Rubykiq::VERSION)
+  VERSION = '1.0.2' unless defined?(Rubykiq::VERSION)
 end

data/spec/spec_helper.rb

@@ -8,6 +8,7 @@ Bundler.setup
 end
 
 require 'rubykiq'
+require 'rspec/its'
 
 RSpec.configure do |config|
   config.expect_with :rspec do |c|

data/spec/support/codeclimate.rb

@@ -0,0 +1,2 @@
+require 'codeclimate-test-reporter'
+CodeClimate::TestReporter.start

data/spec/support/pry.rb

@@ -1,3 +1,6 @@
 if defined?(RUBY_ENGINE) && RUBY_ENGINE == 'ruby'
-  require 'pry' unless ENV['CI']
+  begin
+    require 'pry' unless ENV['CI']
+  rescue LoadError
+  end
 end

data/spec/unit/client_spec.rb

@@ -20,7 +20,7 @@ describe Rubykiq::Client do
         subject { client }
         its(:namespace) { should eq driver }
         its(:driver) { should be driver }
-        its(:retry) { should be_true }
+        its(:retry) { should eq true }
         its(:queue) { should eq 'default' }
       end
 
@@ -59,18 +59,17 @@ describe Rubykiq::Client do
           context "with args #{args}" do
             it "should create #{args.length} job(s)" do
               wrap_in_synchrony?(driver) do
-                client.connection_pool do |connection|
-                  connection.flushdb
-                end
+                client.connection_pool(&:flushdb)
 
                 expect { client.push(class: 'MyWorker', args: args) }.to change {
-                  client.connection_pool do |connection| connection.llen('queue:default'); end
+                  client.connection_pool { |connection| connection.llen('queue:default'); }
                 }.from(0).to(args.length)
 
-                raw_jobs = client.connection_pool do |connection| connection.lrange('queue:default', 0, args.length); end
+                raw_jobs = client.connection_pool { |connection| connection.lrange('queue:default', 0, args.length); }
                 raw_jobs.each do |job|
                   job = MultiJson.decode(job, symbolize_keys: true)
                   expect(job).to have_key(:jid)
+                  expect(job[:enqueued_at]).to be_within(1).of(Time.now.to_f)
                 end
               end
             end
@@ -81,15 +80,13 @@ describe Rubykiq::Client do
               context "with time #{time} (#{time.class})" do
                 it "should create #{args.length} job(s)" do
                   wrap_in_synchrony?(driver) do
-                    client.connection_pool do |connection|
-                      connection.flushdb
-                    end
+                    client.connection_pool(&:flushdb)
 
                     expect { client.push(class: 'MyWorker', args: args, at: time) }.to change {
-                      client.connection_pool do |connection| connection.zcard('schedule'); end
+                      client.connection_pool { |connection| connection.zcard('schedule'); }
                     }.from(0).to(args.length)
 
-                    raw_jobs = client.connection_pool do |connection| connection.zrange('schedule', 0, args.length); end
+                    raw_jobs = client.connection_pool { |connection| connection.zrange('schedule', 0, args.length); }
                     raw_jobs.each do |job|
                       job = MultiJson.decode(job, symbolize_keys: true)
                       expect(job).to have_key(:at)

data/spec/unit/connection_spec.rb

@@ -28,7 +28,7 @@ describe Rubykiq::Connection do
 
   describe :env do
     subject { Rubykiq::Connection.new }
-    [{ name: 'REDISTOGO_URL', value: 'redistogo' }, { name: 'REDIS_PROVIDER', value: 'redisprovider' }, { name: 'REDIS_URL', value: 'redisurl' }].each do | test_case |
+    [{ name: 'REDISTOGO_URL', value: 'redistogo' }, { name: 'REDIS_PROVIDER', value: 'redisprovider' }, { name: 'REDIS_URL', value: 'redisurl' }].each do |test_case|
       context "with ENV[#{test_case[:name]}]" do
         before do
           ENV[test_case[:name]] = "redis://#{test_case[:value]}:6379/0"

data/spec/unit/rubykiq_spec.rb

@@ -1,8 +1,6 @@
 require 'spec_helper'
 
 describe Rubykiq do
-
-
   describe :client do
     it 'should return a Client' do
       expect(Rubykiq.client).to be_kind_of(Rubykiq::Client)
@@ -34,5 +32,4 @@ describe Rubykiq do
       it { should respond_to "#{key}=".to_sym }
     end
   end
-
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rubykiq
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors:
 - Karl Freeman
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-03-20 00:00:00.000000000 Z
+date: 2015-08-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis
@@ -174,6 +174,7 @@ files:
 - lib/rubykiq/version.rb
 - rubykiq.gemspec
 - spec/spec_helper.rb
+- spec/support/codeclimate.rb
 - spec/support/pry.rb
 - spec/support/timecop.rb
 - spec/unit/client_spec.rb
@@ -199,12 +200,13 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.0
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: Sidekiq agnostic enqueuing using Redis
 test_files:
 - spec/spec_helper.rb
+- spec/support/codeclimate.rb
 - spec/support/pry.rb
 - spec/support/timecop.rb
 - spec/unit/client_spec.rb