airbrake_handler 0.3.0 → 0.4.0

data/CHANGELOG.md

@@ -0,0 +1,12 @@
+## 0.4.0 (Feb 15, 2012)
+
+Features:
+
+  - Added support for notify_host option in Toadhopper (@zenops)
+  - Added changelog file
+
+## 0.3.0 (Jan 27, 2012)
+
+Features:
+
+  - Added ability to ignore specific exceptions (@mironov)

data/README.md

@@ -23,6 +23,11 @@ You can pass more options to AirbrakeHander initializer, i.e:
 AirbrakeHandler.new(:api_key => "your-airbrake-api-key", :framework_env => "production")
 ```
 
+Toadhopper options:
+
+* :api_key
+* :notify_host
+
 If you want to ignore specific exceptions, you can do this like that:
 
 ```ruby
@@ -38,6 +43,7 @@ airbrake_handler.ignore << {:class => "Errno::ECONNRESET", :message => /Connecti
 ## Contributors
 
 * [Anton Mironov](https://github.com/mironov)
+* [Benedikt Böhm](https://github.com/hollow)
 
 ## Copyright
 

data/airbrake_handler.gemspec

@@ -3,7 +3,7 @@ $:.push File.expand_path("../lib", __FILE__)
 
 Gem::Specification.new do |s|
   s.name        = "airbrake_handler"
-  s.version     = "0.3.0"
+  s.version     = "0.4.0"
   s.platform    = Gem::Platform::RUBY
   s.authors     = ["Adam Jacob", "Wojciech Wnętrzak"]
   s.email       = ["w.wnetrzak@gmail.com"]

data/lib/airbrake_handler.rb

@@ -20,13 +20,14 @@ require "chef/handler"
 require "toadhopper"
 
 class AirbrakeHandler < Chef::Handler
-  VERSION = "0.3.0"
+  VERSION = "0.4.0"
 
-  attr_accessor :options, :api_key, :ignore
+  attr_accessor :options, :api_key, :ignore, :notify_host
 
   def initialize(options={})
     @api_key = options.delete(:api_key)
-    @ignore  = options.delete(:ignore) || []
+    @notify_host = options.delete(:notify_host) || nil
+    @ignore = options.delete(:ignore) || []
     @options = options
   end
 
@@ -59,6 +60,6 @@ class AirbrakeHandler < Chef::Handler
 
   def client
     raise ArgumentError.new("You must specify Airbrake api key") unless api_key
-    Toadhopper.new(api_key)
+    Toadhopper.new(api_key, :notify_host => @notify_host)
   end
 end

data/test/airbrake_notifier_test.rb

@@ -5,18 +5,24 @@ describe AirbrakeHandler do
     Chef::Log.stubs(:error)
   end
 
-  it "should raise error when api_key is not specified" do
+  # initalization
+
+  it "raises error when api_key is not specified" do
     assert_raises ArgumentError do
       AirbrakeHandler.new.client
     end
   end
 
-  it "should set options" do
+  # options
+
+  it "sets options" do
     handler = AirbrakeHandler.new(:framework_env => "staging")
     assert_equal "staging", handler.options[:framework_env]
   end
 
-  it "should report exception using client" do
+  # exceptions
+
+  it "reports exception using client" do
     exception = Exception.new
     run_status = stub(:failed? => true, :exception => exception)
     client = mock
@@ -29,7 +35,7 @@ describe AirbrakeHandler do
     handler.report
   end
 
-  it "should not report ignored exception" do
+  it "does not report ignored exception" do
     run_status = stub(:failed? => true, :exception => Exception.new)
     client = mock
     handler = AirbrakeHandler.new(:api_key => "fake")
@@ -42,7 +48,7 @@ describe AirbrakeHandler do
     handler.report
   end
 
-  it "should not report ignored exception with specific message" do
+  it "does not report ignored exception with specific message" do
     run_status = stub(:failed? => true, :exception => Exception.new("error"))
     client = mock
     handler = AirbrakeHandler.new(:api_key => "fake")
@@ -55,7 +61,7 @@ describe AirbrakeHandler do
     handler.report
   end
 
-  it "should report exception if its message doesn't match any message of ignored exceptions" do
+  it "reports exception if its message doesn't match any message of ignored exceptions" do
     run_status = stub(:failed? => true, :exception => Exception.new("important error"))
     client = mock
     handler = AirbrakeHandler.new(:api_key => "fake")
@@ -69,14 +75,16 @@ describe AirbrakeHandler do
     handler.report
   end
 
-  it "should ignore exception by message regexp" do
+  it "ignores exception by message regexp" do
     handler = AirbrakeHandler.new(:api_key => "fake")
     handler.ignore << {:class => "Exception", :message => /catch me if you can/}
 
     assert handler.ignore_exception?(Exception.new("catch me if you can"))
   end
 
-  it "should return Airbrake params" do
+  # params
+
+  it "returns Airbrake params" do
     node = stub(:name => "node-name", :run_list => "cookbook::recipe")
     run_status = stub(:node => node, :start_time => Time.mktime(2011,1,1),
       :end_time => Time.mktime(2011,1,2), :elapsed_time => Time.mktime(2011,1,3))

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: airbrake_handler
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
   prerelease: 
 platform: ruby
 authors:
@@ -10,11 +10,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-01-27 00:00:00.000000000 Z
+date: 2012-02-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: chef
-  requirement: &72635450 !ruby/object:Gem::Requirement
+  requirement: &83704960 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -22,10 +22,10 @@ dependencies:
         version: 0.9.0
   type: :runtime
   prerelease: false
-  version_requirements: *72635450
+  version_requirements: *83704960
 - !ruby/object:Gem::Dependency
   name: toadhopper
-  requirement: &72634600 !ruby/object:Gem::Requirement
+  requirement: &83704720 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -33,10 +33,10 @@ dependencies:
         version: '2.0'
   type: :runtime
   prerelease: false
-  version_requirements: *72634600
+  version_requirements: *83704720
 - !ruby/object:Gem::Dependency
   name: mocha
-  requirement: &72634200 !ruby/object:Gem::Requirement
+  requirement: &83704530 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -44,7 +44,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *72634200
+  version_requirements: *83704530
 description: Chef handler for sending exceptions to Airbrake
 email:
 - w.wnetrzak@gmail.com
@@ -54,6 +54,7 @@ extra_rdoc_files: []
 files:
 - .gitignore
 - .travis.yml
+- CHANGELOG.md
 - Gemfile
 - LICENSE
 - README.md
@@ -82,7 +83,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: airbrake_handler
-rubygems_version: 1.8.15
+rubygems_version: 1.8.16
 signing_key: 
 specification_version: 3
 summary: Chef handler for sending exceptions to Airbrake