From b6b5a7ac524a8c22d9ca162e53d752141f974651 Mon Sep 17 00:00:00 2001 From: bogdanvlviv Date: Sun, 16 Sep 2018 17:39:55 +0300 Subject: `retry_job` should publish `enqueue_retry.active_job` notification Also this commit removes `:wait` from payload of `retry_stopped.active_job`. Related to https://github.com/rails/rails/pull/33751#discussion_r214140008 Follow up #33751 /cc @kaspth, @rafaelfranca --- activejob/test/cases/logging_test.rb | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'activejob/test/cases/logging_test.rb') diff --git a/activejob/test/cases/logging_test.rb b/activejob/test/cases/logging_test.rb index 2e8d2ef7c0..b5bf40c83b 100644 --- a/activejob/test/cases/logging_test.rb +++ b/activejob/test/cases/logging_test.rb @@ -173,6 +173,11 @@ class LoggingTest < ActiveSupport::TestCase end end + def test_enqueue_retry_logging_on_retry_job + perform_enqueued_jobs { RescueJob.perform_later "david" } + assert_match(/Retrying RescueJob in nil seconds, due to a nil\. The original exception was nil\./, @logger.messages) + end + def test_retry_stopped_logging perform_enqueued_jobs do RetryJob.perform_later "CustomCatchError", 6 -- cgit v1.2.3