なるようになるブログ

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

rails commit log流し読み(2017/03/05)

2017/03/05分のコミットです。

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

activerecord/CHANGELOG.md

railties/CHANGELOG.md


Fix rake db:schema:load with subdirectories

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

db:schema:load タスクが、サブディレクトリにあるmigrationファイルをload出来ないバグがあったのを修正しています。


[ci skip] Fix ParameterMissing exception name in docs

actionpack/lib/action_controller/metal/strong_parameters.rbのdocの修正です。

StrongParametersのdoc内でActionController::ParameterMissingActionController::MissingParameterにタイポしていたのを修正しています。


Don’t hide method that’s been documented API since Rails 2.1

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

Fix rake db:schema:load with subdirectoriesの対応でprivate APIに変更されたassume_migrated_upto_versionをpublic APIに戻しています。

内部でしか使用していないのでprivate APIで良いのでは、という事で変更されたのですが、Rails 2.1の頃からpublic APIになっており、いきなりprivate APIにするのは良くないだろう、という事でpublic APIに戻っています。


Remove useless Migrator.schema_migrations_table_name

activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rbactiverecord/lib/active_record/migration.rbの修正です。

SchemaMigration.table_nameメソッドをラップしていたActiveRecord::Migrator.schema_migrations_table_nameメソッドを削除しています。SchemaMigration.table_nameを直接呼べば良いだろう、という事で削除されています。


Avoid running system tests by default

railtiesの修正です。

bin/rails test、及びbin/rake testコマンドで、デフォルトでsystem testsは実行しないよう修正しています。

system testsは処理が重いため、テストスイート内で実行されるべきではない、という事でデフォルトでは実行されないようになっています。ちょっと違和感が。


Revert “Merge pull request #28283 from y-yagi/silence_puma_startup_messages_in_system_test”

actionpack/lib/action_dispatch/system_testing/server.rbの修正です

system testでpumaを起動する際に、pumaのスタートアップメッセージをstdoutに表示しないようにした、Silence puma startup messages in system testをrevertしています。 スタートアップメッセージはスレッド数やポートを確認するのに有用なので、見えた方が良いだろう、という事でrevertされています。


Merge pull request #28264 from y-yagi/do_not_take_screenshot_if_driver_does_not_support_it

actionpack/lib/action_dispatch/system_testing/test_helpers/screenshot_helper.rbの修正です。

system testで、driverがスクリーンショットをサポートしてない場合、テスト失敗時のスクリーンショットの取得処理を行わないよう修正しています。


Merge pull request #28259 from ChakreshwarSharma/redirect_to_doc

actionpack/lib/action_controller/metal/redirecting.rbのdocの修正です。

redirect_toメソッドのdocに、redirect_toメソッドの後も処理が実行される旨説明を追加、また、即座に処理を終了したい場合は、returnを使う旨説明を追加しています。