Skip to content

Commit

Permalink
Make sure subscribed tracing events are cleared after each tests
Browse files Browse the repository at this point in the history
  • Loading branch information
st0012 committed Sep 18, 2021
1 parent 5d41ab0 commit b9fa906
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 0 deletions.
Expand Up @@ -79,6 +79,7 @@

after do
Sentry::Rails::Tracing.unsubscribe_tracing_events
expect(Sentry::Rails::Tracing.subscribed_tracing_events).to be_empty
Sentry::Rails::Tracing.remove_active_support_notifications_patch
end

Expand Down
Expand Up @@ -90,6 +90,7 @@

after do
Sentry::Rails::Tracing.unsubscribe_tracing_events
expect(Sentry::Rails::Tracing.subscribed_tracing_events).to be_empty
Sentry::Rails::Tracing.remove_active_support_notifications_patch
end

Expand Down
1 change: 1 addition & 0 deletions sentry-rails/spec/sentry/rails/tracing_spec.rb
Expand Up @@ -13,6 +13,7 @@
transport.events = []

described_class.unsubscribe_tracing_events
expect(described_class.subscribed_tracing_events).to be_empty
described_class.remove_active_support_notifications_patch
end

Expand Down
1 change: 1 addition & 0 deletions sentry-rails/spec/spec_helper.rb
Expand Up @@ -42,6 +42,7 @@

config.after :each do
Sentry::Rails::Tracing.unsubscribe_tracing_events
expect(Sentry::Rails::Tracing.subscribed_tracing_events).to be_empty
Sentry::Rails::Tracing.remove_active_support_notifications_patch
end

Expand Down

0 comments on commit b9fa906

Please sign in to comment.