diff options
| author | Aaron Patterson <aaron.patterson@gmail.com> | 2011-01-03 16:19:57 -0800 |
|---|---|---|
| committer | Aaron Patterson <aaron.patterson@gmail.com> | 2011-01-03 16:19:57 -0800 |
| commit | f5e09790550fcb7d413d495be25f8fe74a571850 (patch) | |
| tree | e69d675e1eec65a3c4dc3df5cf63350675bf633a /test/visitors | |
| parent | ef29263428eb2aa1fdc2b6b020f0d153fd17b5f3 (diff) | |
| download | rails-f5e09790550fcb7d413d495be25f8fe74a571850.tar.gz rails-f5e09790550fcb7d413d495be25f8fe74a571850.tar.bz2 rails-f5e09790550fcb7d413d495be25f8fe74a571850.zip | |
adding better tests surrounding limits in adapter visitors
Diffstat (limited to 'test/visitors')
| -rw-r--r-- | test/visitors/test_mysql.rb | 4 | ||||
| -rw-r--r-- | test/visitors/test_postgres.rb | 6 |
2 files changed, 6 insertions, 4 deletions
diff --git a/test/visitors/test_mysql.rb b/test/visitors/test_mysql.rb index 135348580d..c3b79ca667 100644 --- a/test/visitors/test_mysql.rb +++ b/test/visitors/test_mysql.rb @@ -19,8 +19,8 @@ module Arel it "should escape LIMIT" do sc = Arel::Nodes::UpdateStatement.new - sc.limit = "omg" - assert_match(/LIMIT 'omg'/, @visitor.accept(sc)) + sc.limit = Nodes::Limit.new("omg") + assert_equal("UPDATE NULL LIMIT 'omg'", @visitor.accept(sc)) end it 'uses DUAL for empty from' do diff --git a/test/visitors/test_postgres.rb b/test/visitors/test_postgres.rb index b98f78ca12..6e8f399b6b 100644 --- a/test/visitors/test_postgres.rb +++ b/test/visitors/test_postgres.rb @@ -15,10 +15,12 @@ module Arel it "should escape LIMIT" do sc = Arel::Nodes::SelectStatement.new - sc.limit = "omg" + sc.limit = Nodes::Limit.new("omg") sc.cores.first.projections << 'DISTINCT ON' sc.orders << "xyz" - assert_match(/LIMIT 'omg'/, @visitor.accept(sc)) + sql = @visitor.accept(sc) + assert_match(/LIMIT 'omg'/, sql) + assert_equal 1, sql.scan(/LIMIT/).length, 'should have one limit' end end end |