jordan-brough-hoptoad_notifier 2.3.0

data/test/notifier_test.rb

@@ -0,0 +1,222 @@
+require File.dirname(__FILE__) + '/helper'
+
+class NotifierTest < Test::Unit::TestCase
+
+  class OriginalException < Exception
+  end
+
+  class ContinuedException < Exception
+  end
+
+  include DefinesConstants
+
+  def setup
+    super
+    reset_config
+  end
+
+  def assert_sent(notice, notice_args)
+    assert_received(HoptoadNotifier::Notice, :new) {|expect| expect.with(has_entries(notice_args)) }
+    assert_received(notice, :to_xml)
+    assert_received(HoptoadNotifier.sender, :send_to_hoptoad) {|expect| expect.with(notice.to_xml) }
+  end
+
+  def set_public_env
+    HoptoadNotifier.configure { |config| config.environment_name = 'production' }
+  end
+
+  def set_development_env
+    HoptoadNotifier.configure { |config| config.environment_name = 'development' }
+  end
+
+  should "yield and save a configuration when configuring" do
+    yielded_configuration = nil
+    HoptoadNotifier.configure do |config|
+      yielded_configuration = config
+    end
+
+    assert_kind_of HoptoadNotifier::Configuration, yielded_configuration
+    assert_equal yielded_configuration, HoptoadNotifier.configuration
+  end
+
+  should "not remove existing config options when configuring twice" do
+    first_config = nil
+    HoptoadNotifier.configure do |config|
+      first_config = config
+    end
+    HoptoadNotifier.configure do |config|
+      assert_equal first_config, config
+    end
+  end
+
+  should "configure the sender" do
+    sender = stub_sender
+    HoptoadNotifier::Sender.stubs(:new => sender)
+    configuration = nil
+
+    HoptoadNotifier.configure { |yielded_config| configuration = yielded_config }
+
+    assert_received(HoptoadNotifier::Sender, :new) { |expect| expect.with(configuration) }
+    assert_equal sender, HoptoadNotifier.sender
+  end
+
+  should "create and send a notice for an exception" do
+    set_public_env
+    exception = build_exception
+    stub_sender!
+    notice = stub_notice!
+
+    HoptoadNotifier.notify(exception)
+
+    assert_sent notice, :exception => exception
+  end
+
+  should "create and send a notice for a hash" do
+    set_public_env
+    notice = stub_notice!
+    notice_args = { :error_message => 'uh oh' }
+    stub_sender!
+
+    HoptoadNotifier.notify(notice_args)
+
+    assert_sent(notice, notice_args)
+  end
+
+  should "create and sent a notice for an exception and hash" do
+    set_public_env
+    exception = build_exception
+    notice = stub_notice!
+    notice_args = { :error_message => 'uh oh' }
+    stub_sender!
+
+    HoptoadNotifier.notify(exception, notice_args)
+
+    assert_sent(notice, notice_args.merge(:exception => exception))
+  end
+
+  should "not create a notice in a development environment" do
+    set_development_env
+    sender = stub_sender!
+
+    HoptoadNotifier.notify(build_exception)
+    HoptoadNotifier.notify_or_ignore(build_exception)
+
+    assert_received(sender, :send_to_hoptoad) {|expect| expect.never }
+  end
+
+  should "not deliver an ignored exception when notifying implicitly" do
+    set_public_env
+    exception = build_exception
+    sender = stub_sender!
+    notice = stub_notice!
+    notice.stubs(:ignore? => true)
+
+    HoptoadNotifier.notify_or_ignore(exception)
+
+    assert_received(sender, :send_to_hoptoad) {|expect| expect.never }
+  end
+
+  should "deliver an ignored exception when notifying manually" do
+    set_public_env
+    exception = build_exception
+    sender = stub_sender!
+    notice = stub_notice!
+    notice.stubs(:ignore? => true)
+
+    HoptoadNotifier.notify(exception)
+
+    assert_sent(notice, :exception => exception)
+  end
+
+  should "pass config to created notices" do
+    exception = build_exception
+    config_opts = { 'one' => 'two', 'three' => 'four' }
+    notice = stub_notice!
+    stub_sender!
+    HoptoadNotifier.configuration = stub('config', :merge => config_opts, :public? => true)
+
+    HoptoadNotifier.notify(exception)
+
+    assert_received(HoptoadNotifier::Notice, :new) do |expect|
+      expect.with(has_entries(config_opts))
+    end
+  end
+
+  context "building notice JSON for an exception" do
+    setup do
+      @params    = { :controller => "users", :action => "create" }
+      @exception = build_exception
+      @hash      = HoptoadNotifier.build_lookup_hash_for(@exception, @params)
+    end
+
+    should "set action" do
+      assert_equal @params[:action], @hash[:action]
+    end
+
+    should "set controller" do
+      assert_equal @params[:controller], @hash[:component]
+    end
+
+    should "set line number" do
+      assert @hash[:line_number] =~ /\d+/
+    end
+
+    should "set file" do
+      assert_match /test\/helper\.rb$/, @hash[:file]
+    end
+
+    should "set rails_env to production" do
+      assert_equal 'production', @hash[:environment_name]
+    end
+
+    should "set error class" do
+      assert_equal 'RuntimeError', @hash[:error_class]
+    end
+
+    should "not set file or line number with no backtrace" do
+      @exception.stubs(:backtrace).returns([])
+
+      @hash = HoptoadNotifier.build_lookup_hash_for(@exception)
+
+      assert_nil @hash[:line_number]
+      assert_nil @hash[:file]
+    end
+
+    should "not set action or controller when not provided" do
+      @hash = HoptoadNotifier.build_lookup_hash_for(@exception)
+
+      assert_nil @hash[:action]
+      assert_nil @hash[:controller]
+    end
+
+    context "when an exception that provides #original_exception is raised" do
+      setup do
+        @exception.stubs(:original_exception).returns(begin
+          raise NotifierTest::OriginalException.new
+        rescue Exception => e
+          e
+        end)
+      end
+
+      should "unwrap exceptions that provide #original_exception" do
+        @hash = HoptoadNotifier.build_lookup_hash_for(@exception)
+        assert_equal "NotifierTest::OriginalException", @hash[:error_class]
+      end
+    end
+
+    context "when an exception that provides #continued_exception is raised" do
+      setup do
+        @exception.stubs(:continued_exception).returns(begin
+          raise NotifierTest::ContinuedException.new
+        rescue Exception => e
+          e
+        end)
+      end
+
+      should "unwrap exceptions that provide #continued_exception" do
+        @hash = HoptoadNotifier.build_lookup_hash_for(@exception)
+        assert_equal "NotifierTest::ContinuedException", @hash[:error_class]
+      end
+    end
+  end
+end

data/test/rack_test.rb

@@ -0,0 +1,58 @@
+require File.dirname(__FILE__) + '/helper'
+
+class RackTest < Test::Unit::TestCase
+
+  should "call the upstream app with the environment" do
+    environment = { 'key' => 'value' }
+    app = lambda { |env| ['response', {}, env] }
+    stack = HoptoadNotifier::Rack.new(app)
+
+    response = stack.call(environment)
+
+    assert_equal ['response', {}, environment], response
+  end
+
+  should "deliver an exception raised while calling an upstream app" do
+    HoptoadNotifier.stubs(:notify_or_ignore)
+
+    exception = build_exception
+    environment = { 'key' => 'value' }
+    app = lambda do |env|
+      raise exception
+    end
+
+    begin
+      stack = HoptoadNotifier::Rack.new(app)
+      stack.call(environment)
+    rescue Exception => raised
+      assert_equal exception, raised
+    else
+      flunk "Didn't raise an exception"
+    end
+
+    assert_received(HoptoadNotifier, :notify_or_ignore) do |expect|
+      expect.with(exception, :rack_env => environment)
+    end
+  end
+
+  should "deliver an exception in rack.exception" do
+    HoptoadNotifier.stubs(:notify_or_ignore)
+    exception = build_exception
+    environment = { 'key' => 'value' }
+
+    response = [200, {}, ['okay']]
+    app = lambda do |env|
+      env['rack.exception'] = exception
+      response
+    end
+    stack = HoptoadNotifier::Rack.new(app)
+
+    actual_response = stack.call(environment)
+
+    assert_equal response, actual_response
+    assert_received(HoptoadNotifier, :notify_or_ignore) do |expect|
+      expect.with(exception, :rack_env => environment)
+    end
+  end
+
+end

data/test/rails_initializer_test.rb

@@ -0,0 +1,36 @@
+require File.dirname(__FILE__) + '/helper'
+
+require 'hoptoad_notifier/rails'
+
+class RailsInitializerTest < Test::Unit::TestCase
+  include DefinesConstants
+
+  should "trigger use of Rails' logger if logger isn't set and Rails' logger exists" do
+    rails = Module.new do
+      def self.logger
+        "RAILS LOGGER"
+      end
+    end
+    define_constant("Rails", rails)
+    HoptoadNotifier::Rails.initialize
+    assert_equal "RAILS LOGGER", HoptoadNotifier.logger
+  end
+
+  should "trigger use of Rails' default logger if logger isn't set and Rails.logger doesn't exist" do
+    define_constant("RAILS_DEFAULT_LOGGER", "RAILS DEFAULT LOGGER")
+
+    HoptoadNotifier::Rails.initialize
+    assert_equal "RAILS DEFAULT LOGGER", HoptoadNotifier.logger
+  end
+
+  should "allow overriding of the logger if already assigned" do
+    define_constant("RAILS_DEFAULT_LOGGER", "RAILS DEFAULT LOGGER")
+    HoptoadNotifier::Rails.initialize
+
+    HoptoadNotifier.configure(true) do |config|
+      config.logger = "OVERRIDDEN LOGGER"
+    end
+
+    assert_equal "OVERRIDDEN LOGGER", HoptoadNotifier.logger
+  end
+end

data/test/sender_test.rb

@@ -0,0 +1,123 @@
+require File.dirname(__FILE__) + '/helper'
+
+class SenderTest < Test::Unit::TestCase
+
+  def setup
+    reset_config
+  end
+
+  def build_sender(opts = {})
+    config = HoptoadNotifier::Configuration.new
+    opts.each {|opt, value| config.send(:"#{opt}=", value) }
+    HoptoadNotifier::Sender.new(config)
+  end
+
+  def send_exception(args = {})
+    notice = args.delete(:notice) || build_notice_data
+    sender = args.delete(:sender) || build_sender(args)
+    sender.send_to_hoptoad(notice)
+    sender
+  end
+
+  def stub_http
+    response = stub(:body => 'body')
+    http = stub(:post          => response,
+                :read_timeout= => nil,
+                :open_timeout= => nil,
+                :use_ssl=      => nil)
+    Net::HTTP.stubs(:new => http)
+    http
+  end
+
+  should "post to Hoptoad when using an HTTP proxy" do
+    response = stub(:body => 'body')
+    http     = stub(:post          => response,
+                    :read_timeout= => nil,
+                    :open_timeout= => nil,
+                    :use_ssl=      => nil)
+    proxy    = stub(:new => http)
+    Net::HTTP.stubs(:Proxy => proxy)
+
+    url = "http://hoptoadapp.com:80#{HoptoadNotifier::Sender::NOTICES_URI}"
+    uri = URI.parse(url)
+
+    proxy_host = 'some.host'
+    proxy_port = 88
+    proxy_user = 'login'
+    proxy_pass = 'passwd'
+
+    send_exception(:proxy_host => proxy_host,
+                   :proxy_port => proxy_port,
+                   :proxy_user => proxy_user,
+                   :proxy_pass => proxy_pass)
+    assert_received(http, :post) do |expect| 
+      expect.with(uri.path, anything, HoptoadNotifier::HEADERS)
+    end
+    assert_received(Net::HTTP, :Proxy) do |expect|
+      expect.with(proxy_host, proxy_port, proxy_user, proxy_pass)
+    end
+  end
+
+  should "post to the right url for non-ssl" do
+    http = stub_http
+    url = "http://hoptoadapp.com:80#{HoptoadNotifier::Sender::NOTICES_URI}"
+    uri = URI.parse(url)
+    send_exception(:secure => false)
+    assert_received(http, :post) {|expect| expect.with(uri.path, anything, HoptoadNotifier::HEADERS) }
+  end
+
+  should "post to the right path for ssl" do
+    http = stub_http
+    send_exception(:secure => true)
+    assert_received(http, :post) {|expect| expect.with(HoptoadNotifier::Sender::NOTICES_URI, anything, HoptoadNotifier::HEADERS) }
+  end
+
+  should "default the open timeout to 2 seconds" do
+    http = stub_http
+    send_exception
+    assert_received(http, :open_timeout=) {|expect| expect.with(2) }
+  end
+
+  should "default the read timeout to 5 seconds" do
+    http = stub_http
+    send_exception
+    assert_received(http, :read_timeout=) {|expect| expect.with(5) }
+  end
+
+  should "allow override of the open timeout" do
+    http = stub_http
+    send_exception(:http_open_timeout => 4)
+    assert_received(http, :open_timeout=) {|expect| expect.with(4) }
+  end
+
+  should "allow override of the read timeout" do
+    http = stub_http
+    send_exception(:http_read_timeout => 10)
+    assert_received(http, :read_timeout=) {|expect| expect.with(10) }
+  end
+
+  should "connect to the right port for ssl" do
+    stub_http
+    send_exception(:secure => true)
+    assert_received(Net::HTTP, :new) {|expect| expect.with("hoptoadapp.com", 443) }
+  end
+
+  should "connect to the right port for non-ssl" do
+    stub_http
+    send_exception(:secure => false)
+    assert_received(Net::HTTP, :new) {|expect| expect.with("hoptoadapp.com", 80) }
+  end
+
+  should "use ssl if secure" do
+    stub_http
+    send_exception(:secure => true, :host => 'example.org')
+    assert_received(Net::HTTP, :new) {|expect| expect.with('example.org', 443) }
+  end
+
+  should "not use ssl if not secure" do
+    stub_http
+    send_exception(:secure => false, :host => 'example.org')
+    assert_received(Net::HTTP, :new) {|expect| expect.with('example.org', 80) }
+  end
+
+end

metadata

@@ -0,0 +1,205 @@
+--- !ruby/object:Gem::Specification 
+name: jordan-brough-hoptoad_notifier
+version: !ruby/object:Gem::Version 
+  hash: 3
+  prerelease: false
+  segments: 
+  - 2
+  - 3
+  - 0
+  version: 2.3.0
+platform: ruby
+authors: 
+- thoughtbot, inc
+- Jordan Brough
+autorequire: 
+bindir: bin
+cert_chain: []
+
+date: 2010-07-02 00:00:00 -06:00
+default_executable: 
+dependencies: 
+- !ruby/object:Gem::Dependency 
+  name: activesupport
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  type: :runtime
+  version_requirements: *id001
+- !ruby/object:Gem::Dependency 
+  name: activerecord
+  prerelease: false
+  requirement: &id002 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  type: :development
+  version_requirements: *id002
+- !ruby/object:Gem::Dependency 
+  name: actionpack
+  prerelease: false
+  requirement: &id003 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  type: :development
+  version_requirements: *id003
+- !ruby/object:Gem::Dependency 
+  name: jferris-mocha
+  prerelease: false
+  requirement: &id004 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  type: :development
+  version_requirements: *id004
+- !ruby/object:Gem::Dependency 
+  name: nokogiri
+  prerelease: false
+  requirement: &id005 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  type: :development
+  version_requirements: *id005
+- !ruby/object:Gem::Dependency 
+  name: shoulda
+  prerelease: false
+  requirement: &id006 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  type: :development
+  version_requirements: *id006
+description: 
+email: support@hoptoadapp.com
+executables: []
+
+extensions: []
+
+extra_rdoc_files: 
+- README.rdoc
+files: 
+- CHANGELOG
+- INSTALL
+- MIT-LICENSE
+- Rakefile
+- README.rdoc
+- SUPPORTED_RAILS_VERSIONS
+- TESTING.rdoc
+- generators/hoptoad/hoptoad_generator.rb
+- generators/hoptoad/lib/insert_commands.rb
+- generators/hoptoad/lib/rake_commands.rb
+- generators/hoptoad/templates/capistrano_hook.rb
+- generators/hoptoad/templates/hoptoad_notifier_tasks.rake
+- generators/hoptoad/templates/initializer.rb
+- lib/hoptoad_notifier/backtrace.rb
+- lib/hoptoad_notifier/capistrano.rb
+- lib/hoptoad_notifier/configuration.rb
+- lib/hoptoad_notifier/notice.rb
+- lib/hoptoad_notifier/rack.rb
+- lib/hoptoad_notifier/rails/action_controller_catcher.rb
+- lib/hoptoad_notifier/rails/controller_methods.rb
+- lib/hoptoad_notifier/rails/error_lookup.rb
+- lib/hoptoad_notifier/rails.rb
+- lib/hoptoad_notifier/rails3_tasks.rb
+- lib/hoptoad_notifier/railtie.rb
+- lib/hoptoad_notifier/sender.rb
+- lib/hoptoad_notifier/tasks.rb
+- lib/hoptoad_notifier/version.rb
+- lib/hoptoad_notifier.rb
+- lib/hoptoad_tasks.rb
+- lib/rails/generators/hoptoad/hoptoad_generator.rb
+- test/backtrace_test.rb
+- test/catcher_test.rb
+- test/configuration_test.rb
+- test/helper.rb
+- test/hoptoad_tasks_test.rb
+- test/logger_test.rb
+- test/notice_test.rb
+- test/notifier_test.rb
+- test/rack_test.rb
+- test/rails_initializer_test.rb
+- test/sender_test.rb
+- rails/init.rb
+- script/integration_test.rb
+- lib/templates/rescue.erb
+has_rdoc: true
+homepage: http://www.hoptoadapp.com
+licenses: []
+
+post_install_message: 
+rdoc_options: 
+- --line-numbers
+- --main
+- README.rdoc
+require_paths: 
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement 
+  none: false
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      hash: 3
+      segments: 
+      - 0
+      version: "0"
+required_rubygems_version: !ruby/object:Gem::Requirement 
+  none: false
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      hash: 3
+      segments: 
+      - 0
+      version: "0"
+requirements: []
+
+rubyforge_project: 
+rubygems_version: 1.3.7
+signing_key: 
+specification_version: 3
+summary: Send your application errors to our hosted service and reclaim your inbox.
+test_files: 
+- test/backtrace_test.rb
+- test/catcher_test.rb
+- test/configuration_test.rb
+- test/hoptoad_tasks_test.rb
+- test/logger_test.rb
+- test/notice_test.rb
+- test/notifier_test.rb
+- test/rack_test.rb
+- test/rails_initializer_test.rb
+- test/sender_test.rb