From 99cf7558000090668b137085bfe6bcc06c4571dc Mon Sep 17 00:00:00 2001 From: Xavier Noria Date: Sat, 23 Jul 2016 20:01:56 +0200 Subject: systematic revision of =~ usage in AR Where appropriatei, prefer the more concise Regexp#match?, String#include?, String#start_with?, or String#end_with? --- activerecord/test/cases/scoping/default_scoping_test.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'activerecord/test/cases/scoping/default_scoping_test.rb') diff --git a/activerecord/test/cases/scoping/default_scoping_test.rb b/activerecord/test/cases/scoping/default_scoping_test.rb index dcd09b6973..30d342ac70 100644 --- a/activerecord/test/cases/scoping/default_scoping_test.rb +++ b/activerecord/test/cases/scoping/default_scoping_test.rb @@ -201,7 +201,7 @@ class DefaultScopingTest < ActiveRecord::TestCase def test_order_to_unscope_reordering scope = DeveloperOrderedBySalary.order('salary DESC, name ASC').reverse_order.unscope(:order) - assert !(scope.to_sql =~ /order/i) + assert !/order/i.match?(scope.to_sql) end def test_unscope_reverse_order -- cgit v1.2.3