なるようになるブログ

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

rails commit log流し読み(2014/08/12)

2014/08/12分のコミットです。

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

activesupport/CHANGELOG.md


[ci skip] Fix broken url in plugins guide

rails guideのThe Basics of Creating Rails Pluginsの修正です。

Gemspec Referenceのリンク先を修正しています。


users_dont_suck_but_only_we_suck_and_only_our_tests_are_order_dependent!

各種テストファイルの修正です。

activesupport/lib/active_support/test_case.rbi_suck_and_my_tests_are_order_dependentを呼んでいたのを削除して、各モジュール配下でi_suck_and_my_tests_are_order_dependentを呼び出すよう修正しています。

minitestは5.3.4からテストがランダムで実行されるよう対応されているのですが、ランダムで実行される事を想定していない、つまり、ファイルに記載されている通りにテストを実行して欲しい場合、i_suck_and_my_tests_are_order_dependent!というメソッドを呼び出す事で、順番通りに実行されます。

railsのテスト、現状ランダムでは動かないとの事で、i_suck_and_my_tests_are_order_dependentを呼び出していたのですが、ActiveSupport::TestCaseではユーザのテストにも影響が出てしまうので、削除したのかと。

因みに、影響があるのはrailsデフォルトのテストを使用している場合ですね。RSpecを使用している場合は、関係無いはず。


Remove old setup from AS test case

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

古いセットアップ処理を削除しています。


CHANGELOG entry for TestCase.i_suck_and_my_tests_are_order_dependent! fix

先のi_suck_and_my_tests_are_order_dependentの対応についてCHANGELOGに追加してます。


use 'based on' instead of 'based off' [ci skip]

actionpack/lib/action_dispatch/http/request.rbactionpack/lib/action_dispatch/middleware/request_id.rbのdocの修正です。

based offbased onに修正しています。


Nobody sucks so nobody should call this awful method name

各種テストの修正です。

誰もsuckでは無い、という事でalias :my_tests_are_order_dependent! :i_suck_and_my_tests_are_order_dependent!を設定して、各テストファイルではmy_tests_are_order_dependentメソッドを呼び出すよう修正していない。

Nobody sucks、というのは良いですね。


:bomb:

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

先ほどのmy_tests_are_order_dependenttypoがあったのを修正しています。