remove Ruby warning from Active Job test helper methods
This removes the following warnings. ``` /home/travis/build/rails/rails/activejob/lib/active_job/test_helper.rb:241: warning: shadowing outer local variable - job /home/travis/build/rails/rails/activejob/lib/active_job/test_helper.rb:265: warning: shadowing outer local variable - job ```
This commit is contained in:
parent
796049ef3e
commit
5b825c55fe
|
@ -238,8 +238,8 @@ module ActiveJob
|
||||||
serialized_args = serialize_args_for_assertion(expected)
|
serialized_args = serialize_args_for_assertion(expected)
|
||||||
yield
|
yield
|
||||||
in_block_jobs = enqueued_jobs.drop(original_enqueued_jobs_count)
|
in_block_jobs = enqueued_jobs.drop(original_enqueued_jobs_count)
|
||||||
matching_job = in_block_jobs.find do |job|
|
matching_job = in_block_jobs.find do |in_block_job|
|
||||||
serialized_args.all? { |key, value| value == job[key] }
|
serialized_args.all? { |key, value| value == in_block_job[key] }
|
||||||
end
|
end
|
||||||
assert matching_job, "No enqueued job found with #{expected}"
|
assert matching_job, "No enqueued job found with #{expected}"
|
||||||
instantiate_job(matching_job)
|
instantiate_job(matching_job)
|
||||||
|
@ -262,8 +262,8 @@ module ActiveJob
|
||||||
serialized_args = serialize_args_for_assertion(expected)
|
serialized_args = serialize_args_for_assertion(expected)
|
||||||
perform_enqueued_jobs { yield }
|
perform_enqueued_jobs { yield }
|
||||||
in_block_jobs = performed_jobs.drop(original_performed_jobs_count)
|
in_block_jobs = performed_jobs.drop(original_performed_jobs_count)
|
||||||
matching_job = in_block_jobs.find do |job|
|
matching_job = in_block_jobs.find do |in_block_job|
|
||||||
serialized_args.all? { |key, value| value == job[key] }
|
serialized_args.all? { |key, value| value == in_block_job[key] }
|
||||||
end
|
end
|
||||||
assert matching_job, "No performed job found with #{expected}"
|
assert matching_job, "No performed job found with #{expected}"
|
||||||
instantiate_job(matching_job)
|
instantiate_job(matching_job)
|
||||||
|
|
Loading…
Reference in New Issue