airbrake 8.0.1 → 8.1.0

data/spec/integration/rails/rake_spec.rb

@@ -3,6 +3,14 @@ require 'spec_helper'
 RSpec.describe "Rake integration" do
   let(:endpoint) { 'https://api.airbrake.io/api/v3/projects/113743/notices' }
 
+  let(:routes_endpoint) do
+    'https://api.airbrake.io/api/v5/projects/113743/routes-stats'
+  end
+
+  let(:queries_endpoint) do
+    'https://api.airbrake.io/api/v5/projects/113743/queries-stats'
+  end
+
   def wait_for_a_request_with_body(body)
     wait_for(a_request(:post, endpoint).with(body: body)).to have_been_made.once
   end
@@ -13,8 +21,16 @@ RSpec.describe "Rake integration" do
   end
 
   before do
-    Rails.application.load_tasks
     stub_request(:post, endpoint).to_return(status: 201, body: '{}')
+
+    # Airbrake Ruby has a background thread that sends performance requests
+    # periodically. We don't want this to get in the way.
+    [routes_endpoint, queries_endpoint].each do |e|
+      stub_request(:put, e).to_return(status: 200, body: '')
+    end
+
+    Rails.application.load_tasks
+
     expect { faulty_task.invoke }.to raise_error(AirbrakeTestError)
   end
 

data/spec/integration/shared_examples/rack_examples.rb

@@ -5,15 +5,31 @@ RSpec.shared_examples 'rack examples' do
 
   let(:endpoint) { 'https://api.airbrake.io/api/v3/projects/113743/notices' }
 
+  let(:routes_endpoint) do
+    'https://api.airbrake.io/api/v5/projects/113743/routes-stats'
+  end
+
+  let(:queries_endpoint) do
+    'https://api.airbrake.io/api/v5/projects/113743/queries-stats'
+  end
+
   def wait_for_a_request_with_body(body)
-    wait_for(a_request(:post, endpoint).with(body: body)).to have_been_made.once
+    allow(Airbrake.notifiers[:performance][:default]).
+      to receive(:notify).and_return(nil)
+    wait_for(a_request(:post, endpoint).with(body: body)).
+      to have_been_made.at_least_once
   end
 
   before do
     # Make sure the Logger integration doesn't get in the way.
     allow_any_instance_of(Logger).to receive(:airbrake_notifier).and_return(nil)
 
-    stub_request(:post, endpoint).to_return(status: 201, body: '{}')
+    allow(Airbrake.notifiers[:performance][:default]).
+      to receive(:notify).and_return(nil)
+    stub_request(:post, endpoint).to_return(status: 200, body: '')
+    [routes_endpoint, queries_endpoint].each do |endpoint|
+      stub_request(:put, endpoint).to_return(status: 200, body: '')
+    end
   end
 
   describe "application routes" do
@@ -47,6 +63,7 @@ RSpec.shared_examples 'rack examples' do
       before do
         login_as(OpenStruct.new(user_params))
         get '/crash'
+        sleep 2
       end
 
       context "when the user has first and last names" do
@@ -87,6 +104,7 @@ RSpec.shared_examples 'rack examples' do
     context "when additional parameters present" do
       before do
         get '/crash', nil, 'HTTP_USER_AGENT' => 'Bot', 'HTTP_REFERER' => 'bingo.com'
+        sleep 2
       end
 
       it "contains url" do

data/spec/unit/shoryuken_spec.rb

@@ -56,7 +56,7 @@ RSpec.describe Airbrake::Shoryuken::ErrorHandler do
 
   context 'when Airbrake is not configured' do
     before do
-      @notifiers = Airbrake.instance_variable_get(:@notifiers)
+      @notifiers = Airbrake.notifiers[:notice]
       @default_notifier = @notifiers.delete(:default)
     end
 

data/spec/unit/sidekiq_spec.rb

@@ -34,7 +34,7 @@ if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.2.2')
 
     context "when Airbrake is not configured" do
       before do
-        @notifiers = Airbrake.instance_variable_get(:@notifiers)
+        @notifiers = Airbrake.notifiers[:notice]
         @default_notifier = @notifiers.delete(:default)
       end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: airbrake
 version: !ruby/object:Gem::Version
-  version: 8.0.1
+  version: 8.1.0
 platform: ruby
 authors:
 - Airbrake Technologies, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-01-23 00:00:00.000000000 Z
+date: 2019-02-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: airbrake-ruby
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '3.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '3.2'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -260,9 +260,11 @@ files:
 - lib/airbrake/rack/user_filter.rb
 - lib/airbrake/rails.rb
 - lib/airbrake/rails/action_controller.rb
-- lib/airbrake/rails/action_controller_subscriber.rb
+- lib/airbrake/rails/action_controller_notify_subscriber.rb
+- lib/airbrake/rails/action_controller_route_subscriber.rb
 - lib/airbrake/rails/active_job.rb
 - lib/airbrake/rails/active_record.rb
+- lib/airbrake/rails/active_record_subscriber.rb
 - lib/airbrake/rake.rb
 - lib/airbrake/rake/tasks.rb
 - lib/airbrake/resque.rb