search_cop 1.1.0 → 1.2.2

data/lib/search_cop_grammar/attributes.rb

@@ -1,4 +1,3 @@
-
 require "treetop"
 
 module SearchCopGrammar
@@ -28,9 +27,9 @@ module SearchCopGrammar
       end
 
       [:eq, :not_eq, :lt, :lteq, :gt, :gteq].each do |method|
-         define_method method do |value|
-           attributes.collect! { |attribute| attribute.send method, value }.inject(:or)
-         end
+        define_method method do |value|
+          attributes.collect! { |attribute| attribute.send method, value }.inject(:or)
+        end
       end
 
       def generator(generator, value)
@@ -137,7 +136,7 @@ module SearchCopGrammar
         false
       end
 
-      { :eq => "Equality", :not_eq => "NotEqual", :lt => "LessThan", :lteq => "LessThanOrEqual", :gt => "GreaterThan", :gteq => "GreaterThanOrEqual", :matches => "Matches" }.each do |method, class_name|
+      { eq: "Equality", not_eq: "NotEqual", lt: "LessThan", lteq: "LessThanOrEqual", gt: "GreaterThan", gteq: "GreaterThanOrEqual", matches: "Matches" }.each do |method, class_name|
         define_method method do |value|
           raise(SearchCop::IncompatibleDatatype, "Incompatible datatype for #{value}") unless compatible?(value)
 
@@ -146,19 +145,32 @@ module SearchCopGrammar
       end
 
       def method_missing(name, *args, &block)
-        @attribute.send(name, *args, &block)
+        if @attribute.respond_to?(name)
+          @attribute.send(name, *args, &block)
+        else
+          super
+        end
       end
 
-      def respond_to?(*args)
-        super(*args) || @attribute.respond_to?(*args)
+      def respond_to_missing?(*args)
+        @attribute.respond_to?(*args) || super
       end
     end
 
     class String < Base
       def matches_value(value)
-        return value.gsub(/\*/, "%") if (options[:left_wildcard] != false && value.strip =~ /^[^*]+\*$|^\*[^*]+$/) || value.strip =~ /^[^*]+\*$/
+        res = value.gsub(/[%_\\]/) { |char| "\\#{char}" }
 
-        options[:left_wildcard] != false ? "%#{value}%" : "#{value}%"
+        if value.strip =~ /^\*|\*$/
+          res = res.gsub(/^\*/, "%") if options[:left_wildcard] != false
+          res = res.gsub(/\*$/, "%") if options[:right_wildcard] != false
+
+          return res
+        end
+
+        res = "%#{res}" if options[:left_wildcard] != false
+        res = "#{res}%" if options[:right_wildcard] != false
+        res
       end
 
       def matches(value)
@@ -176,7 +188,7 @@ module SearchCopGrammar
 
     class Float < WithoutMatches
       def compatible?(value)
-        return true if value.to_s =~ /^\-?[0-9]+(\.[0-9]+)?$/
+        return true if value.to_s =~ /^-?[0-9]+(\.[0-9]+)?$/
 
         false
       end
@@ -196,24 +208,24 @@ module SearchCopGrammar
 
     class Datetime < WithoutMatches
       def parse(value)
-        return value .. value unless value.is_a?(::String)
+        return value..value unless value.is_a?(::String)
 
         if value =~ /^[0-9]+ (hour|day|week|month|year)s{0,1} (ago)$/
-          number,period,ago = value.split(' ')
+          number, period, ago = value.split(" ")
           time = number.to_i.send(period.to_sym).send(ago.to_sym)
-          time .. ::Time.now
+          time..::Time.now
         elsif value =~ /^[0-9]{4}$/
-          ::Time.new(value).beginning_of_year .. ::Time.new(value).end_of_year
-        elsif value =~ /^([0-9]{4})(\.|-|\/)([0-9]{1,2})$/
-          ::Time.new($1, $3, 15).beginning_of_month .. ::Time.new($1, $3, 15).end_of_month
-        elsif value =~ /^([0-9]{1,2})(\.|-|\/)([0-9]{4})$/
-          ::Time.new($3, $1, 15).beginning_of_month .. ::Time.new($3, $1, 15).end_of_month
-        elsif value =~ /^[0-9]{4}(\.|-|\/)[0-9]{1,2}(\.|-|\/)[0-9]{1,2}$/ || value =~ /^[0-9]{1,2}(\.|-|\/)[0-9]{1,2}(\.|-|\/)[0-9]{4}$/
+          ::Time.new(value).beginning_of_year..::Time.new(value).end_of_year
+        elsif value =~ %r{^([0-9]{4})(\.|-|/)([0-9]{1,2})$}
+          ::Time.new(Regexp.last_match(1), Regexp.last_match(3), 15).beginning_of_month..::Time.new(Regexp.last_match(1), Regexp.last_match(3), 15).end_of_month
+        elsif value =~ %r{^([0-9]{1,2})(\.|-|/)([0-9]{4})$}
+          ::Time.new(Regexp.last_match(3), Regexp.last_match(1), 15).beginning_of_month..::Time.new(Regexp.last_match(3), Regexp.last_match(1), 15).end_of_month
+        elsif value =~ %r{^[0-9]{4}(\.|-|/)[0-9]{1,2}(\.|-|/)[0-9]{1,2}$} || value =~ %r{^[0-9]{1,2}(\.|-|/)[0-9]{1,2}(\.|-|/)[0-9]{4}$}
           time = ::Time.parse(value)
-          time.beginning_of_day .. time.end_of_day
-        elsif value =~ /[0-9]{4}(\.|-|\/)[0-9]{1,2}(\.|-|\/)[0-9]{1,2}/ || value =~ /[0-9]{1,2}(\.|-|\/)[0-9]{1,2}(\.|-|\/)[0-9]{4}/
+          time.beginning_of_day..time.end_of_day
+        elsif value =~ %r{[0-9]{4}(\.|-|/)[0-9]{1,2}(\.|-|/)[0-9]{1,2}} || value =~ %r{[0-9]{1,2}(\.|-|/)[0-9]{1,2}(\.|-|/)[0-9]{4}}
           time = ::Time.parse(value)
-          time .. time
+          time..time
         else
           raise ArgumentError
         end
@@ -246,21 +258,21 @@ module SearchCopGrammar
 
     class Date < Datetime
       def parse(value)
-        return value .. value unless value.is_a?(::String)
+        return value..value unless value.is_a?(::String)
 
         if value =~ /^[0-9]+ (day|week|month|year)s{0,1} (ago)$/
-          number,period,ago = value.split(' ')
+          number, period, ago = value.split(" ")
           time = number.to_i.send(period.to_sym).send(ago.to_sym)
-          time.to_date .. ::Date.today
+          time.to_date..::Date.today
         elsif value =~ /^[0-9]{4}$/
-          ::Date.new(value.to_i).beginning_of_year .. ::Date.new(value.to_i).end_of_year
-        elsif value =~ /^([0-9]{4})(\.|-|\/)([0-9]{1,2})$/
-          ::Date.new($1.to_i, $3.to_i, 15).beginning_of_month .. ::Date.new($1.to_i, $3.to_i, 15).end_of_month
-        elsif value =~ /^([0-9]{1,2})(\.|-|\/)([0-9]{4})$/
-          ::Date.new($3.to_i, $1.to_i, 15).beginning_of_month .. ::Date.new($3.to_i, $1.to_i, 15).end_of_month
-        elsif value =~ /[0-9]{4}(\.|-|\/)[0-9]{1,2}(\.|-|\/)[0-9]{1,2}/ || value =~ /[0-9]{1,2}(\.|-|\/)[0-9]{1,2}(\.|-|\/)[0-9]{4}/
+          ::Date.new(value.to_i).beginning_of_year..::Date.new(value.to_i).end_of_year
+        elsif value =~ %r{^([0-9]{4})(\.|-|/)([0-9]{1,2})$}
+          ::Date.new(Regexp.last_match(1).to_i, Regexp.last_match(3).to_i, 15).beginning_of_month..::Date.new(Regexp.last_match(1).to_i, Regexp.last_match(3).to_i, 15).end_of_month
+        elsif value =~ %r{^([0-9]{1,2})(\.|-|/)([0-9]{4})$}
+          ::Date.new(Regexp.last_match(3).to_i, Regexp.last_match(1).to_i, 15).beginning_of_month..::Date.new(Regexp.last_match(3).to_i, Regexp.last_match(1).to_i, 15).end_of_month
+        elsif value =~ %r{[0-9]{4}(\.|-|/)[0-9]{1,2}(\.|-|/)[0-9]{1,2}} || value =~ %r{[0-9]{1,2}(\.|-|/)[0-9]{1,2}(\.|-|/)[0-9]{4}}
           date = ::Date.parse(value)
-          date .. date
+          date..date
         else
           raise ArgumentError
         end
@@ -281,4 +293,3 @@ module SearchCopGrammar
     end
   end
 end
-

data/lib/search_cop_grammar/nodes.rb

@@ -1,4 +1,3 @@
-
 require "treetop"
 
 module SearchCopGrammar
@@ -187,4 +186,3 @@ module SearchCopGrammar
     end
   end
 end
-

data/lib/search_cop_grammar.rb

@@ -1,4 +1,3 @@
-
 require "search_cop_grammar/attributes"
 require "search_cop_grammar/nodes"
 
@@ -19,7 +18,7 @@ module SearchCopGrammar
     end
 
     def elements
-      super.select { |element| element.class != Treetop::Runtime::SyntaxNode }
+      super.reject { |element| element.instance_of?(Treetop::Runtime::SyntaxNode) }
     end
 
     def collection_for(key)
@@ -153,4 +152,3 @@ module SearchCopGrammar
 
   class Value < BaseNode; end
 end
-

data/search_cop.gemspec

@@ -1,28 +1,27 @@
-# coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path("lib", __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'search_cop/version'
+require "search_cop/version"
 
 Gem::Specification.new do |spec|
   spec.name          = "search_cop"
   spec.version       = SearchCop::VERSION
   spec.authors       = ["Benjamin Vetter"]
   spec.email         = ["vetter@flakks.com"]
-  spec.description   = %q{Search engine like fulltext query support for ActiveRecord}
-  spec.summary       = %q{Easily perform complex search engine like fulltext queries on your ActiveRecord models}
+  spec.description   = "Search engine like fulltext query support for ActiveRecord"
+  spec.summary       = "Easily perform complex search engine like fulltext queries on your ActiveRecord models"
   spec.homepage      = "https://github.com/mrkamel/search_cop"
   spec.license       = "MIT"
 
-  spec.files         = `git ls-files`.split($/)
+  spec.files         = `git ls-files`.split($INPUT_RECORD_SEPARATOR)
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
-  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
   spec.add_dependency "treetop"
 
-  spec.add_development_dependency "bundler"
-  spec.add_development_dependency "rake"
   spec.add_development_dependency "activerecord", ">= 3.0.0"
+  spec.add_development_dependency "bundler"
   spec.add_development_dependency "factory_bot"
   spec.add_development_dependency "minitest"
+  spec.add_development_dependency "rake"
+  spec.add_development_dependency "rubocop"
 end

data/test/and_test.rb

@@ -1,10 +1,9 @@
-
-require File.expand_path("../test_helper", __FILE__)
+require File.expand_path("test_helper", __dir__)
 
 class AndTest < SearchCop::TestCase
   def test_and_string
-    expected = create(:product, :title => "expected title", :description => "Description")
-    rejected = create(:product, :title => "Rejected title", :description => "Description")
+    expected = create(:product, title: "expected title", description: "Description")
+    rejected = create(:product, title: "Rejected title", description: "Description")
 
     results = Product.search("title: 'Expected title' description: Description")
 
@@ -15,13 +14,12 @@ class AndTest < SearchCop::TestCase
   end
 
   def test_and_hash
-    expected = create(:product, :title => "Expected title", :description => "Description")
-    rejected = create(:product, :title => "Rejected title", :description => "Description")
+    expected = create(:product, title: "Expected title", description: "Description")
+    rejected = create(:product, title: "Rejected title", description: "Description")
 
-    results = Product.search(:and => [{:title => "Expected title"}, {:description => "Description"}])
+    results = Product.search(and: [{ title: "Expected title" }, { description: "Description" }])
 
     assert_includes results, expected
     refute_includes results, rejected
   end
 end
-

data/test/boolean_test.rb

@@ -1,15 +1,14 @@
-
-require File.expand_path("../test_helper", __FILE__)
+require File.expand_path("test_helper", __dir__)
 
 class BooleanTest < SearchCop::TestCase
   def test_mapping
-    product = create(:product, :available => true)
+    product = create(:product, available: true)
 
     assert_includes Product.search("available: 1"), product
     assert_includes Product.search("available: true"), product
     assert_includes Product.search("available: yes"), product
 
-    product = create(:product, :available => false)
+    product = create(:product, available: false)
 
     assert_includes Product.search("available: 0"), product
     assert_includes Product.search("available: false"), product
@@ -17,28 +16,28 @@ class BooleanTest < SearchCop::TestCase
   end
 
   def test_anywhere
-    product = create(:product, :available => true)
+    product = create(:product, available: true)
 
     assert_includes Product.search("true"), product
     refute_includes Product.search("false"), product
   end
 
   def test_includes
-    product = create(:product, :available => true)
+    product = create(:product, available: true)
 
     assert_includes Product.search("available: true"), product
     refute_includes Product.search("available: false"), product
   end
 
   def test_equals
-    product = create(:product, :available => true)
+    product = create(:product, available: true)
 
     assert_includes Product.search("available = true"), product
     refute_includes Product.search("available = false"), product
   end
 
   def test_equals_not
-    product = create(:product, :available => false)
+    product = create(:product, available: false)
 
     assert_includes Product.search("available != true"), product
     refute_includes Product.search("available != false"), product
@@ -50,4 +49,3 @@ class BooleanTest < SearchCop::TestCase
     end
   end
 end
-

data/test/database.yml

@@ -6,11 +6,12 @@ sqlite:
 mysql:
   adapter: mysql2
   database: search_cop
+  host: 127.0.0.1
   username: root
   encoding: utf8
 
 postgres:
-  host: localhost
+  host: 127.0.0.1
   adapter: postgresql
   database: search_cop
   username: search_cop

data/test/date_test.rb

@@ -1,9 +1,8 @@
-
-require File.expand_path("../test_helper", __FILE__)
+require File.expand_path("test_helper", __dir__)
 
 class DateTest < SearchCop::TestCase
   def test_mapping
-    product = create(:product, :created_on => Date.parse("2014-05-01"))
+    product = create(:product, created_on: Date.parse("2014-05-01"))
 
     assert_includes Product.search("created_on: 2014"), product
     assert_includes Product.search("created_on: 2014-05"), product
@@ -11,84 +10,84 @@ class DateTest < SearchCop::TestCase
   end
 
   def test_anywhere
-    product = create(:product, :created_on => Date.parse("2014-05-01"))
+    product = create(:product, created_on: Date.parse("2014-05-01"))
 
     assert_includes Product.search("2014-05-01"), product
     refute_includes Product.search("2014-05-02"), product
   end
 
   def test_includes
-    product = create(:product, :created_on => Date.parse("2014-05-01"))
+    product = create(:product, created_on: Date.parse("2014-05-01"))
 
     assert_includes Product.search("created_on: 2014-05-01"), product
     refute_includes Product.search("created_on: 2014-05-02"), product
   end
 
   def test_equals
-    product = create(:product, :created_on => Date.parse("2014-05-01"))
+    product = create(:product, created_on: Date.parse("2014-05-01"))
 
     assert_includes Product.search("created_on = 2014-05-01"), product
     refute_includes Product.search("created_on = 2014-05-02"), product
   end
 
   def test_equals_not
-    product = create(:product, :created_on => Date.parse("2014-05-01"))
+    product = create(:product, created_on: Date.parse("2014-05-01"))
 
     assert_includes Product.search("created_on != 2014-05-02"), product
     refute_includes Product.search("created_on != 2014-05-01"), product
   end
 
   def test_greater
-    product = create(:product, :created_on => Date.parse("2014-05-01"))
+    product = create(:product, created_on: Date.parse("2014-05-01"))
 
     assert_includes Product.search("created_on > 2014-04-01"), product
     refute_includes Product.search("created_on > 2014-05-01"), product
   end
 
   def test_greater_equals
-    product = create(:product, :created_on => Date.parse("2014-05-01"))
+    product = create(:product, created_on: Date.parse("2014-05-01"))
 
     assert_includes Product.search("created_on >= 2014-05-01"), product
     refute_includes Product.search("created_on >= 2014-05-02"), product
   end
 
   def test_less
-    product = create(:product, :created_on => Date.parse("2014-05-01"))
+    product = create(:product, created_on: Date.parse("2014-05-01"))
 
     assert_includes Product.search("created_on < 2014-05-02"), product
     refute_includes Product.search("created_on < 2014-05-01"), product
   end
 
   def test_less_equals
-    product = create(:product, :created_on => Date.parse("2014-05-02"))
+    product = create(:product, created_on: Date.parse("2014-05-02"))
 
     assert_includes Product.search("created_on <= 2014-05-02"), product
     refute_includes Product.search("created_on <= 2014-05-01"), product
   end
 
   def test_days_ago
-    product = create(:product, :created_at => 2.days.ago.to_date)
+    product = create(:product, created_at: 2.days.ago.to_date)
 
     assert_includes Product.search("created_at <= '1 day ago'"), product
     refute_includes Product.search("created_at <= '3 days ago'"), product
   end
 
   def test_weeks_ago
-    product = create(:product, :created_at => 2.weeks.ago.to_date)
+    product = create(:product, created_at: 2.weeks.ago.to_date)
 
     assert_includes Product.search("created_at <= '1 weeks ago'"), product
     refute_includes Product.search("created_at <= '3 weeks ago'"), product
   end
 
   def test_months_ago
-    product = create(:product, :created_at => 2.months.ago.to_date)
+    product = create(:product, created_at: 2.months.ago.to_date)
 
     assert_includes Product.search("created_at <= '1 months ago'"), product
     refute_includes Product.search("created_at <= '3 months ago'"), product
   end
 
   def test_years_ago
-    product = create(:product, :created_at => 2.years.ago.to_date)
+    product = create(:product, created_at: 2.years.ago.to_date)
 
     assert_includes Product.search("created_at <= '1 years ago'"), product
     refute_includes Product.search("created_at <= '3 years ago'"), product
@@ -106,4 +105,3 @@ class DateTest < SearchCop::TestCase
     end
   end
 end
-

data/test/datetime_test.rb

@@ -1,9 +1,8 @@
-
-require File.expand_path("../test_helper", __FILE__)
+require File.expand_path("test_helper", __dir__)
 
 class DatetimeTest < SearchCop::TestCase
   def test_mapping
-    product = create(:product, :created_at => Time.parse("2014-05-01 12:30:30"))
+    product = create(:product, created_at: Time.parse("2014-05-01 12:30:30"))
 
     assert_includes Product.search("created_at: 2014"), product
     assert_includes Product.search("created_at: 2014-05"), product
@@ -12,91 +11,91 @@ class DatetimeTest < SearchCop::TestCase
   end
 
   def test_anywhere
-    product = create(:product, :created_at => Time.parse("2014-05-01"))
+    product = create(:product, created_at: Time.parse("2014-05-01"))
 
     assert_includes Product.search("2014-05-01"), product
     refute_includes Product.search("2014-05-02"), product
   end
 
   def test_includes
-    product = create(:product, :created_at => Time.parse("2014-05-01"))
+    product = create(:product, created_at: Time.parse("2014-05-01"))
 
     assert_includes Product.search("created_at: 2014-05-01"), product
     refute_includes Product.search("created_at: 2014-05-02"), product
   end
 
   def test_equals
-    product = create(:product, :created_at => Time.parse("2014-05-01"))
+    product = create(:product, created_at: Time.parse("2014-05-01"))
 
     assert_includes Product.search("created_at = 2014-05-01"), product
     refute_includes Product.search("created_at = 2014-05-02"), product
   end
 
   def test_equals_not
-    product = create(:product, :created_at => Time.parse("2014-05-01"))
+    product = create(:product, created_at: Time.parse("2014-05-01"))
 
     assert_includes Product.search("created_at != 2014-05-02"), product
     refute_includes Product.search("created_at != 2014-05-01"), product
   end
 
   def test_greater
-    product = create(:product, :created_at => Time.parse("2014-05-01"))
+    product = create(:product, created_at: Time.parse("2014-05-01"))
 
     assert_includes Product.search("created_at > 2014-04-01"), product
     refute_includes Product.search("created_at > 2014-05-01"), product
   end
 
   def test_greater_equals
-    product = create(:product, :created_at => Time.parse("2014-05-01"))
+    product = create(:product, created_at: Time.parse("2014-05-01"))
 
     assert_includes Product.search("created_at >= 2014-05-01"), product
     refute_includes Product.search("created_at >= 2014-05-02"), product
   end
 
   def test_less
-    product = create(:product, :created_at => Time.parse("2014-05-01"))
+    product = create(:product, created_at: Time.parse("2014-05-01"))
 
     assert_includes Product.search("created_at < 2014-05-02"), product
     refute_includes Product.search("created_at < 2014-05-01"), product
   end
 
   def test_less_equals
-    product = create(:product, :created_at => Time.parse("2014-05-02"))
+    product = create(:product, created_at: Time.parse("2014-05-02"))
 
     assert_includes Product.search("created_at <= 2014-05-02"), product
     refute_includes Product.search("created_at <= 2014-05-01"), product
   end
 
   def test_hours_ago
-    product = create(:product, :created_at => 5.hours.ago)
+    product = create(:product, created_at: 5.hours.ago)
 
     assert_includes Product.search("created_at <= '4 hours ago'"), product
     refute_includes Product.search("created_at <= '6 hours ago'"), product
   end
 
   def test_days_ago
-    product = create(:product, :created_at => 2.days.ago)
+    product = create(:product, created_at: 2.days.ago)
 
     assert_includes Product.search("created_at <= '1 day ago'"), product
     refute_includes Product.search("created_at <= '3 days ago'"), product
   end
 
   def test_weeks_ago
-    product = create(:product, :created_at => 2.weeks.ago)
+    product = create(:product, created_at: 2.weeks.ago)
 
     assert_includes Product.search("created_at <= '1 weeks ago'"), product
     refute_includes Product.search("created_at <= '3 weeks ago'"), product
   end
 
   def test_months_ago
-    product = create(:product, :created_at => 2.months.ago)
+    product = create(:product, created_at: 2.months.ago)
 
     assert_includes Product.search("created_at <= '1 months ago'"), product
     refute_includes Product.search("created_at <= '3 months ago'"), product
   end
 
   def test_years_ago
-    product = create(:product, :created_at => 2.years.ago)
+    product = create(:product, created_at: 2.years.ago)
 
     assert_includes Product.search("created_at <= '1 years ago'"), product
     refute_includes Product.search("created_at <= '3 years ago'"), product
@@ -114,4 +113,3 @@ class DatetimeTest < SearchCop::TestCase
     end
   end
 end
-

data/test/default_operator_test.rb

@@ -1,7 +1,6 @@
-require File.expand_path('../test_helper', __FILE__)
+require File.expand_path("test_helper", __dir__)
 
 class DefaultOperatorTest < SearchCop::TestCase
-
   def test_without_default_operator
     avengers = create(:product, title: "Title Avengers", description: "2012")
     inception = create(:product, title: "Title Inception", description: "2010")
@@ -25,23 +24,28 @@ class DefaultOperatorTest < SearchCop::TestCase
 
   def test_with_specific_default_operator
     matrix = create(:product, title: "Matrix", description: "1999 Fantasy Sci-fi 2h 30m")
-    start_wars = create(:product, title: "Start Wars", description: "2010 Sci-fi Thriller 2h 28m")
+    star_wars = create(:product, title: "Star Wars", description: "2010 Sci-fi Thriller 2h 28m")
+
+    results = Product.search("Star Wars", default_operator: "AND")
+    refute_includes results, matrix
+    assert_includes results, star_wars
 
-    results = Product.search_multi_columns("Matrix movie", default_operator: :or)
+    results = Product.search_multi_columns("Matrix movie", default_operator: "OR")
     assert_includes results, matrix
-    refute_includes results, start_wars
+    refute_includes results, star_wars
 
-    results = Product.search(title: "Matrix", description: "2000", default_operator: :or)
+    results = Product.search({ title: "Matrix", description: "2000" }, default_operator: :or)
     assert_includes results, matrix
-    refute_includes results, start_wars
+    refute_includes results, star_wars
   end
 
   def test_with_invalid_default_operator
     assert_raises SearchCop::UnknownDefaultOperator do
-      Product.search_multi_columns('Matrix movie', default_operator: :xpto)
+      Product.search_multi_columns("Matrix movie", default_operator: :xpto)
     end
+
     assert_raises SearchCop::UnknownDefaultOperator do
-      Product.search_multi_columns(title: 'Matrix movie', default_operator: :xpto)
+      Product.search_multi_columns({ title: "Matrix movie" }, default_operator: :xpto)
     end
   end
-end
+end

data/test/error_test.rb

@@ -1,10 +1,9 @@
-
-require File.expand_path("../test_helper", __FILE__)
+require File.expand_path("test_helper", __dir__)
 
 class ErrorTest < SearchCop::TestCase
   def test_parse_error
     assert_raises SearchCop::ParseError do
-      Product.unsafe_search :title => { :unknown_operator => "Value" }
+      Product.unsafe_search title: { unknown_operator: "Value" }
     end
   end
 
@@ -14,4 +13,3 @@ class ErrorTest < SearchCop::TestCase
     end
   end
 end
-