errdo 0.9.1 → 0.10.0

data/test/integrations/plugins_integration_test.rb

@@ -1,12 +1,12 @@
 require 'test_helper'
 
-FAILSAFE_FAIL_STRING = "If you are the administrator of this website, then please read this web application's log file and/or the web server's log file to find out what went wrong."
+FAILSAFE_FAIL_STRING = "If you are the administrator of this website, then please read this web application's \
+                        log file and/or the web server's log file to find out what went wrong.".freeze
 
 class PluginsIntegrationTest < ActionDispatch::IntegrationTest
 
   setup do
-    Errdo.error_name = "errors"
-    Errdo.slack_webhook = "https://slack.com/test"
+    Errdo.notify_with slack: { webhook: "https://slack.com/test" }
   end
 
   context "slack integration" do
@@ -16,8 +16,8 @@ class PluginsIntegrationTest < ActionDispatch::IntegrationTest
       assert_requested :any, /.*slack.*/
     end
 
-    should "not send a slack notification when error is hit if webhook is nil" do
-      Errdo.slack_webhook = nil
+    should "not send a slack notification when error is hit if webhook is not set" do
+      Errdo.instance_variable_set(:@notifiers, [])
       stub_request :any, /.*slack.*/
       get static_generic_error_path
       assert_not_requested :any, /.*slack.*/
@@ -28,6 +28,7 @@ class PluginsIntegrationTest < ActionDispatch::IntegrationTest
       stub_request :any, /.*slack.*/
       get static_generic_error_path
       assert_requested :any, /.*slack.*/
+      Errdo.error_name = :errors
     end
 
     should "not fail when the slack ping returns an error" do
@@ -38,8 +39,7 @@ class PluginsIntegrationTest < ActionDispatch::IntegrationTest
   end
 
   teardown do
-    Errdo.instance_variable_set(:@slack_notifier, nil)
-    Errdo.slack_webhook = nil
+    Errdo.instance_variable_set(:@notifiers, [])
   end
 
   private

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: errdo
 version: !ruby/object:Gem::Version
-  version: 0.9.1
+  version: 0.10.0
 platform: ruby
 authors:
 - erichaydel
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-09-28 00:00:00.000000000 Z
+date: 2016-09-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -164,8 +164,9 @@ files:
 - lib/errdo/extensions/cancancan/authorization_adapter.rb
 - lib/errdo/helpers/views_helper.rb
 - lib/errdo/models/error_env_parser.rb
-- lib/errdo/models/slack_messager.rb
 - lib/errdo/models/user_parser.rb
+- lib/errdo/notifications/slack.rb
+- lib/errdo/notifications/slack/notification_adapter.rb
 - lib/errdo/version.rb
 - lib/generators/active_record/errdo_generator.rb
 - lib/generators/active_record/templates/migration.rb
@@ -979,7 +980,6 @@ files:
 - test/dummy/tmp/cache/assets/sprockets/v3.0/zt/zt98H6qoLRpSDbtbVQtghzuYtOz1-Po0QcDAVWK7XC4.cache
 - test/dummy/tmp/cache/assets/sprockets/v3.0/zv/zvfByn9a2SXSz6mb_B5wkAIJMltXb2gON_PpjV8Gyl0.cache
 - test/dummy/tmp/cache/assets/sprockets/v3.0/zw/zwqTCzApUNMz710-JBgR6nos_fF_00TLDbBMbRhMAyw.cache
-- test/dummy/tmp/pids/server.pid
 - test/factories.rb
 - test/fixtures/users.yml
 - test/generators/active_record_generator_test.rb
@@ -1808,7 +1808,6 @@ test_files:
 - test/dummy/tmp/cache/assets/sprockets/v3.0/D_/D_KcfHEBsRaZDGqaiFtERXr1g8Av54-9T958jUK7R0A.cache
 - test/dummy/tmp/cache/assets/sprockets/v3.0/ED/EDVZLiJoq6wj93pJxKsK4TI15LIXRqwx4OdFGAAr068.cache
 - test/dummy/tmp/cache/assets/sprockets/v3.0/Ip/IplSosjlbQ5aan_h10fBG52WhsI8ogweDeukCOuwPWI.cache
-- test/dummy/tmp/pids/server.pid
 - test/dummy/README.rdoc
 - test/dummy/bin/bundle
 - test/dummy/bin/setup

data/lib/errdo/models/slack_messager.rb

@@ -1,37 +0,0 @@
-module Errdo
-  module Models
-    class SlackMessager
-
-      include Errdo::Helpers::ViewsHelper # For the naming of the user in the message
-
-      def initialize(error, parser)
-        if error.nil?
-          @user = parser.user
-          @backtrace = parser.short_backtrace
-          @exception_name = parser.exception_name
-          @exception_message = parser.exception_message
-        else
-          @user = error.last_experiencer
-          @backtrace = error.short_backtrace
-          @exception_name = error.exception_class_name
-          @exception_message = error.exception_message
-        end
-      end
-
-      def message
-        "#{exception_string}#{user_message_addon}\n#{@backtrace}"
-      end
-
-      private
-
-      def exception_string
-        "#{@exception_name} | #{@exception_message}"
-      end
-
-      def user_message_addon
-        "\nExperienced by #{user_show_string(@user)} " if @user
-      end
-
-    end
-  end
-end

data/test/dummy/tmp/pids/server.pid

@@ -1 +0,0 @@
-4358