Skip to content

Commit

Permalink
do not rely on method_missing hitting arel
Browse files Browse the repository at this point in the history
arel methods are not supposed to be public API for ActiveRecord
  • Loading branch information
tenderlove committed Mar 25, 2014
1 parent c7191e3 commit f72058a
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions activerecord/test/cases/scoping/relation_scoping_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -192,8 +192,9 @@ def test_merge_options
Developer.where('salary = 80000').scoping do
Developer.limit(10).scoping do
devs = Developer.all
assert_match '(salary = 80000)', devs.to_sql
assert_equal 10, devs.taken
sql = devs.to_sql
assert_match '(salary = 80000)', sql
assert_match 'LIMIT 10', sql
end
end
end
Expand Down

0 comments on commit f72058a

Please sign in to comment.