diff options
author | Johannes Opper <johannes.opper@gmail.com> | 2015-07-07 21:52:28 +0200 |
---|---|---|
committer | Johannes Opper <johannes.opper@gmail.com> | 2015-08-04 00:38:18 +0200 |
commit | 3860e6b2bf486d8b27d433daab358dbc68ae3448 (patch) | |
tree | 136f0b6e7cfeac3ef48ea1c41c37332083dba8d2 /activejob/test/cases | |
parent | e598967548114da4f8d85070584460108a7305ff (diff) | |
download | rails-3860e6b2bf486d8b27d433daab358dbc68ae3448.tar.gz rails-3860e6b2bf486d8b27d433daab358dbc68ae3448.tar.bz2 rails-3860e6b2bf486d8b27d433daab358dbc68ae3448.zip |
Fixes #20799
When `#perform_later` is called the locale isn't stored on the
queue, which results in a locale reset when the job is performed.
An example of the problem:
I18n.locale = 'de'
HelloJob.perform_now # german message, correct
but
I18n.locale = 'de'
HelloJob.perform_later # english message, incorrect
This PR attaches the current I18n.locale to every job during the
serialization process. It is then restored during deserialization
and used to perform the job with the correct locale.
It falls back to the default locale if no serialized locale is
found in order to provide backward compatibility with previously
stored jobs. It is not necessary to clear the queue for the update.
Diffstat (limited to 'activejob/test/cases')
-rw-r--r-- | activejob/test/cases/job_serialization_test.rb | 16 | ||||
-rw-r--r-- | activejob/test/cases/translation_test.rb | 20 |
2 files changed, 36 insertions, 0 deletions
diff --git a/activejob/test/cases/job_serialization_test.rb b/activejob/test/cases/job_serialization_test.rb index db22783030..e23380a7bf 100644 --- a/activejob/test/cases/job_serialization_test.rb +++ b/activejob/test/cases/job_serialization_test.rb @@ -12,4 +12,20 @@ class JobSerializationTest < ActiveSupport::TestCase GidJob.perform_later @person assert_equal "Person with ID: 5", JobBuffer.last_value end + + test 'serialize includes current locale' do + assert_equal :en, HelloJob.new.serialize['locale'] + end + + test 'deserialize sets locale' do + job = HelloJob.new + job.deserialize 'locale' => :es + assert_equal :es, job.locale + end + + test 'deserialize sets default locale' do + job = HelloJob.new + job.deserialize({}) + assert_equal :en, job.locale + end end diff --git a/activejob/test/cases/translation_test.rb b/activejob/test/cases/translation_test.rb new file mode 100644 index 0000000000..d5e3aaf9e3 --- /dev/null +++ b/activejob/test/cases/translation_test.rb @@ -0,0 +1,20 @@ +require 'helper' +require 'jobs/translated_hello_job' + +class TranslationTest < ActiveSupport::TestCase + setup do + JobBuffer.clear + I18n.available_locales = [:en, :de] + @job = TranslatedHelloJob.new('Johannes') + end + + teardown do + I18n.available_locales = [:en] + end + + test 'it performs the job in the given locale' do + @job.locale = :de + @job.perform_now + assert_equal "Johannes says Guten Tag", JobBuffer.last_value + end +end |