merb_hoptoad_notifier 1.0.12 → 1.1.0

data/Rakefile

@@ -1,4 +1,3 @@
-require 'rubygems'
 require 'rake/gempackagetask'
 require 'rubygems/specification'
 require 'date'
@@ -6,10 +5,12 @@ require 'merb-core/version'
 require 'spec/rake/spectask'
 require 'bundler'
 
-install_home = ENV['GEM_HOME'] ? "-i #{ENV['GEM_HOME']}" : ""
+Bundler.require_env
+
+require 'lib/merb_hoptoad_notifier'
 
 NAME = "merb_hoptoad_notifier"
-GEM_VERSION = "1.0.12"
+GEM_VERSION = Merb::HoptoadNotifier::VERSION
 AUTHOR = "Corey Donohoe"
 EMAIL = 'atmos@atmos.org'
 HOMEPAGE = "http://github.com/atmos/merb_hoptoad_notifier"
@@ -28,14 +29,14 @@ spec = Gem::Specification.new do |s|
   s.email = EMAIL
   s.homepage = HOMEPAGE
 
-  manifest = Bundler::Environment.load(File.dirname(__FILE__) + '/Gemfile')
+  manifest = Bundler::Dsl.load_gemfile(File.dirname(__FILE__) + '/Gemfile')
   manifest.dependencies.each do |d|
     next unless d.only && d.only.include?('release')
     s.add_dependency(d.name, d.version)
   end
 
   s.require_path = 'lib'
-  s.files = %w(LICENSE README Rakefile TODO) + Dir.glob("{lib,spec}/**/*")
+  s.files = %w(LICENSE README Rakefile TODO) + Dir.glob("{lib}/**/*")
 end
 
 Rake::GemPackageTask.new(spec) do |pkg|
@@ -50,11 +51,11 @@ task :make_spec do
 end
 
 Spec::Rake::SpecTask.new(:default) do |t|
-  t.spec_opts << %w(-fs --color) << %w(-O spec/spec.opts)
+  t.spec_opts << %w(-fs --color)
   t.spec_opts << '--loadby' << 'random'
   t.spec_files = Dir["spec/*_spec.rb"]
   t.rcov = ENV.has_key?('NO_RCOV') ? ENV['NO_RCOV'] != 'true' : true
-  t.rcov_opts << '--exclude' << '.gem/'
+  t.rcov_opts << '--exclude' << 'vendor'
 
   t.rcov_opts << '--text-summary'
   t.rcov_opts << '--sort' << 'coverage' << '--sort-reverse'

data/lib/merb_hoptoad_notifier.rb

@@ -1,9 +1,9 @@
-if defined?(Merb::Plugins)
-  libdir = File.join(File.dirname(__FILE__), 'merb_hoptoad_notifier')
-  require 'toadhopper'
-  require File.join(libdir, 'hoptoad_notifier')
+require 'toadhopper'
+libdir = File.join(File.dirname(__FILE__), 'merb_hoptoad_notifier')
+require File.join(libdir, 'hoptoad_notifier')
 
+if defined?(Merb::BootLoader)
   Merb::BootLoader.after_app_loads do
-    HoptoadNotifier.configure
+    Merb::HoptoadNotifier.configure
   end
 end

data/lib/merb_hoptoad_notifier/hoptoad_notifier.rb

@@ -1,42 +1,52 @@
-module HoptoadNotifier
-  class << self
-    attr_accessor :api_key, :logger
-  end
+module Merb
+  module HoptoadNotifier
+    VERSION = '1.1.0'
+    class << self
+      attr_accessor :api_key, :logger, :environment_filters
+    end
 
-  def self.configure
-    key = YAML.load_file(Merb.root / 'config' / 'hoptoad.yml')
+    def self.configure
+      key = YAML.load_file(Merb.root / 'config' / 'hoptoad.yml')
 
-    if key
-      env = key[Merb.env.to_sym]
-      env ? @api_key = env[:api_key] : raise(ArgumentError, "No hoptoad key for Merb environment #{Merb.env}")
+      if key
+        env = key[Merb.env.to_sym]
+        env ? @api_key = env[:api_key] : raise(ArgumentError, "No hoptoad key for Merb environment #{Merb.env}")
+      end
     end
-  end
 
-  def self.logger
-    @logger || Merb.logger
-  end
+    def self.environment_filters
+      @environment_filters ||= [ ]
+    end
 
-  def self.notify_hoptoad(request, session)
-    request.exceptions.each do |exception|
-      controller = request.params['controller']
+    def self.environment_filters=(filters)
+      @environment_filters << filters
+    end
 
-      options = {
-        :api_key          => HoptoadNotifier.api_key,
-        :url              => "#{request.protocol}://#{request.host}#{request.path}",
-        :component        => controller && controller.capitalize,
-        :action           => request.params['action'],
-        :request          => request,
-        :framework_env    => Merb.env,
-        :notifier_name    => 'Merb::HoptoadNotifier',
-        :notifier_version => '1.0.10',
-        :session          => session.to_hash
-      }
-      dispatcher.post!(exception, options, {'X-Hoptoad-Client-Name' => 'Merb::HoptoadNotifier'})
+    def self.logger
+      @logger || Merb.logger
     end
-    true
-  end
 
-  def self.dispatcher
-    @dispatcher ||= ToadHopper.new(api_key)
+    def self.notify_hoptoad(request, session)
+      request.exceptions.each do |exception|
+        options = {
+          :api_key          => HoptoadNotifier.api_key,
+          :url              => "#{request.protocol}://#{request.host}#{request.path}",
+          :component        => request.params['controller'],
+          :action           => request.params['action'],
+          :request          => request,
+          :framework_env    => Merb.env,
+          :notifier_name    => 'Merb::HoptoadNotifier',
+          :notifier_version => Merb::HoptoadNotifier::VERSION,
+          :session          => session.to_hash
+        }
+        dispatcher.filters = environment_filters.flatten
+        dispatcher.post!(exception, options, {'X-Hoptoad-Client-Name' => 'Merb::HoptoadNotifier'})
+      end
+      true
+    end
+
+    def self.dispatcher
+      @dispatcher ||= ToadHopper.new(api_key)
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: merb_hoptoad_notifier
 version: !ruby/object:Gem::Version 
-  version: 1.0.12
+  version: 1.1.0
 platform: ruby
 authors: 
 - Corey Donohoe
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-12-16 00:00:00 -08:00
+date: 2009-12-25 00:00:00 -08:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -30,7 +30,7 @@ dependencies:
     requirements: 
     - - ~>
       - !ruby/object:Gem::Version 
-        version: 0.9.2
+        version: 0.9.1
     version: 
 description: Merb plugin that provides hoptoad exception notification
 email: atmos@atmos.org
@@ -49,10 +49,6 @@ files:
 - TODO
 - lib/merb_hoptoad_notifier/hoptoad_notifier.rb
 - lib/merb_hoptoad_notifier.rb
-- spec/fixtures/hoptoad.yml
-- spec/merb_hoptoad_notifier_spec.rb
-- spec/spec.opts
-- spec/spec_helper.rb
 has_rdoc: true
 homepage: http://github.com/atmos/merb_hoptoad_notifier
 licenses: []

data/spec/fixtures/hoptoad.yml

@@ -1,9 +0,0 @@
----
-:development: &defaults
-  :api_key: ZOMGLOLROFLMAO
-
-:test:
-  <<: *defaults
-
-:production:
-  :api_key: UBERSECRETSHIT

data/spec/merb_hoptoad_notifier_spec.rb

@@ -1,15 +0,0 @@
-require File.dirname(__FILE__) + '/spec_helper'
-
-describe "HoptoadNotifier" do
-  before(:each) do
-    config = { :development => { :api_key=> ENV['MY_HOPTOAD_API_KEY'] || 'blah' } }
-    mock(YAML).load_file(File.join(Merb.root / 'config' / 'hoptoad.yml')) { config }
-
-    HoptoadNotifier.configure
-  end
-  describe "notification" do
-    it "posts to hoptoad" do
-      HoptoadNotifier.notify_hoptoad(fake_request_with_exceptions, { :user_id => 42 })
-    end
-  end
-end

data/spec/spec_helper.rb

@@ -1,26 +0,0 @@
-Bundler.require_env(:test)
-$:.push File.join(File.dirname(__FILE__), '..', 'lib')
-require 'rr'
-require 'merb-core'
-require 'merb_hoptoad_notifier'
-require 'tmpdir'
-require 'pp'
-
-class TestError < RuntimeError
-end
-
-Spec::Runner.configure do |config|
-  config.mock_with :rr
-  def fake_request
-    Merb::Test::RequestHelper::FakeRequest.new
-  end
-  def fake_request_with_exceptions(params = { :controller => 'Application', :action => 'index' })
-    request = Merb::Test::RequestHelper::FakeRequest.new(:params => params)
-    begin
-      raise(TestError, 'I like turtles')
-    rescue => e
-      request.exceptions = [ e ]
-    end
-    request
-  end
-end