radar 0.3.0 → 0.4.0

data/test/radar/backtrace_test.rb

@@ -0,0 +1,42 @@
+require 'test_helper'
+
+class BacktraceTest < Test::Unit::TestCase
+  context "backtrace class" do
+    setup do
+      @klass = Radar::Backtrace
+      @instance = @klass.new(["test.rb:14:in `bar'", "test.rb:10:in `foo'", "test.rb:18:in `<main>'"])
+    end
+
+    should "be an array" do
+      assert @instance.is_a?(Array)
+    end
+
+    should "properly parse a backtrace" do
+      assert_equal 3, @instance.length
+      assert @instance.all? { |x| x.is_a?(Radar::Backtrace::Entry) }
+    end
+
+    # This happens if an exception is created, not raised.
+    should "be empty if backtrace is nil" do
+      instance = @klass.new(Exception.new("test").backtrace)
+      assert instance.empty?
+    end
+  end
+
+  context "backtrace entry class" do
+    setup do
+      @klass = Radar::Backtrace::Entry
+      @instance = @klass.new("test.rb:14:in `bar'")
+    end
+
+    should "properly parse out the file, line, and method" do
+      assert_equal "test.rb", @instance.file
+      assert_equal "14", @instance.line
+      assert_equal "bar", @instance.method
+    end
+
+    should "turn into json nicely" do
+      assert @instance.to_json
+    end
+  end
+end

data/test/radar/config_test.rb

@@ -9,7 +9,9 @@ class ConfigTest < Test::Unit::TestCase
 
     context "reporters" do
       setup do
-        @reporter_klass = Class.new
+        @reporter_klass = Class.new do
+          def report(event); end
+        end
       end
 
       teardown do
@@ -23,13 +25,16 @@ class ConfigTest < Test::Unit::TestCase
       should "be able to add reporters" do
         @instance.reporters.use @reporter_klass
         assert !@instance.reporters.empty?
-        assert @instance.reporters.values.first.is_a?(@reporter_klass)
+      end
+
+      should "be able to add reporters using the shortcut singular method" do
+        @instance.reporter @reporter_klass
+        assert !@instance.reporters.empty?
       end
 
       should "be able to add reporters via built-in symbols" do
         @instance.reporters.use :file
         assert !@instance.reporters.empty?
-        assert @instance.reporters.values.first.is_a?(Radar::Reporter::FileReporter)
       end
 
       should "yield the reporter instance if a block is given" do
@@ -61,6 +66,11 @@ class ConfigTest < Test::Unit::TestCase
         assert !@instance.data_extensions.empty?
       end
 
+      should "be able to add data extensions via the shortcut singular method" do
+        @instance.data_extension @extension
+        assert !@instance.data_extensions.empty?
+      end
+
       should "be able to add built-in extensions via symbols" do
         @instance.data_extensions.use :rack
         assert_equal Radar::DataExtensions::Rack, @instance.data_extensions.values.last
@@ -89,7 +99,33 @@ class ConfigTest < Test::Unit::TestCase
 
       should "be able to use built-in matchers as symbols" do
         @instance.match :class, Object
-        assert @instance.matchers.values.first.is_a?(Radar::Matchers::ClassMatcher)
+        assert @instance.matchers.values.first
+      end
+
+      should "key the matchers by name if given" do
+        @instance.match :class, Object
+        assert @instance.matchers.index(:class)
+      end
+    end
+
+    context "rejecters" do
+      setup do
+        @rejecter = Class.new do
+          def matches?(event); end
+        end
+      end
+
+      teardown do
+        @instance.rejecters.clear
+      end
+
+      should "initially have no rejecters" do
+        assert @instance.rejecters.empty?
+      end
+
+      should "be able to add rejecters" do
+        @instance.reject @rejecter
+        assert !@instance.rejecters.empty?
       end
     end
 
@@ -98,6 +134,10 @@ class ConfigTest < Test::Unit::TestCase
         @instance.filters.clear
       end
 
+      should "not have any filters by default" do
+        assert @instance.filters.empty?
+      end
+
       should "raise an ArgumentError if no class or lambda is given" do
         assert_raises(ArgumentError) { @instance.filters.use }
       end
@@ -109,6 +149,11 @@ class ConfigTest < Test::Unit::TestCase
         }
         assert_equal 1, @instance.filters.length
       end
+
+      should "be able to add filters using the shortcut singular method" do
+        @instance.filter {}
+        assert_equal 1, @instance.filters.length
+      end
     end
   end
 

data/test/radar/data_extensions/rails2_test.rb

@@ -0,0 +1,14 @@
+require 'test_helper'
+
+class Rails2DataTest < Test::Unit::TestCase
+  context "rails 2 data extension class" do
+    setup do
+      @klass = Radar::DataExtensions::Rails2
+      @instance = @klass.new(create_exception_event)
+    end
+
+    should "be able to convert to a hash" do
+      assert @instance.respond_to?(:to_hash)
+    end
+  end
+end

data/test/radar/exception_event_test.rb

@@ -16,6 +16,11 @@ class ExceptionEventTest < Test::Unit::TestCase
       assert @instance.occurred_at.is_a?(Time)
     end
 
+    should "include a backtrace from the exception" do
+      assert @instance.backtrace
+      assert_equal @instance.exception.backtrace, @instance.backtrace.original
+    end
+
     should "not have extra data by default" do
       assert @instance.extra.empty?
     end
@@ -50,6 +55,10 @@ class ExceptionEventTest < Test::Unit::TestCase
           @extension.any_instance.stubs(:to_hash).returns(nil)
           assert_nothing_raised { @instance.to_hash }
         end
+
+        should "cache the generated hash" do
+          assert @instance.to_hash.equal?(@instance.to_hash)
+        end
       end
 
       context "filters" do

data/test/radar/integration/rack_test.rb

@@ -6,7 +6,7 @@ class RackIntegrationTest < Test::Unit::TestCase
       @klass = Radar::Integration::Rack
     end
 
-    should "not allow integration via the actual integration test" do
+    should "not allow integration via the actual integration" do
       assert_raises(RuntimeError) {
         @klass.integrate!(nil)
       }

data/test/radar/integration/sinatra_test.rb

@@ -0,0 +1,13 @@
+require 'test_helper'
+
+class SinatraIntegrationTest < Test::Unit::TestCase
+  context "sinatra integration class" do
+    setup do
+      @klass = Radar::Integration::Sinatra
+    end
+
+    should "not allow integration via the actual integration" do
+      assert_raises(RuntimeError) { @klass.integrate!(nil) }
+    end
+  end
+end

data/test/radar/matchers/local_request_matcher_test.rb

@@ -0,0 +1,26 @@
+require 'test_helper'
+
+class LocalRequestMatcherTest < Test::Unit::TestCase
+  context "class matcher class" do
+    setup do
+      @klass = Radar::Matchers::LocalRequestMatcher
+      @hash = {}
+      @event = create_exception_event
+      @event.stubs(:to_hash).returns(@hash)
+    end
+
+    should "match if the IP matches a local IP" do
+      @hash[:request] = { :remote_ip => "127.0.0.1" }
+      assert @klass.new.matches?(@event)
+    end
+
+    should "not match if the IP does not match a local IP" do
+      @hash[:request] = { :remote_ip => "33.33.33.10" }
+      assert !@klass.new.matches?(@event)
+    end
+
+    should "not match if the field doesn't exist" do
+      assert !@klass.new.matches?(@event)
+    end
+  end
+end

data/test/radar/reporter/hoptoad_reporter_test.rb

@@ -0,0 +1,20 @@
+require 'test_helper'
+
+class HoptoadReporterTest < Test::Unit::TestCase
+  context "hoptoad reporter class" do
+    setup do
+      @klass = Radar::Reporter::HoptoadReporter
+    end
+
+    should "default notifier information to Radar" do
+      instance = @klass.new
+      assert_equal "Radar", instance.notifier_name
+      assert_equal Radar::VERSION, instance.notifier_version
+      assert !instance.notifier_url.empty?
+    end
+
+    should "require an API key" do
+      assert_raises(ArgumentError) { @klass.new.report(create_exception_event) }
+    end
+  end
+end

data/test/radar/reporter/logger_reporter_test.rb

@@ -10,10 +10,6 @@ class LoggerReporterTest < Test::Unit::TestCase
       assert_raises(ArgumentError) { @klass.new.report(create_exception_event) }
     end
 
-    should "raise an argument error if an invalid logger is given" do
-      assert_raises(ArgumentError) { @klass.new(:log_object => 7).report(create_exception_event) }
-    end
-
     should "raise an argument error if the logger doesn't respond to the log level" do
       assert_raises(ArgumentError) { @klass.new(:log_object => Logger.new(nil), :log_level => :bananas).report(create_exception_event) }
     end

metadata

@@ -4,9 +4,9 @@ version: !ruby/object:Gem::Version
   prerelease: false
   segments: 
   - 0
-  - 3
+  - 4
   - 0
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors: 
 - Mitchell Hashimoto
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-08-21 00:00:00 -07:00
+date: 2010-10-05 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -33,8 +33,22 @@ dependencies:
   prerelease: false
   version_requirements: *id001
 - !ruby/object:Gem::Dependency 
-  name: bundler
+  name: builder
   requirement: &id002 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ~>
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 2
+        - 0
+        version: "2.0"
+  type: :runtime
+  prerelease: false
+  version_requirements: *id002
+- !ruby/object:Gem::Dependency 
+  name: bundler
+  requirement: &id003 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -48,10 +62,10 @@ dependencies:
         version: 1.0.0.rc.5
   type: :development
   prerelease: false
-  version_requirements: *id002
+  version_requirements: *id003
 - !ruby/object:Gem::Dependency 
   name: shoulda
-  requirement: &id003 !ruby/object:Gem::Requirement 
+  requirement: &id004 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -61,10 +75,10 @@ dependencies:
         version: "0"
   type: :development
   prerelease: false
-  version_requirements: *id003
+  version_requirements: *id004
 - !ruby/object:Gem::Dependency 
   name: mocha
-  requirement: &id004 !ruby/object:Gem::Requirement 
+  requirement: &id005 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -74,10 +88,10 @@ dependencies:
         version: "0"
   type: :development
   prerelease: false
-  version_requirements: *id004
+  version_requirements: *id005
 - !ruby/object:Gem::Dependency 
   name: rake
-  requirement: &id005 !ruby/object:Gem::Requirement 
+  requirement: &id006 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -87,7 +101,7 @@ dependencies:
         version: "0"
   type: :development
   prerelease: false
-  version_requirements: *id005
+  version_requirements: *id006
 description: Radar provides a drop-in solution to catching and reporting errors in your libraries and applications.
 email: 
 - mitchell.hashimoto@gmail.com
@@ -110,42 +124,57 @@ files:
 - examples/README.md
 - examples/rack/README.md
 - examples/rack/config.ru
+- examples/sinatra/README.md
+- examples/sinatra/example.rb
 - lib/radar.rb
 - lib/radar/application.rb
+- lib/radar/backtrace.rb
 - lib/radar/config.rb
 - lib/radar/data_extensions/host_environment.rb
 - lib/radar/data_extensions/rack.rb
+- lib/radar/data_extensions/rails2.rb
+- lib/radar/data_extensions/request_helper.rb
 - lib/radar/error.rb
 - lib/radar/exception_event.rb
 - lib/radar/filters/key_filter.rb
 - lib/radar/integration/rack.rb
+- lib/radar/integration/rails2.rb
+- lib/radar/integration/rails2/action_controller_rescue.rb
 - lib/radar/integration/rails3.rb
 - lib/radar/integration/rails3/generator.rb
 - lib/radar/integration/rails3/railtie.rb
 - lib/radar/integration/rails3/templates/README
 - lib/radar/integration/rails3/templates/radar.rb
+- lib/radar/integration/sinatra.rb
 - lib/radar/logger.rb
 - lib/radar/matchers/backtrace_matcher.rb
 - lib/radar/matchers/class_matcher.rb
+- lib/radar/matchers/local_request_matcher.rb
 - lib/radar/reporter.rb
 - lib/radar/reporter/file_reporter.rb
+- lib/radar/reporter/hoptoad_reporter.rb
 - lib/radar/reporter/io_reporter.rb
 - lib/radar/reporter/logger_reporter.rb
 - lib/radar/support.rb
 - lib/radar/version.rb
 - radar.gemspec
 - test/radar/application_test.rb
+- test/radar/backtrace_test.rb
 - test/radar/config_test.rb
 - test/radar/data_extensions/host_environment_test.rb
 - test/radar/data_extensions/rack_test.rb
+- test/radar/data_extensions/rails2_test.rb
 - test/radar/exception_event_test.rb
 - test/radar/filters/key_filter_test.rb
 - test/radar/integration/rack_test.rb
 - test/radar/integration/rails3_test.rb
+- test/radar/integration/sinatra_test.rb
 - test/radar/logger_test.rb
 - test/radar/matchers/backtrace_matcher_test.rb
 - test/radar/matchers/class_matcher_test.rb
+- test/radar/matchers/local_request_matcher_test.rb
 - test/radar/reporter/file_reporter_test.rb
+- test/radar/reporter/hoptoad_reporter_test.rb
 - test/radar/reporter/io_reporter_test.rb
 - test/radar/reporter/logger_reporter_test.rb
 - test/radar/reporter_test.rb
@@ -165,7 +194,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: -3089432238801880584
+      hash: 2858053667388978751
       segments: 
       - 0
       version: "0"