rails commit log流し読み(2014/08/13) - なるようになるブログ

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

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

index_exists? with :name checks specified columns.

[ci skip] Updated where scope to conform to new style

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

サンプルのwhereメソッドの書き方を修正しています。

-      #   belongs_to :valid_coupon, ->(o) { where "discounts > #{o.payments_count}" },
+      #   belongs_to :valid_coupon, ->(o) { where "discounts > ?", o.payments_count },

index_exists? with :name checks specified columns.

activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rbの修正です。

index_exists?メソッドがカラム名のチェックも行うよう修正しています。

# Before:
index_exists? :articles, :title, name: "idx_title" # => `true`
index_exists? :articles, :body, name: "idx_title" # => `true`

# After:
index_exists? :articles, :title, name: "idx_title" # => `true`
index_exists? :articles, :body, name: "idx_title" # => `false`

より正確にチェックされるようになったのですが、微妙に非互換なので、ちょっと注意が必要そうな。

[ci skip] fix spelling of override

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

overridenをoverriddenに修正しています。typoの修正ですね。

AM, AP, AV, and AMo tests are already order_independent!

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

ActionMailer、ActionPack、ActionView、ActiveModelについては、テストが実行順序に依存してない、との事なので、ActiveSupport::TestCase.my_tests_are_order_dependent!の呼び出しを削除しています。

Duplicated method in the test helper

activesupport/test/dependencies_test.rbの修正です。

重複しているメソッドを削除しています。

Reset ActiveSupport::Dependencies.mechanism to make tests order independent

activesupport/test/dependencies_test.rbの修正です。

実行順序に依存してしまっているテストがあったのを修正しています。

with_loadingというテスト用のヘルパーメソッドがあるのですが、それの呼び出しが足りて無かったようです。

AS tests are now order_independent!

activesupport/test/abstract_unit.rbの修正です。

上記対応に伴い、ActiveSupportもテストが実行順序に依存しなくなったので、ActiveSupport::TestCase.my_tests_are_order_dependent!を削除しています。

Fix Railties tests that were order dependent

railties/test/generators_test.rbの修正です。