use parenthesis so limit works on all dbs

larssmit · Feb 9, 2011 · d3b2596 · d3b2596
1 parent cb1f841
commit d3b2596
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/activerecord/test/cases/base_test.rb b/activerecord/test/cases/base_test.rb
@@ -94,13 +94,11 @@ def test_limit_should_sanitize_sql_injection_for_limit_with_comas
       Topic.limit("1, 7 procedure help()").all
     end
   end
-
-  unless current_adapter?(:MysqlAdapter)
-    def test_limit_should_allow_sql_literal
-      assert_equal 1, Topic.limit(Arel.sql('2-1')).all.length
-    end
+
+  def test_limit_should_allow_sql_literal
+    assert_equal 1, Topic.limit(Arel.sql('(2 - 1)')).all.length
   end
-  
+
   def test_select_symbol
     topic_ids = Topic.select(:id).map(&:id).sort
     assert_equal Topic.find(:all).map(&:id).sort, topic_ids