Small tweak to readme

README.md

@@ -1,14 +1,20 @@
-# Airbrake-Statsd
+# Airbrake-Statsd [![Build Status](https://secure.travis-ci.org/jimeh/airbrake-statsd.png)](http://travis-ci.org/jimeh/airbrake-statsd)
 
-[![Build Status](https://secure.travis-ci.org/jimeh/airbrake-statsd.png)](http://travis-ci.org/jimeh/airbrake-statsd)
+Extends the [Airbrake][] gem to also report exceptions to Esty's [StatsD][]
-
-Extends the [Airbrake][] gem to also report exceptions Esty's [Statsd][]
 statistics aggregator.
 
 ## Installation
 
+Command line:
+
     gem install airbrake-statsd
 
+Gemfile:
+
+```ruby
+gem 'airbrake-statsd'
+```
+
 ## Usage
 
 It's assumed you know what both Airbrake and StatsD are before attempting to

lib/airbrake-statsd/airbrake_ext.rb

@@ -1,21 +1,15 @@
 module Airbrake
   class << self
 
-    def notify_with_statds(*args)
+    private
+
+    def send_notice_with_statsd(*args)
       Airbrake::Statsd.increment
-      notify_without_statsd(*args)
+      send_notice_without_statsd(*args)
     end
 
-    alias :notify_without_statsd :notify
+    alias :send_notice_without_statsd :send_notice
-    alias :notify :notify_with_statds
+    alias :send_notice :send_notice_with_statsd
-
-    def notify_or_ignore_with_statsd(*args)
-      Airbrake::Statsd.increment
-      notify_or_ignore_without_statsd(*args)
-    end
-
-    alias :notify_or_ignore_without_statsd :notify_or_ignore
-    alias :notify_or_ignore :notify_or_ignore_with_statsd
 
   end
 end

lib/airbrake-statsd/version.rb

@@ -1,5 +1,5 @@
 module Airbrake
   module Statsd
-    VERSION = '0.2.0'
+    VERSION = '0.2.1'
   end
 end

spec/unit/airbrake-statsd/airbrake_ext_spec.rb

@@ -4,58 +4,33 @@ describe Airbrake do
 
   subject { Airbrake }
 
-  it '#notify is an alias to #notify_with_statsd' do
+  let(:notice) { mock('Notice', :to_xml => '') }
-    subject.method(:notify).should == subject.method(:notify_with_statds)
+  let(:configuration) { mock('Configuration', :public? => true) }
+
+  let(:sender) do
+    sender = mock('Sender')
+    sender.stub(:send_to_airbrake)
+    sender
   end
 
-  it '#notify_or_ignore is an alias to #notify_or_ignore_with_statsd' do
+  describe '#send_notice_without_statsd' do
-    subject.method(:notify_or_ignore).
+    before do
-      should == subject.method(:notify_or_ignore_with_statsd)
+      Airbrake.stub(:configuration).and_return(configuration)
-  end
+      Airbrake.stub(:sender).and_return(sender)
+    end
 
-  describe '#notify_without_statsd' do
     it 'does not call Airbrake::Statsd.increment' do
-      Airbrake.stub(:send_notice).and_return(nil)
-
-      Airbrake.should_receive(:build_notice_for).with('oops', {})
       Airbrake::Statsd.should_not_receive(:increment)
-
+      Airbrake.send(:send_notice_without_statsd, notice)
-      Airbrake.notify_without_statsd('oops', {})
     end
   end
 
-  describe '#notify' do
+  describe '#send_notice' do
     it 'calls Airbrake::Statsd.increment' do
-      Airbrake.stub(:send_notice).and_return(nil)
-
-      Airbrake.should_receive(:build_notice_for).with('oops', {})
       Airbrake::Statsd.should_receive(:increment)
+      Airbrake.should_receive(:send_notice_without_statsd)
 
-      Airbrake.notify('oops', {})
+      Airbrake.send(:send_notice, notice)
-    end
-  end
-
-  describe '#notify_or_ignore_without_statsd' do
-    it 'does not call Airbrake::Statsd.increment' do
-      Airbrake.stub(:send_notice).and_return(nil)
-
-      Airbrake.should_receive(:build_notice_for).with('oops', {}).
-        and_return(mock('Notice', :ignore? => false))
-      Airbrake::Statsd.should_not_receive(:increment)
-
-      Airbrake.notify_or_ignore_without_statsd('oops', {})
-    end
-  end
-
-  describe '#notify_or_ignore' do
-    it 'calls Airbrake::Statsd.increment' do
-      Airbrake.stub(:send_notice).and_return(nil)
-
-      Airbrake.should_receive(:build_notice_for).with('oops', {}).
-        and_return(mock('Notice', :ignore? => false))
-      Airbrake::Statsd.should_receive(:increment)
-
-      Airbrake.notify_or_ignore('oops', {})
     end
   end