airbrake 3.1.2 → 3.1.3

data/test/notifier_test.rb

@@ -1,4 +1,4 @@
-require File.expand_path( File.join(File.dirname(__FILE__), 'helper') )
+require File.expand_path '../helper', __FILE__
 
 class NotifierTest < Test::Unit::TestCase
 
@@ -17,8 +17,7 @@ class NotifierTest < Test::Unit::TestCase
 
   def assert_sent(notice, notice_args)
     assert_received(Airbrake::Notice, :new) {|expect| expect.with(has_entries(notice_args)) }
-    assert_received(notice, :to_xml)
-    assert_received(Airbrake.sender, :send_to_airbrake) {|expect| expect.with(notice.to_xml) }
+    assert_received(Airbrake.sender, :send_to_airbrake) {|expect| expect.with(notice) }
   end
 
   def set_public_env
@@ -140,6 +139,37 @@ class NotifierTest < Test::Unit::TestCase
     assert_received(sender, :send_to_airbrake) {|expect| expect.never }
   end
 
+  should "deliver exception in async-mode" do
+    Airbrake.configure do |config|
+      config.environment_name = 'production'
+      config.async do |notice|
+        Airbrake.sender.send_to_airbrake(notice)
+      end
+    end
+    exception = build_exception
+    sender = stub_sender!
+    notice = stub_notice!
+
+    Airbrake.notify(exception)
+
+    assert_sent(notice, :exception => exception)
+  end
+
+  should "pass notice in async-mode" do
+    received_notice = nil
+    Airbrake.configure do |config|
+      config.environment_name = 'production'
+      config.async {|notice| received_notice = notice}
+    end
+    exception = build_exception
+    sender = stub_sender!
+    notice = stub_notice!
+
+    Airbrake.notify(exception)
+
+    assert_equal received_notice, notice
+  end
+
   should "deliver an ignored exception when notifying manually" do
     set_public_env
     exception = build_exception
@@ -157,7 +187,7 @@ class NotifierTest < Test::Unit::TestCase
     config_opts = { 'one' => 'two', 'three' => 'four' }
     notice = stub_notice!
     stub_sender!
-    Airbrake.configuration = stub('config', :merge => config_opts, :public? => true)
+    Airbrake.configuration = stub('config', :merge => config_opts, :public? => true,:async? => nil)
 
     Airbrake.notify(exception)
 

data/test/rack_test.rb

@@ -1,4 +1,4 @@
-require File.dirname(__FILE__) + '/helper'
+require File.expand_path '../helper', __FILE__
 
 class RackTest < Test::Unit::TestCase
 

data/test/rails_initializer_test.rb

@@ -1,4 +1,4 @@
-require File.dirname(__FILE__) + '/helper'
+require File.expand_path '../helper', __FILE__
 
 require 'airbrake/rails'
 

data/test/recursion_test.rb

@@ -1,4 +1,4 @@
-require File.dirname(__FILE__) + '/helper'
+require File.expand_path '../helper', __FILE__
 
 class RecursionTest < Test::Unit::TestCase
   should "not allow infinite recursion" do

data/test/sender_test.rb

@@ -1,4 +1,4 @@
-require File.expand_path( File.join(File.dirname(__FILE__), 'helper') )
+require File.expand_path '../helper', __FILE__
 
 class SenderTest < Test::Unit::TestCase
 
@@ -14,6 +14,7 @@ class SenderTest < Test::Unit::TestCase
 
   def send_exception(args = {})
     notice = args.delete(:notice) || build_notice_data
+    notice.stubs(:to_xml)
     sender = args.delete(:sender) || build_sender(args)
     sender.send_to_airbrake(notice)
   end
@@ -51,7 +52,7 @@ class SenderTest < Test::Unit::TestCase
                    :proxy_port => proxy_port,
                    :proxy_user => proxy_user,
                    :proxy_pass => proxy_pass)
-    assert_received(http, :post) do |expect| 
+    assert_received(http, :post) do |expect|
       expect.with(uri.path, anything, Airbrake::HEADERS)
     end
     assert_received(Net::HTTP, :Proxy) do |expect|
@@ -70,19 +71,19 @@ class SenderTest < Test::Unit::TestCase
         proxy = stub()
         proxy.stubs(:new).raises(NoMemoryError)
         Net::HTTP.stubs(:Proxy => proxy)
-      
+
         assert_raise NoMemoryError do
           build_sender.send(:setup_http_connection)
         end
       end
-      
+
       should "be logged" do
         proxy = stub()
         proxy.stubs(:new).raises(RuntimeError)
         Net::HTTP.stubs(:Proxy => proxy)
-        
+
         sender = build_sender
-        sender.expects(:log).with(:error, includes('Failure initializing the HTTP connection'))
+        sender.expects(:log)
 
         assert_raise RuntimeError do
           sender.send(:setup_http_connection)
@@ -90,26 +91,26 @@ class SenderTest < Test::Unit::TestCase
 
       end
     end
-    
+
     context "unexpected exception sending problems" do
       should "be logged" do
         sender  = build_sender
         sender.stubs(:setup_http_connection).raises(RuntimeError.new)
-        
-        sender.expects(:log).with(:error, includes('Cannot send notification. Error'))
-        sender.send_to_airbrake("stuff")
+
+        sender.expects(:log)
+        send_exception(:sender => sender)
       end
-      
+
       should "return nil no matter what" do
         sender  = build_sender
         sender.stubs(:setup_http_connection).raises(LocalJumpError)
-        
+
         assert_nothing_thrown do
-          assert_nil sender.send_to_airbrake("stuff")
+          assert_nil sender.send_to_airbrake(build_notice_data)
         end
       end
     end
-    
+
     should "return nil on failed posting" do
       http = stub_http
       http.stubs(:post).raises(Errno::ECONNREFUSED)
@@ -173,7 +174,7 @@ class SenderTest < Test::Unit::TestCase
       assert_equal(OpenSSL::SSL::VERIFY_PEER,        real_http.verify_mode)
       assert_equal(Airbrake.configuration.local_cert_path, real_http.ca_file)
     end
-    
+
     should "use the default DEFAULT_CERT_FILE if asked to" do
       config = Airbrake::Configuration.new
       config.use_system_ssl_cert_chain = true
@@ -184,23 +185,23 @@ class SenderTest < Test::Unit::TestCase
       http    = sender.send(:setup_http_connection)
       assert_not_equal http.ca_file, config.local_cert_path
     end
-    
+
     should "verify the connection when the use_ssl option is set (VERIFY_PEER)" do
       sender  = build_sender(:secure => true)
       http    = sender.send(:setup_http_connection)
       assert_equal(OpenSSL::SSL::VERIFY_PEER, http.verify_mode)
     end
-    
+
     should "use the default cert (OpenSSL::X509::DEFAULT_CERT_FILE) only if explicitly told to" do
       sender  = build_sender(:secure => true)
       http    = sender.send(:setup_http_connection)
-      
+
       assert_equal(Airbrake.configuration.local_cert_path, http.ca_file)
 
       File.stubs(:exist?).with(OpenSSL::X509::DEFAULT_CERT_FILE).returns(true)
       sender  = build_sender(:secure => true, :use_system_ssl_cert_chain => true)
       http    = sender.send(:setup_http_connection)
-      
+
       assert_not_equal(Airbrake.configuration.local_cert_path, http.ca_file)
       assert_equal(OpenSSL::X509::DEFAULT_CERT_FILE, http.ca_file)
     end
@@ -229,9 +230,9 @@ class SenderTest < Test::Unit::TestCase
       assert_received(Net::HTTP, :new) {|expect| expect.with('example.org', 80) }
     end
 
-    
+
   end
-  
+
   context "network timeouts" do
     should "default the open timeout to 2 seconds" do
       http = stub_http

data/test/user_informer_test.rb

@@ -1,4 +1,4 @@
-require File.dirname(__FILE__) + '/helper'
+require File.expand_path '../helper', __FILE__
 
 class UserInformerTest < Test::Unit::TestCase
   should "modify output if there is an airbrake id" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: airbrake
 version: !ruby/object:Gem::Version
-  version: 3.1.2
+  version: 3.1.3
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-06-29 00:00:00.000000000Z
+date: 2012-09-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: builder
-  requirement: &70111147519640 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,31 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70111147519640
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70111147517460 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: girl_friday
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +53,15 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70111147517460
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: actionpack
-  requirement: &70111147515340 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +69,15 @@ dependencies:
         version: 2.3.8
   type: :development
   prerelease: false
-  version_requirements: *70111147515340
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.3.8
 - !ruby/object:Gem::Dependency
   name: activerecord
-  requirement: &70111147514720 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -54,21 +85,47 @@ dependencies:
         version: 2.3.8
   type: :development
   prerelease: false
-  version_requirements: *70111147514720
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.3.8
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70111147514140 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.3.8
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
         version: 2.3.8
+- !ruby/object:Gem::Dependency
+  name: mocha
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.10.5
   type: :development
   prerelease: false
-  version_requirements: *70111147514140
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.10.5
 - !ruby/object:Gem::Dependency
   name: bourne
-  requirement: &70111147513560 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +133,15 @@ dependencies:
         version: '1.0'
   type: :development
   prerelease: false
-  version_requirements: *70111147513560
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: cucumber
-  requirement: &70111147512980 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -87,10 +149,15 @@ dependencies:
         version: 0.10.6
   type: :development
   prerelease: false
-  version_requirements: *70111147512980
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.10.6
 - !ruby/object:Gem::Dependency
   name: fakeweb
-  requirement: &70111147494720 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -98,10 +165,15 @@ dependencies:
         version: 1.3.0
   type: :development
   prerelease: false
-  version_requirements: *70111147494720
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.3.0
 - !ruby/object:Gem::Dependency
   name: nokogiri
-  requirement: &70111147494200 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -109,10 +181,15 @@ dependencies:
         version: 1.4.3.1
   type: :development
   prerelease: false
-  version_requirements: *70111147494200
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.4.3.1
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70111147493680 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -120,10 +197,15 @@ dependencies:
         version: 2.6.0
   type: :development
   prerelease: false
-  version_requirements: *70111147493680
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.6.0
 - !ruby/object:Gem::Dependency
   name: sham_rack
-  requirement: &70111147493100 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -131,10 +213,15 @@ dependencies:
         version: 1.3.0
   type: :development
   prerelease: false
-  version_requirements: *70111147493100
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 1.3.0
 - !ruby/object:Gem::Dependency
   name: shoulda
-  requirement: &70111147492520 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -142,18 +229,76 @@ dependencies:
         version: 2.11.3
   type: :development
   prerelease: false
-  version_requirements: *70111147492520
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.11.3
 - !ruby/object:Gem::Dependency
   name: capistrano
-  requirement: &70111147491960 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.8.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
         version: 2.8.0
+- !ruby/object:Gem::Dependency
+  name: guard
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70111147491960
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: guard-test
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: 
 email: support@airbrake.io
 executables: []
@@ -206,7 +351,7 @@ files:
 - TESTING.md
 - SUPPORTED_RAILS_VERSIONS
 - install.rb
-- test/airbrake_2_2.xsd
+- test/airbrake_2_3.xsd
 - test/airbrake_tasks_test.rb
 - test/backtrace_test.rb
 - test/capistrano_test.rb
@@ -260,12 +405,12 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.11
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: Send your application errors to our hosted service and reclaim your inbox.
 test_files:
-- test/airbrake_2_2.xsd
+- test/airbrake_2_3.xsd
 - test/airbrake_tasks_test.rb
 - test/backtrace_test.rb
 - test/capistrano_test.rb