diff options
author | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2012-06-10 18:03:41 -0700 |
---|---|---|
committer | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2012-06-10 18:03:41 -0700 |
commit | f48e7676e9b70884c309d3170b7ae8867e4bd252 (patch) | |
tree | fc9bfb567c383f4e48314a13aee62e03b08f63c3 /activerecord/test | |
parent | d17fa452ec1993271ccc6e5035fb27b9c89513cf (diff) | |
parent | 340a93f5026ffb8e877024c23791e8a345605976 (diff) | |
download | rails-f48e7676e9b70884c309d3170b7ae8867e4bd252.tar.gz rails-f48e7676e9b70884c309d3170b7ae8867e4bd252.tar.bz2 rails-f48e7676e9b70884c309d3170b7ae8867e4bd252.zip |
Merge pull request #6521 from Empact/throw-result
Fix that #exists? can blow up with ThrowResult exception
Diffstat (limited to 'activerecord/test')
-rw-r--r-- | activerecord/test/cases/finder_test.rb | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/activerecord/test/cases/finder_test.rb b/activerecord/test/cases/finder_test.rb index 78da8778f0..0914d15c5e 100644 --- a/activerecord/test/cases/finder_test.rb +++ b/activerecord/test/cases/finder_test.rb @@ -69,7 +69,12 @@ class FinderTest < ActiveRecord::TestCase assert Topic.order(:id).uniq.exists? end - def test_does_not_exist_with_empty_table_and_no_args_given + def test_exists_with_includes_limit_and_empty_result + assert !Topic.includes(:replies).limit(0).exists? + assert !Topic.includes(:replies).limit(1).where('0 = 1').exists? + end + + def test_exists_with_empty_table_and_no_args_given Topic.delete_all assert !Topic.exists? end |