aboutsummaryrefslogtreecommitdiffstats
path: root/activesupport/test/notifications_test.rb
Commit message (Collapse)AuthorAgeFilesLines
* Add three new rubocop rulesRafael Mendonça França2016-08-161-1/+1
| | | | | | | | Style/SpaceBeforeBlockBraces Style/SpaceInsideBlockBraces Style/SpaceInsideHashLiteralBraces Fix all violations in the repository.
* modernizes hash syntax in activesupportXavier Noria2016-08-061-13/+13
|
* applies new string literal convention in activesupport/testXavier Noria2016-08-061-14/+14
| | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default.
* Adds exception object to instrumenter's payloadRyan T. Hosford2015-12-311-1/+1
| | | | | | | - Adds new key/value pair to payload when an exception is raised e.g. `:exception_object=> #<RuntimeError: FAIL>` - Updates relevant test - Adds CHANGELOG entry
* Subscribing to notifications while inside the said instrumented section.thedarkone2015-11-281-0/+15
| | | | | | | | | | | | | The issue is that on the exit from Instrumenter#instrument section, an Evented listener will run into an error because its thread local (Thread.current[:_timestack]) has not been set up by the #start method (this obviously happens because the Evented listeners didn't exist at the time, since no subscribtion to that section was made yet). Note: support for subscribing to instrumented sections, while being inside those instrumented sections, might be removed in the future. Maybe fixes #21873.
* Bug fix: Evented notification subscribers can handle published eventsCarl Lerche2013-05-171-0/+22
|
* Fix Typo existant -> existent [ci skip]Prathamesh Sonpatki2013-05-081-1/+1
|
* ActiveSupport::Notifications::Instrumenter#instrument should yieldstopdropandrew2013-03-021-1/+1
| | | | its payload the same way that ActiveSupport::Notifications does. Fix spelling in test name.
* make events not use date and time to determine parent_of. fixes #5932タコ焼き仮面2012-06-181-1/+3
|
* implements AS::Notifications.subscribed, which provides subscriptions to ↵Xavier Noria2011-11-051-0/+20
| | | | events while a block runs
* Requiring delegate.Arun Agrawal2011-08-161-0/+1
|
* Replace references to ActiveSupport::SecureRandom with just SecureRandom, ↵Jon Leighton2011-05-231-1/+1
| | | | and require 'securerandom' from the stdlib when active support is required.
* just use an attr_accessor so we do not pay ||= on every notification callAaron Patterson2011-02-091-2/+7
|
* Revert the previous three commits.José Valim2010-07-251-9/+0
| | | | | | * AS::Notifications#instrument should not measure anything, it is not its responsibility; * Adding another argument to AS::Notifications#instrument API needs to be properly discussed;
* use a hash to collect optional statistics about the instrumentationAaron Patterson2010-07-251-0/+9
|
* Get rid of instrumenter.elapsed.José Valim2010-07-241-9/+0
|
* Add a test for elapsed and require missing benchmark file.José Valim2010-07-201-17/+13
|
* Float comparison adjustmentSantiago Pastorino2010-07-181-1/+1
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* We shouldn't rely on float comparison, delta added just in case float ↵Santiago Pastorino2010-07-121-3/+3
| | | | representation of this values aren't equal
* Event should be aware if yielded block failed or not.José Valim2010-05-021-0/+2
|
* Make notifications go off even when an error is raised, so that we capture ↵Justin George2010-05-021-2/+2
| | | | | | | | | | | | | | the underlying performance data [#4505 state:resolved] This is important when trying to keep track of many layers of interrelated calls i.e.: ActiveRecord::Base.transaction do MyModel.find(1) #ActiveRecord::NotFound end # should capture the full time until the error propagation Signed-off-by: José Valim <jose.valim@gmail.com>
* allow unsubscribe by name or subscription [#4433 state:resolved]David Chelimsky2010-04-241-0/+22
| | | | Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* Mark bang instrumentations as something that you shuold not be listening to.José Valim2010-03-171-3/+3
|
* Optimize AS::Notifications to remember which subscribers don't match and not ↵Carlhuda2010-03-011-2/+29
| | | | run them. This will allow notifications that are only useful in dev or testing to run efficiently in production.
* You can unsubscribe a subscriberCarlhuda2010-03-011-1/+13
|
* Rename Rails::Subscriber to Rails::LogSubscriberPrem Sichanugrist2010-02-161-3/+3
|
* Remove instrument! and require thread from AS::Notifications.José Valim2010-02-041-17/+1
|
* deOMGifying Railties, Active Support, and Action PackMikel Lindsaar2010-01-311-3/+3
|
* Default to sync instrumentation.José Valim2010-01-211-17/+1
|
* Add ActionDispatch::Notifications middleware.José Valim2010-01-171-0/+16
|
* Yield the payload notifications for further modification (like adding the ↵José Valim2010-01-151-2/+2
| | | | result).
* Add instrument! to notifications which adds the result to the payload.José Valim2010-01-141-1/+11
|
* Do not send notifications when instrumentation raise an error.José Valim2010-01-131-7/+4
|
* instrumenter should be accessible from ActiveSupport::Notifications.José Valim2010-01-061-7/+10
|
* Expose Instrumenter id in Notifications.José Valim2010-01-041-0/+4
|
* Remove ActionView inline logging to ActiveSupport::Notifications and create ↵José Valim2009-12-261-12/+10
| | | | ActionController::Base#log_event, so everything can be logged within one listener. Also expose log_process_action as a hook for different modules to include their own information during the action processing. This allow ActiveRecord to hook and any other ORM. Finally, this commit changes 'Processing' and 'Rendering' in logs to 'Processed' and 'Rendered' because at the point it's logged, everying already happened.
* Notifications: synchronous fanout queue pushes events to subscribers rather ↵Jeremy Kemper2009-11-291-0/+10
| | | | than having them concurrently pull
* Notifications: extract central Notifier, cordon off the internal Fanout ↵Jeremy Kemper2009-11-281-158/+112
| | | | implementation, and segregate instrumentation concerns
* Expose a simple Queue#wait to block until all notifications are drainedJeremy Kemper2009-11-281-1/+1
|
* Revert "Create SyncListener. Since they do not rely on Thread, they can be ↵Jeremy Kemper2009-11-281-16/+1
| | | | | | | | used on Google App Engine." Take a step back on this API direction. This reverts commit 8104f65c3225453d13307c3c2733c2a8f99e491a.
* Create SyncListener. Since they do not rely on Thread, they can be used on ↵José Valim2009-11-231-1/+16
| | | | | | Google App Engine. Signed-off-by: Yehuda Katz <wycats@Yehuda-Katz.local>
* Notifications: queue.drained? for testability in place of brittle sleepsJeremy Kemper2009-11-131-10/+14
|
* Fix duration check for longer sleepJeremy Kemper2009-11-131-1/+1
|
* CI: slow down brittle notifications testsJeremy Kemper2009-11-131-9/+10
|
* Change Event#thread_id to #transaction_id. Defaults to one "transaction" per ↵Yehuda Katz2009-10-281-0/+20
| | | | thread but you can explicitly declare the start of a new one. This makes it possible for each request to have it own id.
* AS::Notifications.subscribe blocks are now yielded the arguments to pass to ↵Yehuda Katz2009-10-271-4/+14
| | | | AS::Notifications::Event.new
* Some optimizations on AS::Notifications. This does not change the ↵Yehuda Katz2009-10-271-11/+18
| | | | public-facing API.
* Renamed Orchestra to Notifications once again [#3321 state:resolved]José Valim2009-10-151-0/+164
|
* Revert "Rename Orchestra to Notifications [#3321 state:resolved]"José Valim2009-10-151-161/+0
| | | | This reverts commit 8cbf825425dc8ad3770881ea4e100b9023c69ce2.
* Rename Orchestra to Notifications [#3321 state:resolved]Joshua Peek2009-10-141-0/+161