Skip to content

Commit

Permalink
Fix test: this should return 0 not 7, since Arel was ignoring select …
Browse files Browse the repository at this point in the history
…clause and now it's fixed.

Signed-off-by: José Valim <jose.valim@gmail.com>
  • Loading branch information
miloops authored and josevalim committed Sep 1, 2010
1 parent 0ae81a2 commit 2ce57fd
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion activerecord/test/cases/relations_test.rb
Expand Up @@ -521,7 +521,7 @@ def test_count_explicit_columns
posts = Post.scoped

assert_equal [0], posts.select('comments_count').where('id is not null').group('id').order('id').count.values.uniq
assert_equal 7, posts.where('id is not null').select('comments_count').count
assert_equal 0, posts.where('id is not null').select('comments_count').count

assert_equal 7, posts.select('comments_count').count('id')
assert_equal 0, posts.select('comments_count').count
Expand Down

0 comments on commit 2ce57fd

Please sign in to comment.