なるようになるブログ

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

rails commit log流し読み(2019/07/18)

2019/07/18分のコミットです。

CHANGELOGへの追加はありませんでした。


Revert "Make UTF-8 string requirement explicit for ActiveSupport::Inflector.transliterate"

ActiveSupport::Inflector.transliterateメソッドの先頭で引数に指定されたStringがUTF-8じゃない場合ArgumentErrorをraiseするよう修正した、Merge pull request #36690 from cpruitt/make-parameterize-requires-utf-8-explicitをRevertしています。

元々UTF-8以外のStringを指定した場合エラーになってしまう為上記対応を行ったのですが、UTF-8以外のStringを指定した場合エラーになる、が勘違いだった為Revertしています。


Merge pull request #36691 from Edouard-chin/ec-system-test-route

actionpack/lib/action_dispatch/railtie.rbactionpack/lib/action_dispatch/system_test_case.rbの修正です。

System test classでroute helperを直接includeしないよう修正しています。

Permit running jobs in system testsでroute helperをincludeするようになったのですが、route helperにtest_ではじまるrouteがあった場合に、それがテスト用のメソッドとminitestに判定されてしまい、結果エラーになる、という問題があった為includeしないよう修正しています。

代わりにroute helperをincludeしたクラスをSystemTestCaseクラス内で保持 + method_missingでそのクラスで使用し、route helperメソッドを使用出来るよう対応しています。


Merge pull request #36603 from y-yagi/add_skip_collision_check_option

railties/lib/rails/generators/base.rbの修正です。

generatorに、既に存在しているクラスと同じ名前かどうかをチェックする処理をスキップする為の、skip_collision_checkオプションを追加しています。


Merge pull request #36695 from kamipo/revert_unnecessary_change

activestorage/test/analyzer/video_analyzer_test.rbの修正です。

durationのチェック処理を修正したMerge pull request #36676 from wjessop/change_activestorage_metadata_duration_to_boundをRevertしています。そもそも不要な対応だった為。