なるようになるブログ

読書感想文かrailsについてかrubyについてか

rails commit log流し読み(2016/03/07)

2016/03/07分のコミットです。

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

actionpack/CHANGELOG.md


Silence deprecation message for dynamic controller and actions on action_view test cases

actionview/test/abstract_unit.rbの修正です。

Deprecate :controller and :action path parametersで、routes.rbのpath parametersに:controller:actionを使用するのがdeprecateになたのですが、Action Viewのテストで使用している箇所があり、不要なwarningが出ないようActiveSupport::Deprecation.silenceで処理を囲むよう修正しています。


clarify that helper_method makes both methods available in the view

actionpack/lib/abstract_controller/helpers.rbのdocの修正です。

helper_methodメソッドのdocに記載されているexampleコードが2つのメソッド(#current_user#logged_in?)がviewで出来るようになるコードになっていたのですが、説明は#current_userメソッドについてのみ記載していたので、#logged_inメソッドについても記載するよう修正しています。


Prevent not-intended loading of ActionDispatch::IntegrationTest

actionpack/lib/action_dispatch/testing/integration.rbactionmailer/lib/action_mailer/railtie.rbの修正です。

ActionDispatch::IntegrationTest load時に実行される処理を定義する為のaction_dispatch_integration_test hookを追加しています。

reset ActionMailer::Base.deliveries in `ActionDispatch::Integration… · rails/rails@9d37874の対応により、全てのenvironmentsで`ActionDispatch::IntegrationTest classがloadされるようになってしまいました。

が、ActionDispatch::IntegrationTestクラスは基本的にはtest envでだけloadされていれば良いclassなので、test envでだけ読み込まれるようにする為に、Action MailerのinitializerでActionDispatch::IntegrationTestActionMailer::TestCase::ClearTestDeliveriesをincludeしていたのを、ActionDispatch::IntegrationTest load時にActionMailer::TestCase::ClearTestDeliveriesをincludeするよう修正しています。