2017/01/25分のコミットです。
CHANGELOGにのったコミットは以下の通りです。
Keep the value of $, and restore it
actionview/test/template/output_safety_helper_test.rbの修正です。
to_sentenceメソッドで$,の値を変化させているのですが、テストの後処理で元の値に戻すよう修正しています。
Merge pull request #27772 from y-yagi/remove_unreachable_code
railties/lib/rails/generators/rails/app/app_generator.rb、
railties/lib/rails/generators/rails/plugin/plugin_generator.rbの修正です。
AppGenerator.new及びPluginGenerator.newメソッドからapp_pathがnilだった場合のチェック処理を削除しています。
app_pathはThorの必須パラメータとして定義されている、かつ、app_pathのnilチェックより前にThorのパラメータ解析処理が行われており、app_pathがnilの場合はそのパラメータ解析処理でエラーになりのチェックの時点でapp_pathがnilになっている事は無い為。
Give a message to #test_duplicable assertion
activesupport/test/core_ext/object/duplicable_test.rbの修正です。
duplicable?メソッドのテストで、assertionがエラーになった場合にテスト対象の値の詳細を表示するよう修正しています。
Change ActionView ERB Handler from Erubis to Erubi
Action Viewの修正です。
ERB Handlerで使用するgemをErubisからErubiに変更しています。
ErubiはErubisのフォークなのですが、現在もメンテされている、かつ、--enable-frozen-string-literalをサポートしており、性能的にErubisよりも良いため変更したとの事です。機能互換はある為、基本的にユーザ側は何も変更しなくて問題無いようになっています(一部対応していない機能あり)。また、Erubis handlerはdeprecateになり5.2で削除予定です。
This seems to be working on JRuby 9K
activesupport/test/inflector_test.rbの修正です。
parameterizeメソッドのテストがJRubyでskipされていたのを、JRubyでもテスト行うようskipを削除しています。
JRubyでUTF-8からUTF8-MACへの変換が出来なかった為skipされていたのですが、現状問題なく変換処理が動作する為、テストを行うようにしたとの事です。
activerecord/test/cases/adapters/postgresql/infinity_test.rb、
activerecord/test/cases/adapters/postgresql/numbers_test.rbの修正です。
assert_sendを使用していたのをassertを使用するよう修正した箇所で、assertion失敗時に詳細なエラーメッセージを表示するよう修正しています。
Fix grammar ‘an hyphen’ -> ‘a hyphen’ [ci skip]
actionview/lib/action_view/helpers/number_helper.rb、
activesupport/lib/active_support/number_helper.rbメソッドのdocの修正です。
number_to_phoneメソッドのdoc内のグラマーの修正を行っています。
