rails commit log流し読み(2018/11/22) - なるようになるブログ

2018/11/22分のコミットです。

CHANGELOGにのったコミットは以下の通りです。

activejob/CHANGELOG.md

Allow all assertion helpers that have a only and except keyword to accept Procs.

actionmailer/CHANGELOG.md

Fix ActionMailer assertions not working when a Mail defines a custom delivery job class

Merge pull request #34498 from ThomBian/thombian/fix-opclass-parsing

activerecord/lib/active_record/connection_adapters/postgresql/schema_statements.rbの修正です。

特定のカラム名(今回問題になったのはposition)のカラムに対してindexを作成する際に、opclassに指定した値が使用されないバグがあったのを修正しています。

特定のカラム名の場合のみカラムがquoteされるのですが、そのquoteに対する考慮が無かった為。

Allow all ActiveJob assertion helper to accept Proc in their only kw:

activejob/lib/active_job/queue_adapters/test_adapter.rb、 activejob/lib/active_job/test_helper.rbの修正です。

Active Jobのassertion methosのonly、except引数にProcを指定出来るよう修正しています。

例。

assert_performed_jobs(1, only: ->(job) { job.is_a?(HelloJob) }) do
  HelloJob.perform_later('jeremy')
  LoggingJob.perform_later('stewie')
  RescueJob.perform_later('david')
end

Fix ActionMailer assertion not working for mail defining delivery_job:

actionmailer/lib/action_mailer/test_helper.rbの修正です。

mailerにcustom jobを指定した場合にAction Mailerのassertion methosが動作していなかった(対象となるjobがActionMailer::DeliveryJob, ActionMailer::Parameterized::DeliveryJob固定になっている為)のを、custom jobを指定した場合も動作するよう修正しています。

Fix ruby warnings

activesupport/test/dependencies_test.rbの修正です。

テストでRubyのwarningがでていたのを修正しています。

Compile packs before run test

railties/test/application/rake_test.rbの修正です。

test_scaffold_tests_pass_by_defaultでテスト実行前にpacksをcompileするよう修正しています。

テスト実行時にcompileすると、パラレルテストが有効の場合に同時にcompileが実行され、結果ファイルが不正な状態になってしまう事がある為。

Use squiggly heredoc to strip odd indentation in the executed SQL

Active Recordの修正です。

生SQL実行時に、squiggly heredocを使用してSQLから不要なインデントを削除するよう修正しています。