diff options
| author | Sean Griffin <sean@thoughtbot.com> | 2014-12-26 16:10:47 -0700 |
|---|---|---|
| committer | Sean Griffin <sean@thoughtbot.com> | 2014-12-26 16:12:56 -0700 |
| commit | b98668decb9712f26118de57623fd15d7d28646d (patch) | |
| tree | 7051a3854ea8da9a357566deb14c4a0daadf9d7c /activerecord/test/cases/adapters/postgresql/explain_test.rb | |
| parent | 6d4a19cd503b39bdbefae7c1c527b24ad2e3a36a (diff) | |
| download | rails-b98668decb9712f26118de57623fd15d7d28646d.tar.gz rails-b98668decb9712f26118de57623fd15d7d28646d.tar.bz2 rails-b98668decb9712f26118de57623fd15d7d28646d.zip | |
Go through normal `where` logic when preloading associations
This will allow eager type casting to take place as needed. There
doesn't seem to be any particular reason that the `in` statement was
forced for single values, and the commit message where it was introduced
gives no context.
See
https://github.com/rails/rails/commit/d90b4e2615e8048fdeffc6dffe3246704adee01f
Diffstat (limited to 'activerecord/test/cases/adapters/postgresql/explain_test.rb')
| -rw-r--r-- | activerecord/test/cases/adapters/postgresql/explain_test.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/activerecord/test/cases/adapters/postgresql/explain_test.rb b/activerecord/test/cases/adapters/postgresql/explain_test.rb index d2dd04b84b..6ffb4c9f33 100644 --- a/activerecord/test/cases/adapters/postgresql/explain_test.rb +++ b/activerecord/test/cases/adapters/postgresql/explain_test.rb @@ -18,7 +18,7 @@ module ActiveRecord explain = Developer.where(:id => 1).includes(:audit_logs).explain assert_match %(QUERY PLAN), explain assert_match %(EXPLAIN for: SELECT "developers".* FROM "developers" WHERE "developers"."id" = $1), explain - assert_match %(EXPLAIN for: SELECT "audit_logs".* FROM "audit_logs" WHERE "audit_logs"."developer_id" IN (1)), explain + assert_match %(EXPLAIN for: SELECT "audit_logs".* FROM "audit_logs" WHERE "audit_logs"."developer_id" = 1), explain end end end |