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/mysql2 | |
| 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/mysql2')
| -rw-r--r-- | activerecord/test/cases/adapters/mysql2/explain_test.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/activerecord/test/cases/adapters/mysql2/explain_test.rb b/activerecord/test/cases/adapters/mysql2/explain_test.rb index f67f21fab1..2b01d941b8 100644 --- a/activerecord/test/cases/adapters/mysql2/explain_test.rb +++ b/activerecord/test/cases/adapters/mysql2/explain_test.rb @@ -18,7 +18,7 @@ module ActiveRecord explain = Developer.where(:id => 1).includes(:audit_logs).explain assert_match %(EXPLAIN for: SELECT `developers`.* FROM `developers` WHERE `developers`.`id` = 1), explain assert_match %r(developers |.* const), 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 assert_match %r(audit_logs |.* ALL), explain end end |