なるようになるブログ

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

rails commit log流し読み(2018/01/19)

2018/01/19分のコミットです。

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

activesupport/CHANGELOG.md


Fix ActionView UJS build

ci/qunit-selenium-runner.rbの修正です。

CI上でUJSのテスト用にChromeを起動する際に、--no-sandboxオプションを指定するよう修正しています。

Travis CIのバグで、sanboxオプションを指定するとChromeがcrashする、という問題がある為。

参考:Regression: Chrome fails to start due to incorrect default chrome-sandbox permissions


Use selected_migrations if block_given?

activerecord/lib/active_record/migration.rbの修正です。

MigrationContext#downメソッドで使用するメソッドに誤りがあったのを修正しています。


Add test to properly test down with a block

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

migrateのdownメソッドをblockを指定して呼び出した場合のテストを追加しています。


Merge pull request #31730 from bogdanvlviv/allow_false_for-config-generators-system_tests

railties/lib/rails/generators/test_unit/scaffold/scaffold_generator.rbの修正です。

config.generators.system_testsにfalseを指定した場合もSystem Testを生成しないよう修正しています。元々falseを指定した場合も生成されてしまっていた(生成したく無い場合はnilを指定する必要があった)のですが、他のgeneratorの挙動を合わせる為、falseでも生成しないよう修正しています。


Merge pull request #31577 from prathamesh-sonpatki/upgrade-guide-bootsnap

rails guideのUpgrading Ruby on Railsの修正です。

Upgrading from Rails 5.1 to Rails 5.2の項を追加し、bootsnap gemについての説明を追加しています。


Support for connection pooling on mem cache store

activesupport/lib/active_support/cache/mem_cache_store.rbの修正です。

mem cache storeにconnection poolingのサポートを追加しています。

pool_sizeオプションを指定した場合のみconnection poolが使用されるようになっています。connection pool部分の実装はmperham/connection_pool gemを使用しています。


Support hash as first argument in assert_difference. (#31600)

activesupport/lib/active_support/testing/assertions.rbの修正です。

assert_differenceの第一引数にHashを指定出来るよう修正しています。これにより、一つのassertionで複数要素の値のチェックが出来るようになります。

assert_difference ->{ Article.count } => 1, ->{ Notification.count } => 2 do
  post :create, params: { article: {...} }
end

Add locale selector to email preview (#31596)

railties/lib/rails/mailers_controller.rbrailties/lib/rails/templates/rails/mailers/email.html.erbの修正です。

Action Mailerのemail previewでlocaleを選択出来るよう対応しています。

PRより。

34454066-f8bf06ec-eda5-11e7-82ba-1c2a0961b6b8.gif (833×768)


Merge pull request #31663 from Edouard-chin/remove-without-sql-mode

activerecord/test/cases/fixtures_test.rbの修正です。

NO_AUTO_VALUE_ON_ZERO modeが無効になっている場合にfixtureのinsertが正しく動作する事を確認するテストを追加しています。


Testing actioncable against websocket-driver 0.7.0 (#30711)

actioncable/actioncable.gemspecの修正です。

websocket-driver gemのバージョンが0.6系で固定されてしまっていたのを、0.6.1以上を使用出来るよう修正しています。websocket-driver gemの0.7.0がリリース済みで、そちらを使用出来るようにする為。


Update Gemfile.lock to reflect websocket-driver 0.7.0

Gemfile.lockの修正です。

先のactioncable.gemspecの更新の際に、Gemfile.lockの更新が漏れていたのを修正しています。


Fix minor typo [ci skip]

rails guideのActive Record Associationsの修正です。

association(belongs_tohas_many等)により追加されるメソッドの数が誤っていたのを修正しています。


Restore test_migrations_status_with_schema_define_in_subdirectories

activerecord/test/cases/migrator_test.rbの修正です。

Merge pull request #31727 from eileencodes/refactor-migration-classes-to-allow-for-migrations_paths-on-connの修正で、サブディレクトリに置かれたmigirationファイルが正しくロードされる事を確認するテストが、本来の意図と異なる形に修正されてしまったのを、再度元の意図通りにテストを修正しています。


Remove unused migration_context in DatabaseTasks

activerecord/lib/active_record/tasks/database_tasks.rbの修正です。

使用していないDatabaseTasks#migration_contextメソッドを削除しています。


Remove unused connection argument from MigrationContext#current_version

activerecord/lib/active_record/migration.rbの修正です。

MigrationContext#current_versionメソッドから、使用していないconnection引数を削除しています。


Merge pull request #31732 from koic/enable_autocorrect_for_lint_end_alignment_cop

Lint/EndAlignment copのautocorrectを有効化、及び、各ファイルにcopを適応しています。