なるようになるブログ

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

rails commit log流し読み(2016/05/29)

2016/05/29分のコミットです。

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


Fix assertion count after 69e1bb52.

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

デフォルトで生成されるcontrollers用のテストファイルで、redirectの確認をする際にxxx_urlを使用するよう修正した対応(Redirects are to urls, not paths)の影響で壊れてしまっていたテストがあったのを修正しています。

壊れてしまったテストは、scaffoldで生成したテストが正常に動作する事を確認するテストなのですが、assert_redirected_toの引数をxxx_urlに変える事により実行されるassertionの数が変わってしまった事により壊れてしまっていました(テストが正常に動作する事を確認するのに、assertionsの数を見ていた)。

で、何故assertionsの数が変わってしまったかというと、assert_redirected_toの中では引数と@response.locationの比較処理を行っており、一致したらチェック終了、一致しなかった場合は引数からURLを生成し、assert_operatorでその生成した値が一致するかチェックを行っています。xxx_pathを使ってた際は、@response.locationと値が一致しない為、assert_operatorでのチェック処理も行われていたのですが、xxx_urlを使用する事により@response.locationと値が一致する為、assert_operatorによりチェック処理が行われなく成った為、assertionsの数が変わってしまったようです。へー。


Fix more assertion counts.

railties/test/generators/scaffold_generator_test.rbの修正です。

先のコミット同様、デフォルトで生成されるcontrollers用のテストファイルで、redirectの確認をする際にxxx_urlを使用するよう修正した対応(Redirects are to urls, not paths)の影響で壊れてしまっていたテストがあったのを修正しています。


rename enum_prefix and enum_suffix options to prefix and suffix [ci skip]

rails guideのRuby on Rails 5.0 Release Notesの修正です。

enumに追加されたオプション名が間違えていた(:enum_prefix:enum_suffixになっていたが、正しくは:_prefix:_suffix)のを修正しています。


ActionController::Parameters: Fix #dig doc code

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

ActionController::Parameters#digメソッドのdocで、exampleコードが<code>タグに囲まれるよう、インデントを追加しています。


[ci skip]Fix wrong require path raising LoadError

rails guideのAction Cable Overviewの修正です。

Standaloneの項の、Action CableをStandaloneで起動する為のサンプルスクリプトで、config/environmentへのパスが誤っていたのを修正してます。