exception_handling 2.8.1 → 2.10.0.pre.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 89460324d860cf57a63309d2a1b8b226c46fee136d795a17b326327a464c2fce
-  data.tar.gz: aabbbdd8a222ae751cb6e7355a77cd264c7f49a664afab898078316f1be67132
+  metadata.gz: c5c1447aa42d931a3c1536f82ca2f9ed8131736d993eb663cbb1b48b14450839
+  data.tar.gz: b62e05ededb07b0b0eb9cb40187c5fb98ae1bf57c6a94577174be6161c011199
 SHA512:
-  metadata.gz: a2f0320faf44e19026a70aa112e5199907c21be3e5856706ed95d0459bee832c218fd37c8ab250f0ebba12886002301b18f966129fb265746f258140c827db80
-  data.tar.gz: 0acc4678afc2988d5c265dee7b9e6dc9bd15e0cffdcdecb95f5da8d3c8811dbc1387363674e715bd9ab4941038fb0290201bc1bc4a21cbc23760b42b87299086
+  metadata.gz: 2bb9077980b89d4535e53447b18e0cdf7d188e497c48e1d51db30a22ad31474d6b006f64023e12003775d21633960a5a99420f48ffeea783a3c219976f3570c3
+  data.tar.gz: 4c0e1c5712b3250255808d49faf9c4af26e78cd218a45b720fae552edf104366d168deb4ce86732910e46082bf4539c3d42978a1fd07d49ab6bbb0045b1a1cc8

data/.github/workflows/pipeline.yml

@@ -0,0 +1,27 @@
+---
+name: Pipeline
+on: [push]
+jobs:
+  tests:
+    name: Unit Tests
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby: [2.5, 2.6]
+        gemfile:
+        - Gemfile
+        - gemfiles/rails_4.gemfile
+        - gemfiles/rails_5.gemfile
+        - gemfiles/rails_6.gemfile
+    env:
+      BUNDLE_GEMFILE: ${{ matrix.gemfile }}
+    steps:
+    - uses: actions/checkout@v2
+    - uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby }}
+        bundler: 1.17.3
+        bundler-cache: true
+    - name: Unit tests
+      run: bundle exec rspec

data/CHANGELOG.md

@@ -4,11 +4,23 @@ Inspired by [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 Note: this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [2.8.1] - Unreleased
+## [2.10.0] - Unreleased
+### Removed
+- Remove custom object inspection
+  This removed Honeybadger-specific callbacks (`lib/exception_handling/honeybadger_callbacks.rb`)
+
+### Deprecated
+- Deprecated use of Honeybadger fork
+
+## [2.9.0] - 2020-03-02
+### Added
+- Automatically registers with the `escalate` gem's `on_escalate` callback.
+
+## [2.8.1] - 2020-12-01
 ### Added
 - If the `log_context` key `honeybadger_grouping:` is set, pass that value to the `controller:` keyword argument of `HoneyBadger.notify`.
 
-## [2.8.0] - Unreleased
+## [2.8.0] - 2020-10-19
 ### Deprecated
 - Deprecated Email Escalation Methods: `escalate_to_production_support`, `escalate_error`, `escalate_warning`, `ensure_escalation`
 
@@ -60,6 +72,7 @@ Note: this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0
 ### Changed
 - No longer depends on hobo_support. Uses invoca-utils 0.3 instead.
 
+[2.9.0]: https://github.com/Invoca/exception_handling/compare/v2.8.1...v2.9.0
 [2.8.1]: https://github.com/Invoca/exception_handling/compare/v2.8.0...v2.8.1
 [2.8.0]: https://github.com/Invoca/exception_handling/compare/v2.7.0...v2.8.0
 [2.7.0]: https://github.com/Invoca/exception_handling/compare/v2.6.1...v2.7.0

data/Gemfile

@@ -2,10 +2,17 @@
 
 source 'https://rubygems.org'
 
+git_source(:github) do |repo_name|
+  repo_name = "#{repo_name}/#{repo_name}" unless repo_name.include?("/")
+  "https://github.com/#{repo_name}.git"
+end
+
 gemspec
 
+gem 'actionmailer', '< 6.1'
+gem 'activesupport', '< 6.1'
 gem 'appraisal', '~> 2.2'
-gem 'honeybadger', '3.3.1-1', git: 'git@github.com:Invoca/honeybadger-ruby', ref: 'bb5f2b8a86e4147c38a6270d39ad610fab4dd5e6'
+gem 'honeybadger'
 gem 'pry'
 gem 'pry-byebug'
 gem 'rake'

data/Gemfile.lock

@@ -1,47 +1,41 @@
-GIT
-  remote: git@github.com:Invoca/honeybadger-ruby
-  revision: bb5f2b8a86e4147c38a6270d39ad610fab4dd5e6
-  ref: bb5f2b8a86e4147c38a6270d39ad610fab4dd5e6
-  specs:
-    honeybadger (3.3.1.pre.1)
-
 PATH
   remote: .
   specs:
-    exception_handling (2.8.1)
+    exception_handling (2.10.0.pre.1)
       actionmailer (>= 4.2, < 7.0)
       actionpack (>= 4.2, < 7.0)
       activesupport (>= 4.2, < 7.0)
       contextual_logger (~> 0.7)
+      escalate (~> 0.3)
       eventmachine (~> 1.0)
       invoca-utils (~> 0.3)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    actionmailer (6.0.3.4)
-      actionpack (= 6.0.3.4)
-      actionview (= 6.0.3.4)
-      activejob (= 6.0.3.4)
+    actionmailer (6.0.3.5)
+      actionpack (= 6.0.3.5)
+      actionview (= 6.0.3.5)
+      activejob (= 6.0.3.5)
       mail (~> 2.5, >= 2.5.4)
       rails-dom-testing (~> 2.0)
-    actionpack (6.0.3.4)
-      actionview (= 6.0.3.4)
-      activesupport (= 6.0.3.4)
+    actionpack (6.0.3.5)
+      actionview (= 6.0.3.5)
+      activesupport (= 6.0.3.5)
       rack (~> 2.0, >= 2.0.8)
       rack-test (>= 0.6.3)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.0, >= 1.2.0)
-    actionview (6.0.3.4)
-      activesupport (= 6.0.3.4)
+    actionview (6.0.3.5)
+      activesupport (= 6.0.3.5)
       builder (~> 3.1)
       erubi (~> 1.4)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.1, >= 1.2.0)
-    activejob (6.0.3.4)
-      activesupport (= 6.0.3.4)
+    activejob (6.0.3.5)
+      activesupport (= 6.0.3.5)
       globalid (>= 0.3.6)
-    activesupport (6.0.3.4)
+    activesupport (6.0.3.5)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 0.7, < 2)
       minitest (~> 5.1)
@@ -55,32 +49,35 @@ GEM
     builder (3.2.4)
     byebug (11.1.3)
     coderay (1.1.2)
-    concurrent-ruby (1.1.7)
+    concurrent-ruby (1.1.8)
     contextual_logger (0.11.0)
       activesupport
       json
     crass (1.0.6)
     diff-lcs (1.4.4)
     erubi (1.10.0)
+    escalate (0.3.0)
     eventmachine (1.2.7)
     globalid (0.4.2)
       activesupport (>= 4.2.0)
-    i18n (1.8.5)
+    honeybadger (4.11.0)
+    i18n (1.8.9)
       concurrent-ruby (~> 1.0)
     invoca-utils (0.4.1)
     jaro_winkler (1.5.3)
-    json (2.3.1)
-    loofah (2.8.0)
+    json (2.6.1)
+    loofah (2.9.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.2)
-    nokogiri (1.10.10)
-      mini_portile2 (~> 2.4.0)
+    mini_portile2 (2.5.0)
+    minitest (5.14.4)
+    nokogiri (1.11.1)
+      mini_portile2 (~> 2.5.0)
+      racc (~> 1.4)
     parallel (1.17.0)
     parser (2.6.3.0)
       ast (~> 2.4.0)
@@ -91,6 +88,7 @@ GEM
     pry-byebug (3.8.0)
       byebug (~> 11.0)
       pry (~> 0.10)
+    racc (1.5.2)
     rack (2.2.3)
     rack-test (1.1.0)
       rack (>= 1.0, < 3)
@@ -128,7 +126,7 @@ GEM
       power_assert
     thor (1.0.1)
     thread_safe (0.3.6)
-    tzinfo (1.2.8)
+    tzinfo (1.2.9)
       thread_safe (~> 0.1)
     unicode-display_width (1.6.0)
     zeitwerk (2.4.2)
@@ -137,9 +135,11 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
+  actionmailer (< 6.1)
+  activesupport (< 6.1)
   appraisal (~> 2.2)
   exception_handling!
-  honeybadger (= 3.3.1.pre.1)!
+  honeybadger
   pry
   pry-byebug
   rake

data/exception_handling.gemspec

@@ -24,6 +24,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'actionpack',        '>= 4.2', '< 7.0'
   spec.add_dependency 'activesupport',     '>= 4.2', '< 7.0'
   spec.add_dependency 'contextual_logger', '~> 0.7'
+  spec.add_dependency 'escalate',          '~> 0.3'
   spec.add_dependency 'eventmachine',      '~> 1.0'
   spec.add_dependency 'invoca-utils',      '~> 0.3'
 end

data/gemfiles/rails_4.gemfile

@@ -2,8 +2,10 @@
 
 source "https://rubygems.org"
 
+gem "actionmailer", "~> 4.2"
+gem "activesupport", "~> 4.2"
 gem "appraisal", "~> 2.2"
-gem "honeybadger", "3.3.1-1", git: "git@github.com:Invoca/honeybadger-ruby", ref: "bb5f2b8a86e4147c38a6270d39ad610fab4dd5e6"
+gem "honeybadger"
 gem "pry"
 gem "pry-byebug"
 gem "rake"
@@ -11,8 +13,6 @@ gem "rspec"
 gem "rspec_junit_formatter"
 gem "rubocop"
 gem "test-unit"
-gem "actionmailer", "~> 4.2"
 gem "actionpack", "~> 4.2"
-gem "activesupport", "~> 4.2"
 
 gemspec path: "../"

data/gemfiles/rails_5.gemfile

@@ -2,8 +2,10 @@
 
 source "https://rubygems.org"
 
+gem "actionmailer", "~> 5.2"
+gem "activesupport", "~> 5.2"
 gem "appraisal", "~> 2.2"
-gem "honeybadger", "3.3.1-1", git: "git@github.com:Invoca/honeybadger-ruby", ref: "bb5f2b8a86e4147c38a6270d39ad610fab4dd5e6"
+gem "honeybadger"
 gem "pry"
 gem "pry-byebug"
 gem "rake"
@@ -11,8 +13,6 @@ gem "rspec"
 gem "rspec_junit_formatter"
 gem "rubocop"
 gem "test-unit"
-gem "actionmailer", "~> 5.2"
 gem "actionpack", "~> 5.2"
-gem "activesupport", "~> 5.2"
 
 gemspec path: "../"

data/gemfiles/rails_6.gemfile

@@ -2,8 +2,10 @@
 
 source "https://rubygems.org"
 
+gem "actionmailer", "~> 6.0"
+gem "activesupport", "~> 6.0"
 gem "appraisal", "~> 2.2"
-gem "honeybadger", "3.3.1-1", git: "git@github.com:Invoca/honeybadger-ruby", ref: "bb5f2b8a86e4147c38a6270d39ad610fab4dd5e6"
+gem "honeybadger"
 gem "pry"
 gem "pry-byebug"
 gem "rake"
@@ -11,8 +13,6 @@ gem "rspec"
 gem "rspec_junit_formatter"
 gem "rubocop"
 gem "test-unit"
-gem "actionmailer", "~> 6.0"
 gem "actionpack", "~> 6.0"
-gem "activesupport", "~> 6.0"
 
 gemspec path: "../"

data/lib/exception_handling/escalate_callback.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+require 'escalate'
+
+module ExceptionHandling
+  module EscalateCallback
+    class << self
+      def register_if_configured!
+        register! if ::ExceptionHandling.configured?
+      end
+
+      def register!
+        Escalate.on_escalate(log_first: false) do |exception, location_message, **context|
+          ::ExceptionHandling.log_error(exception, location_message, **context)
+        end
+      end
+    end
+  end
+end

data/lib/exception_handling/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ExceptionHandling
-  VERSION = '2.8.1'
+  VERSION = '2.10.0.pre.1'
 end

data/lib/exception_handling.rb

@@ -7,14 +7,14 @@ require 'contextual_logger'
 
 require 'invoca/utils'
 
-require "exception_handling/mailer"
-require "exception_handling/sensu"
-require "exception_handling/methods"
-require "exception_handling/log_stub_error"
-require "exception_handling/exception_description"
-require "exception_handling/exception_catalog"
-require "exception_handling/exception_info"
-require "exception_handling/honeybadger_callbacks.rb"
+require 'exception_handling/mailer'
+require 'exception_handling/sensu'
+require 'exception_handling/methods'
+require 'exception_handling/log_stub_error'
+require 'exception_handling/exception_description'
+require 'exception_handling/exception_catalog'
+require 'exception_handling/exception_info'
+require 'exception_handling/escalate_callback'
 
 _ = ActiveSupport::HashWithIndifferentAccess
 
@@ -52,6 +52,10 @@ module ExceptionHandling # never included
       @exception_recipients or raise ArgumentError, "You must assign a value to #{name}.exception_recipients"
     end
 
+    def configured?
+      !@logger.nil?
+    end
+
     def logger
       @logger or raise ArgumentError, "You must assign a value to #{name}.logger"
     end
@@ -63,6 +67,7 @@ module ExceptionHandling # never included
                   Deprecation3_0.deprecation_warning('implicit extend with ContextualLogger::LoggerMixin', 'extend your logger instance or include into your logger class first')
                   logger.extend(ContextualLogger::LoggerMixin)
                 end
+      EscalateCallback.register_if_configured!
     end
 
     def default_metric_name(exception_data, exception, treat_like_warning)
@@ -484,4 +489,6 @@ module ExceptionHandling # never included
       end
     end
   end
+
+  EscalateCallback.register_if_configured!
 end

data/spec/unit/exception_handling/escalate_callback_spec.rb

@@ -0,0 +1,81 @@
+# frozen_string_literal: true
+
+require 'exception_handling/escalate_callback'
+require File.expand_path('../../spec_helper',  __dir__)
+
+module ExceptionHandling
+  describe EscalateCallback do
+    before do
+      class TestGem
+        class << self
+          attr_accessor :logger
+        end
+        include Escalate.mixin
+      end
+      TestGem.logger = logger
+      Escalate.clear_on_escalate_callbacks
+    end
+
+    after do
+      Escalate.clear_on_escalate_callbacks
+    end
+
+    let(:exception) do
+      raise "boom!"
+    rescue => ex
+      ex
+    end
+    let(:location_message) { "happened in TestGem" }
+    let(:context_hash) { { cuid: 'AABBCD' } }
+    let(:logger) { double("logger") }
+
+    describe '.register_if_configured!' do
+      context 'when already configured' do
+        before do
+          @original_logger = ExceptionHandling.logger
+          ExceptionHandling.logger = ::Logger.new('/dev/null')
+        end
+
+        after do
+          ExceptionHandling.logger = @original_logger
+        end
+
+        it 'registers a callback' do
+          EscalateCallback.register_if_configured!
+
+          expect(logger).to_not receive(:error)
+          expect(logger).to_not receive(:fatal)
+          expect(ExceptionHandling).to receive(:log_error).with(exception, location_message, context_hash)
+
+          TestGem.escalate(exception, location_message, context: context_hash)
+        end
+      end
+
+      context 'when not yet configured' do
+        before do
+          @original_logger = ExceptionHandling.logger
+          ExceptionHandling.logger = nil
+        end
+
+        after do
+          ExceptionHandling.logger = @original_logger
+        end
+
+        it 'registers a callback once the logger is set' do
+          EscalateCallback.register_if_configured!
+
+          expect(Escalate.on_escalate_callbacks).to be_empty
+
+          ExceptionHandling.logger = ::Logger.new('/dev/null')
+          expect(Escalate.on_escalate_callbacks).to_not be_empty
+
+          expect(logger).to_not receive(:error)
+          expect(logger).to_not receive(:fatal)
+          expect(ExceptionHandling).to receive(:log_error).with(exception, location_message, context_hash)
+
+          TestGem.escalate(exception, location_message, context: context_hash)
+        end
+      end
+    end
+  end
+end

data/spec/unit/exception_handling/mailer_spec.rb

@@ -93,6 +93,5 @@ module ExceptionHandling
         end
       end
     end
-
   end
 end

data/spec/unit/exception_handling_spec.rb

@@ -117,7 +117,7 @@ describe ExceptionHandling do
       before { @original_logger = ExceptionHandling.logger }
       after { ExceptionHandling.logger = @original_logger }
 
-      it "store logger as-is if it has ContextualLogger::Mixin" do
+      it "stores logger as-is if it has ContextualLogger::Mixin" do
         logger = Logger.new('/dev/null')
         logger.extend(ContextualLogger::LoggerMixin)
         ancestors = logger.singleton_class.ancestors.*.name
@@ -126,33 +126,33 @@ describe ExceptionHandling do
         expect(ExceptionHandling.logger.singleton_class.ancestors.*.name).to eq(ancestors)
       end
 
-      it "allow logger = nil (no deprecation warning)" do
+      it "allows logger = nil (no deprecation warning)" do
         expect(STDERR).to receive(:puts).with(/DEPRECATION WARNING/).never
         ExceptionHandling.logger = nil
       end
 
-      it "[deprecated] mix in ContextualLogger::Mixin if not there" do
+      it "[deprecated] mixes in ContextualLogger::Mixin if not there" do
         expect(STDERR).to receive(:puts).with(/DEPRECATION WARNING: implicit extend with ContextualLogger::LoggerMixin is deprecated and will be removed from exception_handling 3\.0/)
         logger = Logger.new('/dev/null')
         ancestors = logger.singleton_class.ancestors.*.name
 
         ExceptionHandling.logger = logger
-        expect(ExceptionHandling.logger.singleton_class.ancestors.*.name).to_not eq(ancestors)
         expect(ExceptionHandling.logger).to be_kind_of(ContextualLogger::LoggerMixin)
+        expect(ExceptionHandling.logger.singleton_class.ancestors.*.name).to_not eq(ancestors)
       end
-    end
 
-    context "#log_error" do
-      it "take in additional logging context hash and pass it to the logger" do
-        ExceptionHandling.log_error('This is an Error', 'This is the prefix context', service_name: 'exception_handling')
-        expect(logged_excluding_reload_filter.last[:message]).to match(/This is an Error/)
-        expect(logged_excluding_reload_filter.last[:context]).to_not be_empty
-        expect(service_name: 'exception_handling').to eq(logged_excluding_reload_filter.last[:context])
-      end
+      context "#log_error" do
+        it "takes in additional logging context hash and pass it to the logger" do
+          ExceptionHandling.log_error('This is an Error', 'This is the prefix context', service_name: 'exception_handling')
+          expect(logged_excluding_reload_filter.last[:message]).to match(/This is an Error/)
+          expect(logged_excluding_reload_filter.last[:context]).to_not be_empty
+          expect(service_name: 'exception_handling').to eq(logged_excluding_reload_filter.last[:context])
+        end
 
-      it "log with Severity::FATAL" do
-        ExceptionHandling.log_error('This is a Warning', service_name: 'exception_handling')
-        expect('FATAL').to eq(logged_excluding_reload_filter.last[:severity])
+        it "logs with Severity::FATAL" do
+          ExceptionHandling.log_error('This is a Warning', service_name: 'exception_handling')
+          expect('FATAL').to eq(logged_excluding_reload_filter.last[:severity])
+        end
       end
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: exception_handling
 version: !ruby/object:Gem::Version
-  version: 2.8.1
+  version: 2.10.0.pre.1
 platform: ruby
 authors:
 - Invoca
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-12-01 00:00:00.000000000 Z
+date: 2022-03-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionmailer
@@ -84,6 +84,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.7'
+- !ruby/object:Gem::Dependency
+  name: escalate
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.3'
 - !ruby/object:Gem::Dependency
   name: eventmachine
   requirement: !ruby/object:Gem::Requirement
@@ -119,9 +133,8 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/pipeline.yml"
 - ".gitignore"
-- ".jenkins/Jenkinsfile"
-- ".jenkins/ruby_build_pod.yml"
 - ".rspec"
 - ".rubocop.yml"
 - ".ruby-version"
@@ -134,15 +147,14 @@ files:
 - Rakefile
 - config/exception_filters.yml
 - exception_handling.gemspec
-- gemfiles/.bundle/config
 - gemfiles/rails_4.gemfile
 - gemfiles/rails_5.gemfile
 - gemfiles/rails_6.gemfile
 - lib/exception_handling.rb
+- lib/exception_handling/escalate_callback.rb
 - lib/exception_handling/exception_catalog.rb
 - lib/exception_handling/exception_description.rb
 - lib/exception_handling/exception_info.rb
-- lib/exception_handling/honeybadger_callbacks.rb
 - lib/exception_handling/log_stub_error.rb
 - lib/exception_handling/logging_methods.rb
 - lib/exception_handling/mailer.rb
@@ -154,10 +166,10 @@ files:
 - spec/helpers/exception_helpers.rb
 - spec/rake_test_warning_false.rb
 - spec/spec_helper.rb
+- spec/unit/exception_handling/escalate_callback_spec.rb
 - spec/unit/exception_handling/exception_catalog_spec.rb
 - spec/unit/exception_handling/exception_description_spec.rb
 - spec/unit/exception_handling/exception_info_spec.rb
-- spec/unit/exception_handling/honeybadger_callbacks_spec.rb
 - spec/unit/exception_handling/log_error_stub_spec.rb
 - spec/unit/exception_handling/logging_methods_spec.rb
 - spec/unit/exception_handling/mailer_spec.rb
@@ -183,9 +195,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubygems_version: 3.0.3
 signing_key:
@@ -197,10 +209,10 @@ test_files:
 - spec/helpers/exception_helpers.rb
 - spec/rake_test_warning_false.rb
 - spec/spec_helper.rb
+- spec/unit/exception_handling/escalate_callback_spec.rb
 - spec/unit/exception_handling/exception_catalog_spec.rb
 - spec/unit/exception_handling/exception_description_spec.rb
 - spec/unit/exception_handling/exception_info_spec.rb
-- spec/unit/exception_handling/honeybadger_callbacks_spec.rb
 - spec/unit/exception_handling/log_error_stub_spec.rb
 - spec/unit/exception_handling/logging_methods_spec.rb
 - spec/unit/exception_handling/mailer_spec.rb

data/.jenkins/Jenkinsfile

@@ -1,113 +0,0 @@
-#!/usr/bin/groovy
-@Library('jenkins-pipeline@v0.4.5')
-import com.invoca.docker.*;
-pipeline {
-  agent {
-    kubernetes {
-      defaultContainer "ruby"
-      yamlFile ".jenkins/ruby_build_pod.yml"
-    }
-  }
-
-  environment {
-    GITHUB_TOKEN = credentials('github_token')
-    GITHUB_KEY = credentials('github_key')
-    BUNDLE_GEM__FURY__IO = credentials('gemfury_deploy_token')
-  }
-
-  stages {
-    stage('Setup') {
-      steps {
-        script {
-          updateGitHubStatus('clean-build', 'pending', 'Unit tests.')
-          sh '''
-            # get SSH setup inside the container
-            eval `ssh-agent -s`
-            echo "$GITHUB_KEY" | ssh-add -
-            mkdir -p /root/.ssh
-            ssh-keyscan -t rsa github.com > /root/.ssh/known_hosts
-
-            bundle install
-            bundle exec appraisal install
-          '''
-        }
-      }
-    }
-    stage('Appraisals') {
-      parallel {
-        stage('Current') {
-          environment {
-            JUNIT_OUTPUT = 'spec/reports/current'
-          }
-
-          steps {
-            sh "bundle exec rspec --format RspecJunitFormatter --out ${JUNIT_OUTPUT}/rspec.xml"
-          }
-
-          post {
-            always { junit "${JUNIT_OUTPUT}/*.xml" }
-          }
-        }
-
-        stage('Rails 4') {
-          environment {
-            JUNIT_OUTPUT = 'spec/reports/rails-4'
-          }
-
-          steps {
-            sh "bundle exec appraisal rails-4 rspec --format RspecJunitFormatter --out ${JUNIT_OUTPUT}/rspec.xml"
-          }
-
-          post {
-            always { junit "${JUNIT_OUTPUT}/*.xml" }
-          }
-        }
-
-        stage('Rails 5') {
-          environment {
-            JUNIT_OUTPUT = 'spec/reports/rails-5'
-          }
-
-          steps {
-            sh "bundle exec appraisal rails-5 rspec --format RspecJunitFormatter --out ${JUNIT_OUTPUT}/rspec.xml"
-          }
-
-          post {
-            always { junit "${JUNIT_OUTPUT}/*.xml" }
-          }
-        }
-
-        stage('Rails 6') {
-          environment {
-            JUNIT_OUTPUT = 'spec/reports/rails-6'
-          }
-
-          steps {
-            sh "bundle exec appraisal rails-6 rspec --format RspecJunitFormatter --out ${JUNIT_OUTPUT}/rspec.xml"
-          }
-
-          post {
-            always { junit "${JUNIT_OUTPUT}/*.xml" }
-          }
-        }
-      }
-
-      post {
-        success { updateGitHubStatus('clean-build', 'success', 'Unit tests.') }
-        failure { updateGitHubStatus('clean-build', 'failure', 'Unit tests.') }
-      }
-    }
-  }
-}
-
-void updateGitHubStatus(String context, String status, String description) {
-  gitHubStatus([
-    repoSlug:    'Invoca/exception_handling',
-    sha:         env.GIT_COMMIT,
-    description: description,
-    context:     context,
-    targetURL:   env.RUN_DISPLAY_URL,
-    token:       env.GITHUB_TOKEN,
-    status:      status
-  ])
-}

data/.jenkins/ruby_build_pod.yml

@@ -1,18 +0,0 @@
----
-apiVersion: v1
-kind: Pod
-metadata:
-  labels:
-  jenkins/exception-handling: 'true'
-  namespace: jenkins
-  name: exception-handling
-spec:
-  containers:
-  - name: ruby
-    image: ruby:2.6.5
-    tty: true
-    resources:
-    requests:
-      memory: "100Mi"
-    command:
-      - cat

data/gemfiles/.bundle/config

@@ -1,2 +0,0 @@
----
-BUNDLE_RETRY: "1"

data/lib/exception_handling/honeybadger_callbacks.rb

@@ -1,59 +0,0 @@
-# frozen_string_literal: true
-
-module ExceptionHandling
-  module HoneybadgerCallbacks
-    class << self
-      def register_callbacks
-        if ExceptionHandling.honeybadger_defined?
-          Honeybadger.local_variable_filter(&method(:local_variable_filter))
-        end
-      end
-
-      private
-
-      def inspect_object(object, filter_keys)
-        inspection_output = object.inspect
-
-        if contains_filter_key?(filter_keys, inspection_output)
-          filtered_object(object)
-        else
-          inspection_output
-        end
-      rescue => ex
-        details = if object.respond_to?(:to_pk)
-                    " @pk=#{object.to_pk}"
-                  elsif object.respond_to?(:id)
-                    " @id=#{object.id}"
-                  end
-
-        "#<#{object.class.name}#{details} [error '#{ex.class.name}: #{ex.message}' while calling #inspect]>"
-      end
-
-      def local_variable_filter(_symbol, object, filter_keys)
-        case object
-        # Honeybadger will filter these data types for us
-        when String, Hash, Array, Set, Numeric, TrueClass, FalseClass, NilClass
-          object
-        else # handle other Ruby objects, intended for POROs
-          inspect_object(object, filter_keys)
-        end
-      end
-
-      def contains_filter_key?(filter_keys, string)
-        filter_keys&.any? { |key| string.include?(key) }
-      end
-
-      def filtered_object(object)
-        # make the output look similar to inspect
-        # use [FILTERED], just like honeybadger does
-        if object.respond_to?(:to_pk)
-          "#<#{object.class.name} @pk=#{object.to_pk}, [FILTERED]>"
-        elsif object.respond_to?(:id)
-          "#<#{object.class.name} @id=#{object.id}, [FILTERED]>"
-        else
-          "#<#{object.class.name} [FILTERED]>"
-        end
-      end
-    end
-  end
-end

data/spec/unit/exception_handling/honeybadger_callbacks_spec.rb

@@ -1,122 +0,0 @@
-# frozen_string_literal: true
-
-require File.expand_path('../../spec_helper',  __dir__)
-
-module ExceptionHandling
-  describe HoneybadgerCallbacks do
-
-    class TestPoroWithAttribute
-      attr_reader :test_attribute
-
-      def initialize
-        @test_attribute = 'test'
-      end
-    end
-
-    class TestPoroWithFilteredAttribute
-      attr_reader :password
-
-      def initialize
-        @password = 'secret'
-      end
-    end
-
-    class TestPoroWithFilteredAttributeAndId < TestPoroWithFilteredAttribute
-      attr_reader :id
-
-      def initialize
-        super
-        @id = 1
-      end
-    end
-
-    class TestPoroWithFilteredAttributePkAndId < TestPoroWithFilteredAttributeAndId
-      def to_pk
-        'TestPoroWithFilteredAttributePkAndId_1'
-      end
-    end
-
-    class TestRaiseOnInspect < TestPoroWithAttribute
-      def inspect
-        raise "some error"
-      end
-    end
-
-    class TestRaiseOnInspectWithId < TestRaiseOnInspect
-      def id
-        123
-      end
-    end
-
-    class TestRaiseOnInspectWithToPk < TestRaiseOnInspect
-      def to_pk
-        "SomeRecord-123"
-      end
-    end
-
-    context "register_callbacks" do
-      it "store the callbacks in the honeybadger object" do
-        HoneybadgerCallbacks.register_callbacks
-        result = Honeybadger.config.local_variable_filter.call(:variable_name, 'test', [])
-        expect(result).to eq('test')
-      end
-    end
-
-    context "local_variable_filter" do
-      it "not inspect String, Hash, Array, Set, Numeric, TrueClass, FalseClass, NilClass" do
-        [
-          ['test', String],
-          [{ a: 1 }, Hash],
-          [[1, 2], Array],
-          [Set.new([1, 2]), Set],
-          [4.5, Numeric],
-          [true, TrueClass],
-          [false, FalseClass],
-          [nil, NilClass]
-        ].each do |object, expected_class|
-          result = HoneybadgerCallbacks.send(:local_variable_filter, :variable_name, object, [])
-          expect(result.is_a?(expected_class)).to be_truthy
-        end
-      end
-
-      it "inspect other classes" do
-        result = HoneybadgerCallbacks.send(:local_variable_filter, :variable_name, TestPoroWithAttribute.new, ['password'])
-        expect(result).to match(/#<ExceptionHandling::TestPoroWithAttribute:.* @test_attribute="test">/)
-      end
-
-      context "when inspect raises exceptions" do
-        it "handle exceptions for objects" do
-          result = HoneybadgerCallbacks.send(:local_variable_filter, :variable_name, TestRaiseOnInspect.new, ['password'])
-          expect(result).to eq("#<ExceptionHandling::TestRaiseOnInspect [error 'RuntimeError: some error' while calling #inspect]>")
-        end
-
-        it "handle exceptions for objects responding to id" do
-          result = HoneybadgerCallbacks.send(:local_variable_filter, :variable_name, TestRaiseOnInspectWithId.new, ['password'])
-          expect(result).to eq("#<ExceptionHandling::TestRaiseOnInspectWithId @id=123 [error 'RuntimeError: some error' while calling #inspect]>")
-        end
-
-        it "handle exceptions for objects responding to to_pk" do
-          result = HoneybadgerCallbacks.send(:local_variable_filter, :variable_name, TestRaiseOnInspectWithToPk.new, ['password'])
-          expect(result).to eq("#<ExceptionHandling::TestRaiseOnInspectWithToPk @pk=SomeRecord-123 [error 'RuntimeError: some error' while calling #inspect]>")
-        end
-      end
-
-      context "not inspect objects that contain filter keys" do
-        it "use to_pk if available, even if id is available" do
-          result = HoneybadgerCallbacks.send(:local_variable_filter, :variable_name, TestPoroWithFilteredAttributePkAndId.new, ['password'])
-          expect(result).to match(/#<ExceptionHandling::TestPoroWithFilteredAttributePkAndId @pk=TestPoroWithFilteredAttributePkAndId_1, \[FILTERED\]>/)
-        end
-
-        it "use id if to_pk is not available" do
-          result = HoneybadgerCallbacks.send(:local_variable_filter, :variable_name, TestPoroWithFilteredAttributeAndId.new, ['password'])
-          expect(result).to match(/#<ExceptionHandling::TestPoroWithFilteredAttributeAndId @id=1, \[FILTERED\]>/)
-        end
-
-        it "print the object name if no id or to_pk" do
-          result = HoneybadgerCallbacks.send(:local_variable_filter, :variable_name, TestPoroWithFilteredAttribute.new, ['password'])
-          expect(result).to match(/#<ExceptionHandling::TestPoroWithFilteredAttribute \[FILTERED\]>/)
-        end
-      end
-    end
-  end
-end