diff options
author | Marcelo Silveira <marcelo@mhfs.com.br> | 2012-05-05 12:07:20 -0300 |
---|---|---|
committer | Marcelo Silveira <marcelo@mhfs.com.br> | 2012-05-05 12:10:52 -0300 |
commit | 56bf1f74557e68455552eeac1bc975cf9ba57766 (patch) | |
tree | 5f3f5013aba1129a035b074a49e17fd59f3fe25b /activerecord/test/cases/finder_test.rb | |
parent | acb39848ae4cfe1d22cd8a83c5db636d80c22b47 (diff) | |
download | rails-56bf1f74557e68455552eeac1bc975cf9ba57766.tar.gz rails-56bf1f74557e68455552eeac1bc975cf9ba57766.tar.bz2 rails-56bf1f74557e68455552eeac1bc975cf9ba57766.zip |
Use `take` instead of `first` to avoid unwanted implicit ordering (fixes #6147)
Diffstat (limited to 'activerecord/test/cases/finder_test.rb')
-rw-r--r-- | activerecord/test/cases/finder_test.rb | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/activerecord/test/cases/finder_test.rb b/activerecord/test/cases/finder_test.rb index 54801bd101..630acdbc46 100644 --- a/activerecord/test/cases/finder_test.rb +++ b/activerecord/test/cases/finder_test.rb @@ -113,6 +113,10 @@ class FinderTest < ActiveRecord::TestCase assert_equal [], Topic.find([]) end + def test_find_doesnt_have_implicit_ordering + assert_sql(/^((?!ORDER).)*$/) { Topic.find(1) } + end + def test_find_by_ids_missing_one assert_raise(ActiveRecord::RecordNotFound) { Topic.find(1, 2, 45) } end |