Permalink
Browse files

Merge pull request #16132 from tgxworld/fix_test_not_unsubscribing

Fix incorrect unsubscription.
  • Loading branch information...
2 parents d294ef0 + d9396a0 commit 6e23c8242d9364e295ace14b3b01ae33fe31989b @spastorino spastorino committed Jul 11, 2014
Showing with 2 additions and 2 deletions.
  1. +2 −2 railties/test/rack_logger_test.rb
@@ -39,11 +39,11 @@ def finish(name, id, payload)
def setup
@subscriber = Subscriber.new
@notifier = ActiveSupport::Notifications.notifier
- notifier.subscribe 'request.action_dispatch', subscriber
+ @subscription = notifier.subscribe 'request.action_dispatch', subscriber
end
def teardown
- notifier.unsubscribe subscriber
+ notifier.unsubscribe @subscription
end
def test_notification

0 comments on commit 6e23c82

Please sign in to comment.