inst-jobs-statsd 2.1.0 → 2.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c7d2f71f91477c6e30a43226165ea6506e6e7b186f3fcf4876ce3b99d8595cd1
-  data.tar.gz: a57c33ba7eb726155f0aa767a18dffc7cfb39f0faae21c51133d233076f7954d
+  metadata.gz: b7109a5cd6de633fa11fc099411ebee5734f42e70fc5647d548bb6b0b600fdd2
+  data.tar.gz: dc22dd71405515b383151a3bf96549f8544102bf53ef4f676b50560bb01cff17
 SHA512:
-  metadata.gz: 7a692a9342cceb3ab900493cda108515956203e30d83acd048b96f3da2a9978f59ea711196aac5ed1c0156001847bd06548a36cd2b8e395be80a2309f077a620
-  data.tar.gz: 70018e665a4e434fc6828391d0f67f753fb09569db8991baee0d44bfa72c73b9b3e770d5ecf8d6e0f1173961497ca586c96c552c10dfdef746c97157aea501e1
+  metadata.gz: efdc1e13bc1101c87497433447a11b71300ca5dae150b80d0c3cb036fbdf7538bc0f82086f5ddfb65f8b4c8a0356c5425a4c66e9a0d8172a922f28d5426f1b95
+  data.tar.gz: 64dfa45fd9551d1507ed79d2f4e06d828614fe26e261e427e9f85418cfaad8e4aff9a934f9fd993487b4fa5000cf39a1bf6546290fea204c941dc58e7da037f3

data/lib/inst-jobs-statsd.rb

@@ -9,7 +9,6 @@ require_relative 'inst_jobs_statsd/jobs_tracker'
 require_relative 'inst_jobs_statsd/naming'
 
 require_relative 'inst_jobs_statsd/stats/counters'
-require_relative 'inst_jobs_statsd/stats/counters/failed'
 require_relative 'inst_jobs_statsd/stats/counters/run'
 
 require_relative 'inst_jobs_statsd/stats/periodic'
@@ -21,5 +20,6 @@ require_relative 'inst_jobs_statsd/stats/timing'
 require_relative 'inst_jobs_statsd/stats/timing/failed'
 require_relative 'inst_jobs_statsd/stats/timing/perform'
 require_relative 'inst_jobs_statsd/stats/timing/pop'
+require_relative 'inst_jobs_statsd/ext/job'
 
 ::InstStatsd::DefaultTracking.include InstJobsStatsd::DefaultTracking

data/lib/inst_jobs_statsd/ext/job.rb

@@ -0,0 +1,11 @@
+module InstJobsStatsd
+  module Ext
+    module Job
+      def fail!
+        failed_job = super
+        InstJobsStatsd::Stats::Counters.report_count(:failed, 1, job: failed_job)
+        failed_job
+      end
+    end
+  end
+end

data/lib/inst_jobs_statsd/jobs_tracker.rb

@@ -9,8 +9,8 @@ module InstJobsStatsd
     end
 
     def initialize(enable_periodic_queries: true)
-      Stats::Counters::Failed.enable
       Stats::Counters::Run.enable
+      ::Delayed::Job.prepend InstJobsStatsd::Ext::Job
 
       if enable_periodic_queries
         Stats::Periodic::Failed.enable

data/lib/inst_jobs_statsd/naming.rb

@@ -82,7 +82,7 @@ module InstJobsStatsd
 
     def self.dd_region_tags
       return {} unless ENV['INST_JOBS_STATSD_NAMESPACE']
-      {namespace: ENV['INST_JOBS_STATSD_NAMESPACE']}
+      { namespace: ENV['INST_JOBS_STATSD_NAMESPACE'] }
     end
 
     private
@@ -91,7 +91,7 @@ module InstJobsStatsd
       obj_tag, method_tag = job.tag.split(/[\.#]/, 2).map do |v|
         InstStatsd::Statsd.escape(v).gsub('::', '-')
       end
-      return method_tag, obj_tag
+      [method_tag, obj_tag]
     end
   end
 end

data/lib/inst_jobs_statsd/version.rb

@@ -1,3 +1,3 @@
 module InstJobsStatsd
-  VERSION = '2.1.0'.freeze
+  VERSION = '2.1.1'.freeze
 end

data/spec/gemfiles/42.gemfile.lock

@@ -0,0 +1,206 @@
+PATH
+  remote: ../..
+  specs:
+    inst-jobs-statsd (2.1.0)
+      inst-jobs (> 1.0, < 3.0)
+      inst_statsd (>= 2.1.2, < 3.0)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    actionmailer (4.2.11.1)
+      actionpack (= 4.2.11.1)
+      actionview (= 4.2.11.1)
+      activejob (= 4.2.11.1)
+      mail (~> 2.5, >= 2.5.4)
+      rails-dom-testing (~> 1.0, >= 1.0.5)
+    actionpack (4.2.11.1)
+      actionview (= 4.2.11.1)
+      activesupport (= 4.2.11.1)
+      rack (~> 1.6)
+      rack-test (~> 0.6.2)
+      rails-dom-testing (~> 1.0, >= 1.0.5)
+      rails-html-sanitizer (~> 1.0, >= 1.0.2)
+    actionview (4.2.11.1)
+      activesupport (= 4.2.11.1)
+      builder (~> 3.1)
+      erubis (~> 2.7.0)
+      rails-dom-testing (~> 1.0, >= 1.0.5)
+      rails-html-sanitizer (~> 1.0, >= 1.0.3)
+    activejob (4.2.11.1)
+      activesupport (= 4.2.11.1)
+      globalid (>= 0.3.0)
+    activemodel (4.2.11.1)
+      activesupport (= 4.2.11.1)
+      builder (~> 3.1)
+    activerecord (4.2.11.1)
+      activemodel (= 4.2.11.1)
+      activesupport (= 4.2.11.1)
+      arel (~> 6.0)
+    activesupport (4.2.11.1)
+      i18n (~> 0.7)
+      minitest (~> 5.1)
+      thread_safe (~> 0.3, >= 0.3.4)
+      tzinfo (~> 1.1)
+    after_transaction_commit (1.1.2)
+      activerecord (>= 4.0)
+    arel (6.0.4)
+    aroi (0.0.7)
+      rails (>= 3.2)
+    ast (2.4.0)
+    builder (3.2.4)
+    bump (0.8.0)
+    byebug (11.1.1)
+    coderay (1.1.2)
+    concurrent-ruby (1.1.5)
+    crass (1.0.6)
+    database_cleaner (1.7.0)
+    debug_inspector (0.0.3)
+    diff-lcs (1.3)
+    docile (1.3.2)
+    dogstatsd-ruby (4.8.2)
+    erubis (2.7.0)
+    et-orbi (1.2.4)
+      tzinfo
+    factory_girl (4.9.0)
+      activesupport (>= 3.0.0)
+    fugit (1.4.2)
+      et-orbi (~> 1.1, >= 1.1.8)
+      raabro (~> 1.4)
+    globalid (0.4.2)
+      activesupport (>= 4.2.0)
+    i18n (0.9.5)
+      concurrent-ruby (~> 1.0)
+    inst-jobs (1.0.4)
+      activerecord (>= 4.2)
+      activesupport (>= 4.2)
+      after_transaction_commit (>= 1.0, < 3)
+      debug_inspector (~> 0.0.3)
+      fugit (~> 1.3)
+      railties (>= 4.2)
+      redis (> 3.0)
+      redis-scripting (~> 1.0.1)
+    inst_statsd (2.1.6)
+      aroi (~> 0.0.7)
+      dogstatsd-ruby (~> 4.2)
+      statsd-ruby (~> 1.0)
+    json (2.3.0)
+    loofah (2.4.0)
+      crass (~> 1.0.2)
+      nokogiri (>= 1.5.9)
+    mail (2.7.1)
+      mini_mime (>= 0.1.1)
+    method_source (0.9.2)
+    mini_mime (1.0.2)
+    mini_portile2 (2.4.0)
+    minitest (5.14.0)
+    nokogiri (1.10.7)
+      mini_portile2 (~> 2.4.0)
+    parser (2.7.0.2)
+      ast (~> 2.4.0)
+    pg (0.21.0)
+    powerpack (0.1.2)
+    pry (0.12.2)
+      coderay (~> 1.1.0)
+      method_source (~> 0.9.0)
+    raabro (1.4.0)
+    rack (1.6.12)
+    rack-test (0.6.3)
+      rack (>= 1.0)
+    rails (4.2.11.1)
+      actionmailer (= 4.2.11.1)
+      actionpack (= 4.2.11.1)
+      actionview (= 4.2.11.1)
+      activejob (= 4.2.11.1)
+      activemodel (= 4.2.11.1)
+      activerecord (= 4.2.11.1)
+      activesupport (= 4.2.11.1)
+      bundler (>= 1.3.0, < 2.0)
+      railties (= 4.2.11.1)
+      sprockets-rails
+    rails-deprecated_sanitizer (1.0.3)
+      activesupport (>= 4.2.0.alpha)
+    rails-dom-testing (1.0.9)
+      activesupport (>= 4.2.0, < 5.0)
+      nokogiri (~> 1.6)
+      rails-deprecated_sanitizer (>= 1.0.1)
+    rails-html-sanitizer (1.3.0)
+      loofah (~> 2.3)
+    railties (4.2.11.1)
+      actionpack (= 4.2.11.1)
+      activesupport (= 4.2.11.1)
+      rake (>= 0.8.7)
+      thor (>= 0.18.1, < 2.0)
+    rainbow (2.2.2)
+      rake
+    rake (13.0.1)
+    redis (4.2.5)
+    redis-scripting (1.0.1)
+      redis (>= 3.0)
+    rspec (3.9.0)
+      rspec-core (~> 3.9.0)
+      rspec-expectations (~> 3.9.0)
+      rspec-mocks (~> 3.9.0)
+    rspec-core (3.9.1)
+      rspec-support (~> 3.9.1)
+    rspec-expectations (3.9.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.9.0)
+    rspec-mocks (3.9.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.9.0)
+    rspec-support (3.9.2)
+    rubocop (0.48.1)
+      parser (>= 2.3.3.1, < 3.0)
+      powerpack (~> 0.1)
+      rainbow (>= 1.99.1, < 3.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (~> 1.0, >= 1.0.1)
+    ruby-progressbar (1.10.1)
+    simplecov (0.17.1)
+      docile (~> 1.1)
+      json (>= 1.8, < 3)
+      simplecov-html (~> 0.10.0)
+    simplecov-html (0.10.2)
+    sprockets (3.7.2)
+      concurrent-ruby (~> 1.0)
+      rack (> 1, < 3)
+    sprockets-rails (3.2.1)
+      actionpack (>= 4.0)
+      activesupport (>= 4.0)
+      sprockets (>= 3.0.0)
+    statsd-ruby (1.5.0)
+    test_after_commit (1.1.0)
+      activerecord (>= 3.2)
+    thor (1.0.1)
+    thread_safe (0.3.6)
+    timecop (0.9.1)
+    tzinfo (1.2.6)
+      thread_safe (~> 0.1)
+    unicode-display_width (1.6.1)
+    wwtd (1.4.0)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  after_transaction_commit (< 2)
+  bump
+  bundler
+  byebug
+  database_cleaner (~> 1.7)
+  factory_girl
+  inst-jobs-statsd!
+  pg (~> 0.21.0)
+  pry
+  rails (~> 4.2.11)
+  rake
+  rspec (~> 3.9)
+  rubocop (~> 0.48.0)
+  simplecov (~> 0.17)
+  test_after_commit (= 1.1.0)
+  timecop
+  wwtd (~> 1.4.0)
+
+BUNDLED WITH
+   1.17.3

data/spec/gemfiles/51.gemfile.lock

@@ -0,0 +1,231 @@
+PATH
+  remote: ../..
+  specs:
+    inst-jobs-statsd (2.1.0)
+      inst-jobs (> 1.0, < 3.0)
+      inst_statsd (>= 2.1.2, < 3.0)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    actioncable (5.1.7)
+      actionpack (= 5.1.7)
+      nio4r (~> 2.0)
+      websocket-driver (~> 0.6.1)
+    actionmailer (5.1.7)
+      actionpack (= 5.1.7)
+      actionview (= 5.1.7)
+      activejob (= 5.1.7)
+      mail (~> 2.5, >= 2.5.4)
+      rails-dom-testing (~> 2.0)
+    actionpack (5.1.7)
+      actionview (= 5.1.7)
+      activesupport (= 5.1.7)
+      rack (~> 2.0)
+      rack-test (>= 0.6.3)
+      rails-dom-testing (~> 2.0)
+      rails-html-sanitizer (~> 1.0, >= 1.0.2)
+    actionview (5.1.7)
+      activesupport (= 5.1.7)
+      builder (~> 3.1)
+      erubi (~> 1.4)
+      rails-dom-testing (~> 2.0)
+      rails-html-sanitizer (~> 1.0, >= 1.0.3)
+    activejob (5.1.7)
+      activesupport (= 5.1.7)
+      globalid (>= 0.3.6)
+    activemodel (5.1.7)
+      activesupport (= 5.1.7)
+    activerecord (5.1.7)
+      activemodel (= 5.1.7)
+      activesupport (= 5.1.7)
+      arel (~> 8.0)
+    activesupport (5.1.7)
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      i18n (>= 0.7, < 2)
+      minitest (~> 5.1)
+      tzinfo (~> 1.1)
+    after_transaction_commit (2.1.0)
+      activerecord (>= 5.0)
+    arel (8.0.0)
+    aroi (0.0.7)
+      rails (>= 3.2)
+    ast (2.4.0)
+    backports (3.15.0)
+    builder (3.2.4)
+    bump (0.8.0)
+    byebug (11.1.1)
+    coderay (1.1.2)
+    concurrent-ruby (1.1.5)
+    crass (1.0.6)
+    database_cleaner (1.7.0)
+    debug_inspector (0.0.3)
+    diff-lcs (1.3)
+    docile (1.3.2)
+    dogstatsd-ruby (4.8.2)
+    erubi (1.9.0)
+    et-orbi (1.2.4)
+      tzinfo
+    factory_girl (4.9.0)
+      activesupport (>= 3.0.0)
+    fugit (1.4.2)
+      et-orbi (~> 1.1, >= 1.1.8)
+      raabro (~> 1.4)
+    globalid (0.4.2)
+      activesupport (>= 4.2.0)
+    i18n (1.8.2)
+      concurrent-ruby (~> 1.0)
+    inst-jobs (1.0.4)
+      activerecord (>= 4.2)
+      activesupport (>= 4.2)
+      after_transaction_commit (>= 1.0, < 3)
+      debug_inspector (~> 0.0.3)
+      fugit (~> 1.3)
+      railties (>= 4.2)
+      redis (> 3.0)
+      redis-scripting (~> 1.0.1)
+    inst_statsd (2.1.6)
+      aroi (~> 0.0.7)
+      dogstatsd-ruby (~> 4.2)
+      statsd-ruby (~> 1.0)
+    json (2.3.0)
+    loofah (2.4.0)
+      crass (~> 1.0.2)
+      nokogiri (>= 1.5.9)
+    mail (2.7.1)
+      mini_mime (>= 0.1.1)
+    method_source (0.9.2)
+    mini_mime (1.0.2)
+    mini_portile2 (2.4.0)
+    minitest (5.14.0)
+    multi_json (1.14.1)
+    mustermann (1.1.1)
+      ruby2_keywords (~> 0.0.1)
+    nio4r (2.5.2)
+    nokogiri (1.10.7)
+      mini_portile2 (~> 2.4.0)
+    parser (2.7.0.2)
+      ast (~> 2.4.0)
+    pg (1.2.2)
+    powerpack (0.1.2)
+    pry (0.12.2)
+      coderay (~> 1.1.0)
+      method_source (~> 0.9.0)
+    raabro (1.4.0)
+    rack (2.1.1)
+    rack-protection (2.0.8.1)
+      rack
+    rack-test (1.1.0)
+      rack (>= 1.0, < 3)
+    rails (5.1.7)
+      actioncable (= 5.1.7)
+      actionmailer (= 5.1.7)
+      actionpack (= 5.1.7)
+      actionview (= 5.1.7)
+      activejob (= 5.1.7)
+      activemodel (= 5.1.7)
+      activerecord (= 5.1.7)
+      activesupport (= 5.1.7)
+      bundler (>= 1.3.0)
+      railties (= 5.1.7)
+      sprockets-rails (>= 2.0.0)
+    rails-dom-testing (2.0.3)
+      activesupport (>= 4.2.0)
+      nokogiri (>= 1.6)
+    rails-html-sanitizer (1.3.0)
+      loofah (~> 2.3)
+    railties (5.1.7)
+      actionpack (= 5.1.7)
+      activesupport (= 5.1.7)
+      method_source
+      rake (>= 0.8.7)
+      thor (>= 0.18.1, < 2.0)
+    rainbow (2.2.2)
+      rake
+    rake (13.0.1)
+    redis (4.2.5)
+    redis-scripting (1.0.1)
+      redis (>= 3.0)
+    rspec (3.9.0)
+      rspec-core (~> 3.9.0)
+      rspec-expectations (~> 3.9.0)
+      rspec-mocks (~> 3.9.0)
+    rspec-core (3.9.1)
+      rspec-support (~> 3.9.1)
+    rspec-expectations (3.9.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.9.0)
+    rspec-mocks (3.9.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.9.0)
+    rspec-support (3.9.2)
+    rubocop (0.48.1)
+      parser (>= 2.3.3.1, < 3.0)
+      powerpack (~> 0.1)
+      rainbow (>= 1.99.1, < 3.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (~> 1.0, >= 1.0.1)
+    ruby-progressbar (1.10.1)
+    ruby2_keywords (0.0.2)
+    simplecov (0.17.1)
+      docile (~> 1.1)
+      json (>= 1.8, < 3)
+      simplecov-html (~> 0.10.0)
+    simplecov-html (0.10.2)
+    sinatra (2.0.8.1)
+      mustermann (~> 1.0)
+      rack (~> 2.0)
+      rack-protection (= 2.0.8.1)
+      tilt (~> 2.0)
+    sinatra-contrib (2.0.8.1)
+      backports (>= 2.8.2)
+      multi_json
+      mustermann (~> 1.0)
+      rack-protection (= 2.0.8.1)
+      sinatra (= 2.0.8.1)
+      tilt (~> 2.0)
+    sprockets (3.7.2)
+      concurrent-ruby (~> 1.0)
+      rack (> 1, < 3)
+    sprockets-rails (3.2.1)
+      actionpack (>= 4.0)
+      activesupport (>= 4.0)
+      sprockets (>= 3.0.0)
+    statsd-ruby (1.5.0)
+    thor (1.0.1)
+    thread_safe (0.3.6)
+    tilt (2.0.10)
+    timecop (0.9.1)
+    tzinfo (1.2.6)
+      thread_safe (~> 0.1)
+    unicode-display_width (1.6.1)
+    websocket-driver (0.6.5)
+      websocket-extensions (>= 0.1.0)
+    websocket-extensions (0.1.4)
+    wwtd (1.4.0)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  bump
+  bundler
+  byebug
+  database_cleaner (~> 1.7)
+  factory_girl
+  inst-jobs-statsd!
+  pg (~> 1.2)
+  pry
+  rails (~> 5.1.7)
+  rake
+  rspec (~> 3.9)
+  rubocop (~> 0.48.0)
+  simplecov (~> 0.17)
+  sinatra (~> 2.0.8)
+  sinatra-contrib (~> 2.0.8)
+  sprockets (~> 3.7)
+  timecop
+  wwtd (~> 1.4.0)
+
+BUNDLED WITH
+   1.17.3

data/spec/gemfiles/60.gemfile.lock

@@ -0,0 +1,259 @@
+PATH
+  remote: ../..
+  specs:
+    inst-jobs-statsd (2.1.0)
+      inst-jobs (> 1.0, < 3.0)
+      inst_statsd (>= 2.1.2, < 3.0)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    actioncable (6.1.3)
+      actionpack (= 6.1.3)
+      activesupport (= 6.1.3)
+      nio4r (~> 2.0)
+      websocket-driver (>= 0.6.1)
+    actionmailbox (6.1.3)
+      actionpack (= 6.1.3)
+      activejob (= 6.1.3)
+      activerecord (= 6.1.3)
+      activestorage (= 6.1.3)
+      activesupport (= 6.1.3)
+      mail (>= 2.7.1)
+    actionmailer (6.1.3)
+      actionpack (= 6.1.3)
+      actionview (= 6.1.3)
+      activejob (= 6.1.3)
+      activesupport (= 6.1.3)
+      mail (~> 2.5, >= 2.5.4)
+      rails-dom-testing (~> 2.0)
+    actionpack (6.1.3)
+      actionview (= 6.1.3)
+      activesupport (= 6.1.3)
+      rack (~> 2.0, >= 2.0.9)
+      rack-test (>= 0.6.3)
+      rails-dom-testing (~> 2.0)
+      rails-html-sanitizer (~> 1.0, >= 1.2.0)
+    actiontext (6.1.3)
+      actionpack (= 6.1.3)
+      activerecord (= 6.1.3)
+      activestorage (= 6.1.3)
+      activesupport (= 6.1.3)
+      nokogiri (>= 1.8.5)
+    actionview (6.1.3)
+      activesupport (= 6.1.3)
+      builder (~> 3.1)
+      erubi (~> 1.4)
+      rails-dom-testing (~> 2.0)
+      rails-html-sanitizer (~> 1.1, >= 1.2.0)
+    activejob (6.1.3)
+      activesupport (= 6.1.3)
+      globalid (>= 0.3.6)
+    activemodel (6.1.3)
+      activesupport (= 6.1.3)
+    activerecord (6.1.3)
+      activemodel (= 6.1.3)
+      activesupport (= 6.1.3)
+    activestorage (6.1.3)
+      actionpack (= 6.1.3)
+      activejob (= 6.1.3)
+      activerecord (= 6.1.3)
+      activesupport (= 6.1.3)
+      marcel (~> 0.3.1)
+      mimemagic (~> 0.3.2)
+    activesupport (6.1.3)
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
+      zeitwerk (~> 2.3)
+    after_transaction_commit (2.2.1)
+      activerecord (>= 5.2)
+    aroi (0.0.7)
+      rails (>= 3.2)
+    ast (2.4.2)
+    backports (3.20.2)
+    builder (3.2.4)
+    bump (0.10.0)
+    byebug (11.1.3)
+    coderay (1.1.3)
+    concurrent-ruby (1.1.8)
+    crass (1.0.6)
+    database_cleaner (1.99.0)
+    debug_inspector (0.0.3)
+    diff-lcs (1.4.4)
+    docile (1.3.5)
+    dogstatsd-ruby (4.8.2)
+    erubi (1.10.0)
+    et-orbi (1.2.4)
+      tzinfo
+    factory_girl (4.9.0)
+      activesupport (>= 3.0.0)
+    fugit (1.4.2)
+      et-orbi (~> 1.1, >= 1.1.8)
+      raabro (~> 1.4)
+    globalid (0.4.2)
+      activesupport (>= 4.2.0)
+    i18n (1.8.9)
+      concurrent-ruby (~> 1.0)
+    inst-jobs (2.2.0)
+      activerecord (>= 4.2)
+      activesupport (>= 4.2)
+      after_transaction_commit (>= 1.0, < 3)
+      debug_inspector (~> 0.0.3)
+      fugit (~> 1.3)
+      railties (>= 4.2)
+      redis (> 3.0)
+      redis-scripting (~> 1.0.1)
+    inst_statsd (2.1.6)
+      aroi (~> 0.0.7)
+      dogstatsd-ruby (~> 4.2)
+      statsd-ruby (~> 1.0)
+    loofah (2.9.0)
+      crass (~> 1.0.2)
+      nokogiri (>= 1.5.9)
+    mail (2.7.1)
+      mini_mime (>= 0.1.1)
+    marcel (0.3.3)
+      mimemagic (~> 0.3.2)
+    method_source (1.0.0)
+    mimemagic (0.3.5)
+    mini_mime (1.0.2)
+    mini_portile2 (2.5.0)
+    minitest (5.14.3)
+    multi_json (1.15.0)
+    mustermann (1.1.1)
+      ruby2_keywords (~> 0.0.1)
+    nio4r (2.5.5)
+    nokogiri (1.11.1)
+      mini_portile2 (~> 2.5.0)
+      racc (~> 1.4)
+    parser (2.7.2.0)
+      ast (~> 2.4.1)
+    pg (1.2.3)
+    powerpack (0.1.3)
+    pry (0.14.0)
+      coderay (~> 1.1)
+      method_source (~> 1.0)
+    raabro (1.4.0)
+    racc (1.5.2)
+    rack (2.2.3)
+    rack-protection (2.0.8.1)
+      rack
+    rack-test (1.1.0)
+      rack (>= 1.0, < 3)
+    rails (6.1.3)
+      actioncable (= 6.1.3)
+      actionmailbox (= 6.1.3)
+      actionmailer (= 6.1.3)
+      actionpack (= 6.1.3)
+      actiontext (= 6.1.3)
+      actionview (= 6.1.3)
+      activejob (= 6.1.3)
+      activemodel (= 6.1.3)
+      activerecord (= 6.1.3)
+      activestorage (= 6.1.3)
+      activesupport (= 6.1.3)
+      bundler (>= 1.15.0)
+      railties (= 6.1.3)
+      sprockets-rails (>= 2.0.0)
+    rails-dom-testing (2.0.3)
+      activesupport (>= 4.2.0)
+      nokogiri (>= 1.6)
+    rails-html-sanitizer (1.3.0)
+      loofah (~> 2.3)
+    railties (6.1.3)
+      actionpack (= 6.1.3)
+      activesupport (= 6.1.3)
+      method_source
+      rake (>= 0.8.7)
+      thor (~> 1.0)
+    rainbow (2.2.2)
+      rake
+    rake (13.0.3)
+    redis (4.2.5)
+    redis-scripting (1.0.1)
+      redis (>= 3.0)
+    rspec (3.10.0)
+      rspec-core (~> 3.10.0)
+      rspec-expectations (~> 3.10.0)
+      rspec-mocks (~> 3.10.0)
+    rspec-core (3.10.1)
+      rspec-support (~> 3.10.0)
+    rspec-expectations (3.10.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.10.0)
+    rspec-mocks (3.10.2)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.10.0)
+    rspec-support (3.10.2)
+    rubocop (0.48.1)
+      parser (>= 2.3.3.1, < 3.0)
+      powerpack (~> 0.1)
+      rainbow (>= 1.99.1, < 3.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (~> 1.0, >= 1.0.1)
+    ruby-progressbar (1.11.0)
+    ruby2_keywords (0.0.4)
+    simplecov (0.21.2)
+      docile (~> 1.1)
+      simplecov-html (~> 0.11)
+      simplecov_json_formatter (~> 0.1)
+    simplecov-html (0.12.3)
+    simplecov_json_formatter (0.1.2)
+    sinatra (2.0.8.1)
+      mustermann (~> 1.0)
+      rack (~> 2.0)
+      rack-protection (= 2.0.8.1)
+      tilt (~> 2.0)
+    sinatra-contrib (2.0.8.1)
+      backports (>= 2.8.2)
+      multi_json
+      mustermann (~> 1.0)
+      rack-protection (= 2.0.8.1)
+      sinatra (= 2.0.8.1)
+      tilt (~> 2.0)
+    sprockets (4.0.2)
+      concurrent-ruby (~> 1.0)
+      rack (> 1, < 3)
+    sprockets-rails (3.2.2)
+      actionpack (>= 4.0)
+      activesupport (>= 4.0)
+      sprockets (>= 3.0.0)
+    statsd-ruby (1.5.0)
+    thor (1.1.0)
+    tilt (2.0.10)
+    timecop (0.9.4)
+    tzinfo (2.0.4)
+      concurrent-ruby (~> 1.0)
+    unicode-display_width (1.7.0)
+    websocket-driver (0.7.3)
+      websocket-extensions (>= 0.1.0)
+    websocket-extensions (0.1.5)
+    wwtd (1.4.1)
+    zeitwerk (2.4.2)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  bump
+  bundler
+  byebug
+  database_cleaner (~> 1.7)
+  factory_girl
+  inst-jobs-statsd!
+  pg (~> 1.2)
+  pry
+  rails (~> 6.0)
+  rake
+  rspec (~> 3.9)
+  rubocop (~> 0.48.0)
+  simplecov (~> 0.17)
+  sinatra (~> 2.0.8)
+  sinatra-contrib (~> 2.0.8)
+  timecop
+  wwtd (~> 1.4.0)
+
+BUNDLED WITH
+   2.1.4

data/spec/inst_jobs_statsd/ext/job_spec.rb

@@ -0,0 +1,15 @@
+RSpec.describe 'InstJobsStatsd::Ext::Job' do
+  describe 'sends count on job failure' do
+    before do
+      InstJobsStatsd::JobsTracker.new
+    end
+    let(:x) { Struct.new(:perform).new(true) }
+    it 'sends a stat' do
+      expect(InstStatsd::Statsd).to receive(:count)
+        .with(array_including(/\.failed$/), 1, 1, short_stat: :failed, tags: {})
+
+      x.delay.perform
+      Delayed::Job.first.fail!
+    end
+  end
+end

data/spec/inst_jobs_statsd/jobs_tracker_spec.rb

@@ -12,7 +12,6 @@ RSpec.describe InstJobsStatsd::JobsTracker do
 
   describe '.initialize' do
     it 'enables everything' do
-      expect(InstJobsStatsd::Stats::Counters::Failed).to receive(:enable)
       expect(InstJobsStatsd::Stats::Counters::Run).to receive(:enable)
 
       expect(InstJobsStatsd::Stats::Periodic::Failed).to receive(:enable)

data/spec/inst_jobs_statsd/naming_spec.rb

@@ -44,12 +44,12 @@ RSpec.describe InstJobsStatsd::Naming do
     describe '.dd_job_tags' do
       it 'works' do
         job = double(tag: 'Account.run_reports_later', shard: double(id: 101), strand: 'special')
-        expect(InstJobsStatsd::Naming.dd_job_tags(job)).to eq({ tag: 'Account.run_reports_later', jobshard: 101})
+        expect(InstJobsStatsd::Naming.dd_job_tags(job)).to eq(tag: 'Account.run_reports_later', jobshard: 101)
       end
 
       it 'properly munges job tags' do
         job = double(tag: 'Quizzes::Quiz#do_something', strand: nil)
-        expect(InstJobsStatsd::Naming.dd_job_tags(job)).to eq({ tag: 'Quizzes-Quiz.do_something'})
+        expect(InstJobsStatsd::Naming.dd_job_tags(job)).to eq(tag: 'Quizzes-Quiz.do_something')
       end
     end
   end

data/spec/inst_jobs_statsd/stats/counters/run_spec.rb

@@ -18,7 +18,7 @@ RSpec.describe InstJobsStatsd::Stats::Counters::Run do
 
     it do
       expect(InstStatsd::Statsd).to receive(:count)
-        .twice.with(array_including(/\.run$/), 1, 1, { short_stat: anything, tags: {} })
+        .twice.with(array_including(/\.run$/), 1, 1, short_stat: anything, tags: {})
       Delayed::Job.all.each do |job|
         Delayed::Worker.lifecycle.run_callbacks(:perform, {}, job) {}
       end

data/spec/inst_jobs_statsd/stats/periodic/failed_spec.rb

@@ -24,7 +24,7 @@ RSpec.describe InstJobsStatsd::Stats::Periodic::Failed do
 
     it do
       expect(InstStatsd::Statsd).to receive(:gauge)
-        .with(array_including(/\.failed_depth$/), 1, 1, { short_stat: anything, tags: {} })
+        .with(array_including(/\.failed_depth$/), 1, 1, short_stat: anything, tags: {})
       InstJobsStatsd::Stats::Periodic::Failed.report_failed_depth
     end
   end

data/spec/inst_jobs_statsd/stats/periodic/queue_spec.rb

@@ -28,13 +28,13 @@ RSpec.describe InstJobsStatsd::Stats::Periodic::Queue do
 
     it do
       expect(InstStatsd::Statsd).to receive(:gauge)
-        .with(array_including(/\.queue_depth$/), 1, 1, { short_stat: anything, tags: {} })
+        .with(array_including(/\.queue_depth$/), 1, 1, short_stat: anything, tags: {})
       InstJobsStatsd::Stats::Periodic::Queue.report_queue_depth
     end
 
     it do
       expect(InstStatsd::Statsd).to receive(:gauge)
-        .with(array_including(/\.queue_depth$/), 2, 1, { short_stat: anything, tags: {} })
+        .with(array_including(/\.queue_depth$/), 2, 1, short_stat: anything, tags: {})
       Timecop.freeze(2.minutes.from_now) do
         InstJobsStatsd::Stats::Periodic::Queue.report_queue_depth
       end
@@ -42,7 +42,7 @@ RSpec.describe InstJobsStatsd::Stats::Periodic::Queue do
 
     it do
       expect(InstStatsd::Statsd).to receive(:gauge)
-        .with(array_including(/\.queue_depth$/), 3, 1, { short_stat: anything, tags: {} })
+        .with(array_including(/\.queue_depth$/), 3, 1, short_stat: anything, tags: {})
       Timecop.freeze(20.minutes.from_now) do
         InstJobsStatsd::Stats::Periodic::Queue.report_queue_depth
       end
@@ -66,17 +66,17 @@ RSpec.describe InstJobsStatsd::Stats::Periodic::Queue do
 
     it do
       expect(InstStatsd::Statsd).to receive(:gauge)
-        .ordered.with(array_including(/\.queue_age_total$/), number_near(0), 1, { short_stat: anything, tags: {} })
+        .ordered.with(array_including(/\.queue_age_total$/), number_near(0), 1, short_stat: anything, tags: {})
       expect(InstStatsd::Statsd).to receive(:gauge)
-        .ordered.with(array_including(/\.queue_age_max$/), number_near(0), 1, { short_stat: anything, tags: {} })
+        .ordered.with(array_including(/\.queue_age_max$/), number_near(0), 1, short_stat: anything, tags: {})
       InstJobsStatsd::Stats::Periodic::Queue.report_queue_age
     end
 
     it do
       expect(InstStatsd::Statsd).to receive(:gauge)
-        .ordered.with(array_including(/\.queue_age_total$/), number_near(180), 1, { short_stat: anything, tags: {} })
+        .ordered.with(array_including(/\.queue_age_total$/), number_near(180), 1, short_stat: anything, tags: {})
       expect(InstStatsd::Statsd).to receive(:gauge)
-        .ordered.with(array_including(/\.queue_age_max$/), number_near(120), 1, { short_stat: anything, tags: {} })
+        .ordered.with(array_including(/\.queue_age_max$/), number_near(120), 1, short_stat: anything, tags: {})
       Timecop.freeze(2.minutes.from_now) do
         InstJobsStatsd::Stats::Periodic::Queue.report_queue_age
       end
@@ -84,9 +84,9 @@ RSpec.describe InstJobsStatsd::Stats::Periodic::Queue do
 
     it do
       expect(InstStatsd::Statsd).to receive(:gauge)
-        .ordered.with(array_including(/\.queue_age_total$/), number_near(2940), 1, { short_stat: anything, tags: {} })
+        .ordered.with(array_including(/\.queue_age_total$/), number_near(2940), 1, short_stat: anything, tags: {})
       expect(InstStatsd::Statsd).to receive(:gauge)
-        .ordered.with(array_including(/\.queue_age_max$/), number_near(1200), 1, { short_stat: anything, tags: {} })
+        .ordered.with(array_including(/\.queue_age_max$/), number_near(1200), 1, short_stat: anything, tags: {})
       Timecop.freeze(20.minutes.from_now) do
         InstJobsStatsd::Stats::Periodic::Queue.report_queue_age
       end

data/spec/inst_jobs_statsd/stats/periodic/run_spec.rb

@@ -25,7 +25,7 @@ RSpec.describe InstJobsStatsd::Stats::Periodic::Run do
 
     it do
       expect(InstStatsd::Statsd).to receive(:gauge)
-        .with(array_including(/\.run_depth$/), 1, 1, { short_stat: anything, tags: {} })
+        .with(array_including(/\.run_depth$/), 1, 1, short_stat: anything, tags: {})
       InstJobsStatsd::Stats::Periodic::Run.report_run_depth
     end
   end
@@ -44,9 +44,9 @@ RSpec.describe InstJobsStatsd::Stats::Periodic::Run do
 
     it do
       expect(InstStatsd::Statsd).to receive(:gauge)
-        .ordered.with(array_including(/\.run_age_total$/), number_near(0), 1, { short_stat: anything, tags: {} })
+        .ordered.with(array_including(/\.run_age_total$/), number_near(0), 1, short_stat: anything, tags: {})
       expect(InstStatsd::Statsd).to receive(:gauge)
-        .ordered.with(array_including(/\.run_age_max$/), number_near(0), 1, { short_stat: anything, tags: {} })
+        .ordered.with(array_including(/\.run_age_max$/), number_near(0), 1, short_stat: anything, tags: {})
       InstJobsStatsd::Stats::Periodic::Run.report_run_age
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: inst-jobs-statsd
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 2.1.1
 platform: ruby
 authors:
 - Jeremy Slade
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-12-08 00:00:00.000000000 Z
+date: 2021-03-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: inst-jobs
@@ -241,10 +241,10 @@ extra_rdoc_files: []
 files:
 - lib/inst-jobs-statsd.rb
 - lib/inst_jobs_statsd/default_tracking.rb
+- lib/inst_jobs_statsd/ext/job.rb
 - lib/inst_jobs_statsd/jobs_tracker.rb
 - lib/inst_jobs_statsd/naming.rb
 - lib/inst_jobs_statsd/stats/counters.rb
-- lib/inst_jobs_statsd/stats/counters/failed.rb
 - lib/inst_jobs_statsd/stats/counters/run.rb
 - lib/inst_jobs_statsd/stats/periodic.rb
 - lib/inst_jobs_statsd/stats/periodic/failed.rb
@@ -258,11 +258,14 @@ files:
 - spec/factories/jobs.rb
 - spec/factories/workers.rb
 - spec/gemfiles/42.gemfile
+- spec/gemfiles/42.gemfile.lock
 - spec/gemfiles/51.gemfile
+- spec/gemfiles/51.gemfile.lock
 - spec/gemfiles/60.gemfile
+- spec/gemfiles/60.gemfile.lock
+- spec/inst_jobs_statsd/ext/job_spec.rb
 - spec/inst_jobs_statsd/jobs_tracker_spec.rb
 - spec/inst_jobs_statsd/naming_spec.rb
-- spec/inst_jobs_statsd/stats/counters/failed_spec.rb
 - spec/inst_jobs_statsd/stats/counters/run_spec.rb
 - spec/inst_jobs_statsd/stats/periodic/failed_spec.rb
 - spec/inst_jobs_statsd/stats/periodic/queue_spec.rb
@@ -300,24 +303,27 @@ signing_key:
 specification_version: 4
 summary: Stats reporting for inst-jobs
 test_files:
+- spec/spec_helper.rb
+- spec/inst_statsd/default_tracking_spec.rb
+- spec/setup_test_db.rb
 - spec/inst_jobs_statsd/naming_spec.rb
+- spec/inst_jobs_statsd/jobs_tracker_spec.rb
+- spec/inst_jobs_statsd/stats/timing/perform_spec.rb
+- spec/inst_jobs_statsd/stats/timing/pop_spec.rb
+- spec/inst_jobs_statsd/stats/timing/failed_spec.rb
+- spec/inst_jobs_statsd/stats/periodic/queue_spec.rb
 - spec/inst_jobs_statsd/stats/periodic/failed_spec.rb
 - spec/inst_jobs_statsd/stats/periodic/run_spec.rb
-- spec/inst_jobs_statsd/stats/periodic/queue_spec.rb
-- spec/inst_jobs_statsd/stats/timing/failed_spec.rb
-- spec/inst_jobs_statsd/stats/timing/pop_spec.rb
-- spec/inst_jobs_statsd/stats/timing/perform_spec.rb
-- spec/inst_jobs_statsd/stats/periodic_spec.rb
-- spec/inst_jobs_statsd/stats/counters/failed_spec.rb
 - spec/inst_jobs_statsd/stats/counters/run_spec.rb
 - spec/inst_jobs_statsd/stats/timing_spec.rb
-- spec/inst_jobs_statsd/jobs_tracker_spec.rb
-- spec/spec_helper.rb
-- spec/setup_test_db.rb
+- spec/inst_jobs_statsd/stats/periodic_spec.rb
+- spec/inst_jobs_statsd/ext/job_spec.rb
 - spec/matchers.rb
+- spec/gemfiles/51.gemfile
+- spec/gemfiles/51.gemfile.lock
+- spec/gemfiles/42.gemfile.lock
 - spec/gemfiles/60.gemfile
+- spec/gemfiles/60.gemfile.lock
 - spec/gemfiles/42.gemfile
-- spec/gemfiles/51.gemfile
 - spec/factories/workers.rb
 - spec/factories/jobs.rb
-- spec/inst_statsd/default_tracking_spec.rb

data/lib/inst_jobs_statsd/stats/counters/failed.rb

@@ -1,28 +0,0 @@
-module InstJobsStatsd
-  module Stats
-    module Counters
-      module Failed
-        def self.enable
-          enable_failed_count
-        end
-
-        def self.enable_failed_count
-          return if Delayed::Job::Failed < AfterCreateHook
-          Delayed::Job::Failed.include AfterCreateHook
-        end
-
-        module AfterCreateHook
-          def self.included(base)
-            base.after_create do
-              InstJobsStatsd::Stats::Counters::Failed.report_failed_count(self)
-            end
-          end
-        end
-
-        def self.report_failed_count(job)
-          Counters.report_count(:failed, 1, job: job)
-        end
-      end
-    end
-  end
-end

data/spec/inst_jobs_statsd/stats/counters/failed_spec.rb

@@ -1,20 +0,0 @@
-RSpec.describe InstJobsStatsd::Stats::Counters::Failed do
-  describe '.enable' do
-    it 'enables all the things' do
-      expect(InstJobsStatsd::Stats::Counters::Failed).to receive(:enable_failed_count)
-      InstJobsStatsd::Stats::Counters::Failed.enable
-    end
-  end
-
-  describe '.report_failed_count' do
-    let(:x) { Struct.new(:perform).new(true) }
-    it do
-      expect(InstStatsd::Statsd).to receive(:count)
-        .with(array_including(/\.failed$/), 1, 1, { short_stat: :failed, tags: {} })
-
-      InstJobsStatsd::Stats::Counters::Failed.enable_failed_count
-      x.delay.perform
-      Delayed::Job.first.fail!
-    end
-  end
-end