Permalink
Browse files

Merge pull request #3274 from arunagw/warning_removed_mysql2

ambiguous first argument; put parentheses or even spaces Warning removed!
  • Loading branch information...
2 parents b27ca99 + af7eeaa commit e9413b5de34615948b9092fe6976272a36cc2fef @vijaydev vijaydev committed Oct 10, 2011
Showing with 2 additions and 2 deletions.
  1. +2 −2 activerecord/test/cases/adapters/mysql2/case_sensitivity_test.rb
@@ -21,15 +21,15 @@ def test_case_insensitive_comparison_for_ci_column
CollationTest.create!(:string_ci_column => 'A')
invalid = CollationTest.new(:string_ci_column => 'a')
queries = assert_sql { invalid.save }
- ci_uniqueness_query = queries.detect { |q| q.match /string_ci_column/ }
+ ci_uniqueness_query = queries.detect { |q| q.match(/string_ci_column/) }
assert_no_match(/lower/i, ci_uniqueness_query)
end
def test_case_insensitive_comparison_for_cs_column
CollationTest.create!(:string_cs_column => 'A')
invalid = CollationTest.new(:string_cs_column => 'a')
queries = assert_sql { invalid.save }
- cs_uniqueness_query = queries.detect { |q| q.match /string_cs_column/ }
+ cs_uniqueness_query = queries.detect { |q| q.match(/string_cs_column/)}
assert_match(/lower/i, cs_uniqueness_query)
end
end

0 comments on commit e9413b5

Please sign in to comment.