errplane 0.0.2 → 0.0.3

data/lib/errplane.rb

@@ -19,8 +19,6 @@ require "errplane/rack"
 require "errplane/railtie" #if defined?(Rails)
 
 module Errplane
-  API_HOST = "api.errplane.com"
-
   class << self
     attr_writer :configuration
     attr_accessor :transmitter
@@ -39,7 +37,7 @@ module Errplane
     end
 
     def ignorable_exception?(exception)
-      return configuration.ignored_exceptions.include?(exception.class.to_s)
+      configuration.ignore_current_environment? || configuration.ignored_exceptions.include?(exception.class.to_s)
     end
 
     private

data/lib/errplane/configuration.rb

@@ -1,19 +1,29 @@
 module Errplane
   class Configuration
     attr_accessor :api_key
+    attr_accessor :api_host
     attr_accessor :application_id
 
     attr_accessor :logger
-    attr_accessor :environment_name
-    attr_accessor :project_root
+    attr_accessor :rails_environment
+    attr_accessor :rails_root
     attr_accessor :framework
     attr_accessor :ignored_exceptions
+    attr_accessor :ignored_environments
 
+    DEFAULT_API_HOST = "api.errplane.com"
     DEFAULT_IGNORED_EXCEPTIONS = %w{ActiveRecord::RecordNotFound
                                     ActionController::RoutingError}
+    DEFAULT_IGNORED_ENVIRONMENTS = %w{development test cucumber selenium}
 
     def initialize
+      @api_host = DEFAULT_API_HOST
       @ignored_exceptions = DEFAULT_IGNORED_EXCEPTIONS.dup
+      @ignored_environments = DEFAULT_IGNORED_ENVIRONMENTS.dup
+    end
+
+    def ignore_current_environment?
+      return self.ignored_environments.include?(self.rails_environment)
     end
   end
 end

data/lib/errplane/railtie.rb

@@ -12,10 +12,10 @@ module Errplane
 
     config.after_initialize do
       Errplane.configure(true) do |config|
-        config.logger           ||= ::Rails.logger
-        config.environment_name ||= ::Rails.env
-        config.project_root     ||= ::Rails.root
-        config.framework        = "Rails: #{::Rails::VERSION::STRING}"
+        config.logger            ||= ::Rails.logger
+        config.rails_environment ||= ::Rails.env
+        config.rails_root        ||= ::Rails.root
+        config.framework           = "Rails #{::Rails::VERSION::STRING}"
       end
 
       if defined?(::ActionDispatch::DebugExceptions)

data/lib/errplane/transmitter.rb

@@ -7,9 +7,9 @@ module Errplane
       http = initialize_http_connection
       data = black_box.to_json
       response = begin
-                   url = "/api/v1/applications/#{Errplane.configuration.application_id}/exceptions/#{Errplane.configuration.environment_name}?api_key=#{Errplane.configuration.api_key}"
+                   url = "/api/v1/applications/#{Errplane.configuration.application_id}/exceptions/#{Errplane.configuration.rails_environment}?api_key=#{Errplane.configuration.api_key}"
                    ::Rails.logger.info(url)
-                   http.post("/api/v1/applications/#{Errplane.configuration.application_id}/exceptions/#{Errplane.configuration.environment_name}?api_key=#{Errplane.configuration.api_key}", data)
+                   http.post("/api/v1/applications/#{Errplane.configuration.application_id}/exceptions/#{Errplane.configuration.rails_environment}?api_key=#{Errplane.configuration.api_key}", data)
                  rescue Exception => e
                    e
                  end
@@ -27,7 +27,7 @@ module Errplane
 
     private
     def initialize_http_connection
-      connection = Net::HTTP.new(API_HOST, "80")
+      connection = Net::HTTP.new(Errplane.configuration.api_host, "80")
     end
   end
 end

data/lib/errplane/version.rb

@@ -1,3 +1,3 @@
 module Errplane
-  VERSION = "0.0.2"
+  VERSION = "0.0.3"
 end

data/spec/integration/exceptions_spec.rb

@@ -1,13 +1,19 @@
 require File.dirname(__FILE__) + "/integration_helper"
 
 feature "exception handling" do
+  before do
+    Errplane.configure do |config|
+      config.ignored_environments = %w{development}
+    end
+  end
+
   describe "in an action that raises an exception" do
     scenario "should make an HTTP call to the API" do
-      stub_request(:post, "#{Errplane::API_HOST}/exceptions").to_return(status: 200)
+      stub_request(:post, "#{Errplane.configuration.api_host}/exceptions").to_return(status: 200)
 
       lambda { visit new_widget_path }.should raise_error
 
-      assert_requested :post, "#{Errplane::API_HOST}/api/v1/applications/#{Errplane.configuration.application_id}/exceptions/test?api_key=f123-e456-d789c012"
+      assert_requested :post, "#{Errplane.configuration.api_host}/api/v1/applications/#{Errplane.configuration.application_id}/exceptions/test?api_key=f123-e456-d789c012"
     end
   end
 
@@ -15,7 +21,7 @@ feature "exception handling" do
     scenario "should not make an HTTP call to the API" do
       lambda { visit widgets_path }.should_not raise_error
 
-      assert_not_requested :post, "#{Errplane::API_HOST}/api/v1/applications/#{Errplane.configuration.application_id}/exceptions/test?api_key=f123-e456-d789c012"
+      assert_not_requested :post, "#{Errplane.configuration.api_host}/api/v1/applications/#{Errplane.configuration.application_id}/exceptions/test?api_key=f123-e456-d789c012"
     end
   end
 end

data/spec/unit/errplane_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+
+describe Errplane do
+  describe ".ignorable_exception?" do
+    it "should be true for exception types specified in the configuration" do
+      class DummyException < Exception; end
+      exception = DummyException.new
+
+      Errplane.configure do |config|
+        config.ignored_exceptions << 'DummyException'
+      end
+
+      Errplane.ignorable_exception?(exception).should be_true
+    end
+
+    it "should be true for exception types specified in the configuration" do
+      exception = ActionController::RoutingError.new("foo")
+      Errplane.ignorable_exception?(exception).should be_true
+    end
+
+    it "should be false for valid exceptions" do
+      exception = ZeroDivisionError.new
+      Errplane.ignorable_exception?(exception).should be_false
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: errplane
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
   prerelease: 
 platform: ruby
 authors:
@@ -49,6 +49,7 @@ files:
 - spec/internal/public/favicon.ico
 - spec/spec_helper.rb
 - spec/unit/black_box_spec.rb
+- spec/unit/errplane_spec.rb
 homepage: http://errplane.com
 licenses: []
 post_install_message: 
@@ -88,3 +89,4 @@ test_files:
 - spec/internal/public/favicon.ico
 - spec/spec_helper.rb
 - spec/unit/black_box_spec.rb
+- spec/unit/errplane_spec.rb