searchable-by 0.5.1 → 0.5.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8c1c5cfdea465a6a9ff73159a062558c5475889d27e0119e81212bb8bb37bbc1
-  data.tar.gz: c77c158c890b37b67d1243c93b4d5c982731b55fb48d284ecc404de58142da26
+  metadata.gz: 2137729d80e65107cadf27ede26a60887b6888c41f6940caea9828e3416f137f
+  data.tar.gz: 6edd61ef3f32e0fa6e882b07f5d24004a5c7a4cd5ec2f9c99ba2e6e12332f5e2
 SHA512:
-  metadata.gz: 1ba79022ae7056fc1cd6f6ef7af109ed6e0a7228601241ac2bc8f501818b6f487a6fd929f77bc3ce0dba9956b879def3dc705159c3696246d4653e3197f3d71f
-  data.tar.gz: c023dff5eb1bb4685dcfe7d10b7164242a090a930189ccc1b336c6de3437ab05d1de0dc90d13f52412e8e15b16a0a1386d5dacb289b208e00eefca5eab0cc2f7
+  metadata.gz: 704a8beb8c791dc20e6d156f73a3c3b21f6aed0df0d94d0cc7dda78440d9051f4a9f9a3d076fa3afa5af54a4312e9251335d2ab2911c228506c23d877ef67789
+  data.tar.gz: '0906f2424988065b3d22a177ab260d42029ad3e531a8b5e051fc7170d87e87784fa18225eba4f6631416af7a0c9a3df5c1b3e464babdb8d56ace5f226b18ee39'

data/.rubocop.yml

@@ -1,10 +1,14 @@
-require: rubocop-performance
+require:
+  - rubocop-performance
+  - rubocop-rake
+  - rubocop-rspec
+
 inherit_from:
   - https://gitlab.com/bsm/misc/raw/master/rubocop/default.yml
 
 AllCops:
-  TargetRubyVersion: "2.4"
+  TargetRubyVersion: "2.5"
 
 Naming/FileName:
   Exclude:
-    - lib/searchable-by.rb
+    - lib/searchable-by.rb

data/.travis.yml

@@ -2,7 +2,6 @@ language: ruby
 rvm:
   - 2.6
   - 2.5
-  - 2.4
 cache: bundler
 before_install:
   - gem install bundler

data/Gemfile.lock

@@ -1,65 +1,75 @@
 PATH
   remote: .
   specs:
-    searchable-by (0.5.1)
+    searchable-by (0.5.7)
       activerecord
       activesupport
 
 GEM
   remote: http://rubygems.org/
   specs:
-    activemodel (5.2.3)
-      activesupport (= 5.2.3)
-    activerecord (5.2.3)
-      activemodel (= 5.2.3)
-      activesupport (= 5.2.3)
-      arel (>= 9.0)
-    activesupport (5.2.3)
+    activemodel (6.1.1)
+      activesupport (= 6.1.1)
+    activerecord (6.1.1)
+      activemodel (= 6.1.1)
+      activesupport (= 6.1.1)
+    activesupport (6.1.1)
       concurrent-ruby (~> 1.0, >= 1.0.2)
-      i18n (>= 0.7, < 2)
-      minitest (~> 5.1)
-      tzinfo (~> 1.1)
-    arel (9.0.0)
-    ast (2.4.0)
-    concurrent-ruby (1.1.5)
-    diff-lcs (1.3)
-    i18n (1.6.0)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
+      zeitwerk (~> 2.3)
+    ast (2.4.1)
+    concurrent-ruby (1.1.7)
+    diff-lcs (1.4.4)
+    i18n (1.8.7)
       concurrent-ruby (~> 1.0)
-    jaro_winkler (1.5.2)
-    minitest (5.11.3)
-    parallel (1.17.0)
-    parser (2.6.3.0)
-      ast (~> 2.4.0)
+    minitest (5.14.3)
+    parallel (1.20.1)
+    parser (3.0.0.0)
+      ast (~> 2.4.1)
     rainbow (3.0.0)
-    rake (12.3.2)
-    rspec (3.8.0)
-      rspec-core (~> 3.8.0)
-      rspec-expectations (~> 3.8.0)
-      rspec-mocks (~> 3.8.0)
-    rspec-core (3.8.0)
-      rspec-support (~> 3.8.0)
-    rspec-expectations (3.8.3)
+    rake (13.0.3)
+    regexp_parser (2.0.3)
+    rexml (3.2.4)
+    rspec (3.10.0)
+      rspec-core (~> 3.10.0)
+      rspec-expectations (~> 3.10.0)
+      rspec-mocks (~> 3.10.0)
+    rspec-core (3.10.1)
+      rspec-support (~> 3.10.0)
+    rspec-expectations (3.10.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.8.0)
-    rspec-mocks (3.8.0)
+      rspec-support (~> 3.10.0)
+    rspec-mocks (3.10.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.8.0)
-    rspec-support (3.8.0)
-    rubocop (0.68.1)
-      jaro_winkler (~> 1.5.1)
+      rspec-support (~> 3.10.0)
+    rspec-support (3.10.1)
+    rubocop (1.8.1)
       parallel (~> 1.10)
-      parser (>= 2.5, != 2.5.1.1)
+      parser (>= 3.0.0.0)
       rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.8, < 3.0)
+      rexml
+      rubocop-ast (>= 1.2.0, < 2.0)
       ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 1.4.0, < 1.6)
-    rubocop-performance (1.2.0)
-      rubocop (>= 0.68.0)
-    ruby-progressbar (1.10.0)
-    sqlite3 (1.4.1)
-    thread_safe (0.3.6)
-    tzinfo (1.2.5)
-      thread_safe (~> 0.1)
-    unicode-display_width (1.5.0)
+      unicode-display_width (>= 1.4.0, < 3.0)
+    rubocop-ast (1.4.0)
+      parser (>= 2.7.1.5)
+    rubocop-performance (1.9.2)
+      rubocop (>= 0.90.0, < 2.0)
+      rubocop-ast (>= 0.4.0)
+    rubocop-rake (0.5.1)
+      rubocop
+    rubocop-rspec (2.1.0)
+      rubocop (~> 1.0)
+      rubocop-ast (>= 1.1.0)
+    ruby-progressbar (1.11.0)
+    sqlite3 (1.4.2)
+    tzinfo (2.0.4)
+      concurrent-ruby (~> 1.0)
+    unicode-display_width (2.0.0)
+    zeitwerk (2.4.2)
 
 PLATFORMS
   ruby
@@ -70,8 +80,10 @@ DEPENDENCIES
   rspec
   rubocop
   rubocop-performance
+  rubocop-rake
+  rubocop-rspec
   searchable-by!
   sqlite3
 
 BUNDLED WITH
-   2.0.1
+   2.1.4

data/LICENSE

@@ -1,4 +1,4 @@
-Copyright 2019 Black Square Media Ltd
+Copyright 2020 Black Square Media Ltd
 
 Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.

data/README.md

@@ -14,8 +14,10 @@ class Post < ActiveRecord::Base
 
   # Limit the number of terms per query to 3.
   searchable_by max_terms: 3 do
-    # Allow to search strings.
-    column :title
+    # Allow to search strings with custom match type.
+    # Use btree index-friendly prefix match, e.g. `ILIKE 'term%'` instead of default `ILIKE '%term%'`.
+    # For phrases use exact match type, e.g. searching for `"My Post"` will query `WHERE LOWER(title) = 'my post'`.
+    column :title, match: :prefix, match_phrase: :exact
 
     # ... and integers.
     column :id, type: :integer

data/lib/searchable_by.rb

@@ -1,132 +1,12 @@
 require 'active_record'
-require 'shellwords'
 
-module ActiveRecord
-  module SearchableBy
-    class Column
-      attr_reader :attr, :type
-      attr_accessor :node
+module SearchableBy
+  autoload :Column, 'searchable_by/column'
+  autoload :Concern, 'searchable_by/concern'
+  autoload :Config, 'searchable_by/config'
+  autoload :Util, 'searchable_by/util'
 
-      def initialize(attr, type: :string)
-        @attr = attr
-        @type = type.to_sym
-      end
-    end
-
-    class Config
-      attr_reader :columns, :scoping
-      attr_accessor :max_terms
-
-      def initialize
-        @columns = []
-        @max_terms = 5
-        scope { all }
-      end
-
-      def initialize_copy(other)
-        @columns = other.columns.dup
-        super
-      end
-
-      def column(*attrs, &block)
-        opts = attrs.extract_options!
-        attrs.each do |attr|
-          columns.push Column.new(attr, opts)
-        end
-        columns.push Column.new(block, opts) if block
-        columns
-      end
-
-      def scope(&block)
-        @scoping = block
-      end
-    end
-
-    def self.norm_values(query)
-      values = Shellwords.split(query.to_s)
-      values.flatten!
-      values.reject!(&:blank?)
-      values.uniq!
-      values
-    end
-
-    def self.build_clauses(columns, values)
-      clauses = values.map do |value|
-        negate = value[0] == '-'
-        value.slice!(0) if negate || value[0] == '+'
-
-        grouping = columns.map do |column|
-          build_condition(column, value)
-        end
-        grouping.compact!
-        next if grouping.empty?
-
-        clause = grouping.inject(&:or)
-        clause = clause.not if negate
-        clause
-      end
-      clauses.compact!
-      clauses
-    end
-
-    def self.build_condition(column, value)
-      case column.type
-      when :int, :integer
-        begin
-          column.node.eq(Integer(value))
-        rescue ArgumentError
-          nil
-        end
-      else
-        value = value.dup
-        value.gsub!('%', '\%')
-        value.gsub!('_', '\_')
-        column.node.matches("%#{value}%")
-      end
-    end
-
-    module ClassMethods
-      def self.extended(base) # :nodoc:
-        base.class_attribute :_searchable_by_config, instance_accessor: false, instance_predicate: false
-        base._searchable_by_config = Config.new
-        super
-      end
-
-      def inherited(base) # :nodoc:
-        base._searchable_by_config = _searchable_by_config.dup
-        super
-      end
-
-      def searchable_by(max_terms: 5, &block)
-        _searchable_by_config.instance_eval(&block)
-        _searchable_by_config.max_terms = max_terms if max_terms
-      end
-
-      # @param [String] query the search query
-      # @return [ActiveRecord::Relation] the scoped relation
-      def search_by(query)
-        columns = _searchable_by_config.columns
-        return all if columns.empty?
-
-        values = SearchableBy.norm_values(query).first(_searchable_by_config.max_terms)
-        return all if values.empty?
-
-        columns.each do |col|
-          col.node ||= col.attr.is_a?(Proc) ? col.attr.call : arel_table[col.attr]
-        end
-        clauses = SearchableBy.build_clauses(columns, values)
-        return all if clauses.empty?
-
-        scope = instance_exec(&_searchable_by_config.scoping)
-        clauses.each do |clause|
-          scope = scope.where(clause)
-        end
-        scope
-      end
-    end
-  end
-
-  class Base
-    extend SearchableBy::ClassMethods
-  end
+  Value = Struct.new(:term, :negate, :phrase)
 end
+
+ActiveRecord::Base.extend SearchableBy::Concern if defined?(::ActiveRecord::Base)

data/lib/searchable_by/column.rb

@@ -0,0 +1,51 @@
+module SearchableBy
+  class Column
+    attr_reader :attr, :type, :match, :match_phrase
+    attr_accessor :node
+
+    def initialize(attr, type: :string, match: :all, match_phrase: nil)
+      @attr  = attr
+      @type  = type.to_sym
+      @match = match
+      @match_phrase = match_phrase || match
+    end
+
+    def build_condition(value)
+      scope = node.not_eq(nil)
+
+      case type
+      when :int, :integer
+        int_condition(scope, value)
+      else
+        str_condition(scope, value)
+      end
+    end
+
+    private
+
+    def int_condition(scope, value)
+      scope.and(node.eq(Integer(value.term)))
+    rescue ArgumentError
+      nil
+    end
+
+    def str_condition(scope, value)
+      term = value.term.dup
+      type = value.phrase ? match_phrase : match
+
+      case type
+      when :exact
+        term.downcase!
+        scope.and(node.lower.eq(term))
+      when :prefix
+        term.gsub!('%', '\%')
+        term.gsub!('_', '\_')
+        scope.and(node.matches("#{term}%"))
+      else
+        term.gsub!('%', '\%')
+        term.gsub!('_', '\_')
+        scope.and(node.matches("%#{term}%"))
+      end
+    end
+  end
+end

data/lib/searchable_by/concern.rb

@@ -0,0 +1,43 @@
+module SearchableBy
+  module Concern
+    def self.extended(base) # :nodoc:
+      base.class_attribute :_searchable_by_config, instance_accessor: false, instance_predicate: false
+      base._searchable_by_config = Config.new
+      super
+    end
+
+    def inherited(base) # :nodoc:
+      base._searchable_by_config = _searchable_by_config.dup
+      super
+    end
+
+    def searchable_by(max_terms: nil, **options, &block)
+      _searchable_by_config.instance_eval(&block)
+      _searchable_by_config.max_terms = max_terms if max_terms
+      _searchable_by_config.options.update(options) unless options.empty?
+      _searchable_by_config
+    end
+
+    # @param [String] query the search query
+    # @return [ActiveRecord::Relation] the scoped relation
+    def search_by(query)
+      columns = _searchable_by_config.columns
+      return all if columns.empty?
+
+      values = Util.norm_values(query).first(_searchable_by_config.max_terms)
+      return all if values.empty?
+
+      columns.each do |col|
+        col.node ||= col.attr.is_a?(Proc) ? col.attr.call : arel_table[col.attr]
+      end
+      clauses = Util.build_clauses(columns, values)
+      return all if clauses.empty?
+
+      scope = instance_exec(&_searchable_by_config.scoping)
+      clauses.each do |clause|
+        scope = scope.where(clause)
+      end
+      scope
+    end
+  end
+end

data/lib/searchable_by/config.rb

@@ -0,0 +1,31 @@
+module SearchableBy
+  class Config
+    attr_reader :columns, :scoping, :options
+    attr_accessor :max_terms
+
+    def initialize
+      @columns = []
+      @max_terms = 5
+      @options = {}
+      scope { all }
+    end
+
+    def initialize_copy(other)
+      @columns = other.columns.dup
+      super
+    end
+
+    def column(*attrs, &block)
+      opts = attrs.extract_options!
+      attrs.each do |attr|
+        columns.push Column.new(attr, **@options, **opts)
+      end
+      columns.push Column.new(block, **@options, **opts) if block
+      columns
+    end
+
+    def scope(&block)
+      @scoping = block
+    end
+  end
+end

data/lib/searchable_by/util.rb

@@ -0,0 +1,46 @@
+module SearchableBy
+  module Util
+    def self.norm_values(query)
+      values = []
+      query  = query.to_s.dup
+
+      # capture any phrases inside double quotes
+      # exclude from search if preceded by '-'
+      query.gsub!(/([\-+]?)"+([^"]*)"+/) do |_|
+        term = Regexp.last_match(2)
+        negate = Regexp.last_match(1) == '-'
+
+        values.push Value.new(term, negate, true) unless term.blank?
+        ''
+      end
+
+      # for the remaining terms remove sign if precedes
+      # exclude term from search if sign preceding is '-'
+      query.split.each do |term|
+        negate = term[0] == '-'
+        term.slice!(0) if negate || term[0] == '+'
+
+        values.push Value.new(term, negate, false) unless term.blank?
+      end
+
+      values.uniq!
+      values
+    end
+
+    def self.build_clauses(columns, values)
+      clauses = values.map do |value|
+        grouping = columns.map do |column|
+          column.build_condition(value)
+        end
+        grouping.compact!
+        next if grouping.empty?
+
+        clause = grouping.inject(&:or)
+        clause = clause.not if value.negate
+        clause
+      end
+      clauses.compact!
+      clauses
+    end
+  end
+end

data/searchable-by.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name        = 'searchable-by'
-  s.version     = '0.5.1'
+  s.version     = '0.5.7'
   s.authors     = ['Dimitrij Denissenko']
   s.email       = ['dimitrij@blacksquaremedia.com']
   s.summary     = 'Generate search scopes'
@@ -8,10 +8,10 @@ Gem::Specification.new do |s|
   s.homepage    = 'https://github.com/bsm/sortable-by'
   s.license     = 'MIT'
 
-  s.files         = `git ls-files -z`.split("\x0").reject {|f| f.match(%r{^spec/}) }
+  s.files         = `git ls-files -z`.split("\x0").reject {|f| f.start_with?('spec/') }
   s.test_files    = `git ls-files -z -- spec/*`.split("\x0")
   s.require_paths = ['lib']
-  s.required_ruby_version = '>= 2.4'
+  s.required_ruby_version = '>= 2.5'
 
   s.add_dependency 'activerecord'
   s.add_dependency 'activesupport'
@@ -21,5 +21,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'rspec'
   s.add_development_dependency 'rubocop'
   s.add_development_dependency 'rubocop-performance'
+  s.add_development_dependency 'rubocop-rake'
+  s.add_development_dependency 'rubocop-rspec'
   s.add_development_dependency 'sqlite3'
 end

data/spec/searchable_by/util_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+describe SearchableBy::Util do
+  context 'with norm_values' do
+    def norm(str)
+      described_class.norm_values(str).each_with_object({}) do |val, acc|
+        acc[val.term] = val.negate
+      end
+    end
+
+    it 'tokenises strings' do
+      expect(norm(nil)).to eq({})
+      expect(norm('""')).to eq({})
+      expect(norm('-+""')).to eq({})
+      expect(norm('simple words')).to eq('simple' => false, 'words' => false)
+      expect(norm(" with   \t spaces\n")).to eq('with' => false, 'spaces' => false)
+      expect(norm('with with duplicates with')).to eq('with' => false, 'duplicates' => false)
+      expect(norm('with "full term"')).to eq('full term' => false, 'with' => false)
+      expect(norm('"""odd double quotes around"""')).to eq('odd double quotes around' => false)
+      expect(norm('""even double quotes around""')).to eq('even double quotes around' => false)
+      expect(norm('with\'apostrophe')).to eq("with'apostrophe" => false)
+      expect(norm('with -minus')).to eq('minus' => true, 'with' => false)
+      expect(norm('with +plus')).to eq('plus' => false, 'with' => false)
+      expect(norm('with-minus')).to eq('with-minus' => false)
+      expect(norm('with+plus')).to eq('with+plus' => false)
+      expect(norm('with -"minus before"')).to eq('minus before' => true, 'with' => false)
+      expect(norm('with "-minus within"')).to eq('-minus within' => false, 'with' => false)
+      expect(norm('with +"plus before"')).to eq('plus before' => false, 'with' => false)
+      expect(norm('with "+plus within"')).to eq('+plus within' => false, 'with' => false)
+      expect(norm('+plus "in other term"')).to eq('in other term' => false, 'plus' => false)
+      expect(norm('with_blank \'\'')).to eq('with_blank' => false, '\'\'' => false)
+      expect(norm('with_blank_doubles ""')).to eq('with_blank_doubles' => false)
+    end
+  end
+end

data/spec/searchable_by_spec.rb

@@ -1,48 +1,72 @@
 require 'spec_helper'
 
-describe ActiveRecord::SearchableBy do
-
-  it 'should ignore bad inputs' do
-    expect(Post.search_by(nil).count).to eq(4)
-    expect(Post.search_by('').count).to eq(4)
+describe SearchableBy do
+  it 'ignores bad inputs' do
+    expect(Post.search_by(nil).count).to eq(5)
+    expect(Post.search_by('').count).to eq(5)
   end
 
-  it 'should configure correctly' do
+  it 'configures correctly' do
     expect(AbstractModel._searchable_by_config.columns.size).to eq(1)
-    expect(Post._searchable_by_config.columns.size).to eq(4)
+    expect(Post._searchable_by_config.columns.size).to eq(5)
   end
 
-  it 'should generate SQL' do
+  it 'generates SQL' do
     sql = Post.search_by('123').to_sql
-    expect(sql).to include(%("posts"."id" = 123))
-    expect(sql).to include(%("posts"."title" LIKE '%123%'))
+    expect(sql).to include(%("posts"."id" IS NOT NULL AND "posts"."id" = 123))
+    expect(sql).to include(%("posts"."title" IS NOT NULL AND "posts"."title" LIKE '123%'))
+    expect(sql).to include(%("posts"."body" IS NOT NULL AND "posts"."body" LIKE '%123%'))
+    expect(sql).to include(%("users"."name" IS NOT NULL AND LOWER("users"."name") = '123'))
 
     sql = Post.search_by('foo%bar').to_sql
     expect(sql).not_to include(%("posts"."id"))
-    expect(sql).to include(%("posts"."title" LIKE '%foo\\%bar%'))
+    expect(sql).to include(%("posts"."title" LIKE 'foo\\%bar%'))
+    expect(sql).to include(%("posts"."body" LIKE '%foo\\%bar%'))
+  end
+
+  it 'searches' do
+    expect(Post.search_by('ALICE').pluck(:title)).to match_array(%w[ax1 ax2 ab1])
+    expect(Post.search_by('bOb').pluck(:title)).to match_array(%w[bx1 bx2 ab1])
   end
 
-  it 'should search' do
-    expect(Post.search_by('ALICE').pluck(:title)).to match_array(%w[titla title])
-    expect(Post.search_by('bOb').pluck(:title)).to match_array(%w[titlo titlu])
+  it 'searches across multiple words' do
+    expect(Post.search_by('ALICE your').pluck(:title)).to match_array(%w[ax2])
   end
 
-  it 'should search across multiple words' do
-    expect(Post.search_by('ALICE title').pluck(:title)).to match_array(%w[title])
+  it 'supports search markers' do
+    expect(Post.search_by('aLiCe -your').pluck(:title)).to match_array(%w[ax1 ab1])
+    expect(Post.search_by('+alice "your recipe"').pluck(:title)).to match_array(%w[ax2])
+    expect(Post.search_by('bob -"her recipe"').pluck(:title)).to match_array(%w[bx2 ab1])
+    expect(Post.search_by('bob +"her recipe"').pluck(:title)).to match_array(%w[bx1])
   end
 
-  it 'should support search markers' do
-    expect(Post.search_by('aLiCe -title').pluck(:title)).to match_array(%w[titla])
-    expect(Post.search_by('+alice "pie recipe"').pluck(:title)).to match_array(%w[title])
-    expect(Post.search_by('bob -"piu recipe"').pluck(:title)).to match_array(%w[titlo])
+  it 'respects match options' do
+    # name uses match: :exact
+    expect(Post.search_by('alice').pluck(:title)).to match_array(%w[ax1 ax2 ab1])
+    expect(Post.search_by('ali').pluck(:title)).to be_empty
+    expect(Post.search_by('lice').pluck(:title)).to be_empty
+    expect(Post.search_by('li').pluck(:title)).to be_empty
+
+    # title uses match: :prefix
+    expect(Post.search_by('ax').pluck(:title)).to match_array(%w[ax1 ax2])
+    expect(Post.search_by('bx').pluck(:title)).to match_array(%w[bx1 bx2])
+    expect(Post.search_by('ab').pluck(:title)).to match_array(%w[ab1])
+    expect(Post.search_by('ba').pluck(:title)).to be_empty
+
+    # title uses match_phrase: :exact
+    expect(Post.search_by('"ab"').pluck(:title)).to be_empty
+    expect(Post.search_by('"ab1"').pluck(:title)).to match_array(%w[ab1])
+
+    # body uses match: :all (default)
+    expect(Post.search_by('recip').pluck(:title)).to match_array(%w[ax1 ax2 bx1 bx2 ab1])
   end
 
-  it 'should search within scopes' do
-    expect(Post.where(title: 'title').search_by('ALICE').pluck(:title)).to match_array(%w[title])
-    expect(Post.where(title: 'title').search_by('bOb').pluck(:title)).to match_array(%w[])
+  it 'searches within scopes' do
+    expect(Post.where(title: 'ax1').search_by('ALICE').pluck(:title)).to match_array(%w[ax1])
+    expect(Post.where(title: 'ax1').search_by('bOb').pluck(:title)).to be_empty
   end
 
-  it 'should search integers' do
-    expect(Post.search_by(POSTS[:alice1].id.to_s).count).to eq(1)
+  it 'searches integers' do
+    expect(Post.search_by(POSTS[:ab1].id.to_s).count).to eq(1)
   end
 end

data/spec/spec_helper.rb

@@ -2,15 +2,16 @@ ENV['RACK_ENV'] ||= 'test'
 require 'searchable-by'
 require 'rspec'
 
-ActiveRecord::Base.configurations['test'] = { 'adapter' => 'sqlite3', 'database' => ':memory:' }
+ActiveRecord::Base.configurations = { 'test' => { 'adapter' => 'sqlite3', 'database' => ':memory:' } }
 ActiveRecord::Base.establish_connection :test
 
 ActiveRecord::Base.connection.instance_eval do
-  create_table :authors do |t|
+  create_table :users do |t|
     t.string  :name
   end
   create_table :posts do |t|
     t.integer :author_id, null: false
+    t.integer :reviewer_id
     t.string  :title
     t.text    :body
   end
@@ -24,31 +25,35 @@ class AbstractModel < ActiveRecord::Base
   end
 end
 
-class Author < AbstractModel
-  has_many :posts
+class User < AbstractModel
+  has_many :posts, foreign_key: :author_id
 end
 
 class Post < AbstractModel
-  belongs_to :author
+  belongs_to :author, class_name: 'User'
+  belongs_to :reviewer, class_name: 'User'
 
   searchable_by do
-    column :title, :body
-    column { Author.arel_table[:name] }
+    column :title, match: :prefix, match_phrase: :exact
+    column :body
+    column proc { User.arel_table[:name] }, match: :exact
+    column { User.arel_table.alias('reviewers_posts')[:name] }
 
     scope do
-      joins(:author)
+      joins(:author).left_outer_joins(:reviewer)
     end
   end
 end
 
-AUTHORS = {
-  alice: Author.create!(name: 'Alice'),
-  bob: Author.create!(name: 'Bob'),
+USERS = {
+  a: User.create!(name: 'Alice'),
+  b: User.create!(name: 'Bob'),
 }.freeze
 
 POSTS = {
-  alice1: AUTHORS[:alice].posts.create!(title: 'titla', body: 'my pia recipe '),
-  alice2: AUTHORS[:alice].posts.create!(title: 'title', body: 'your pie recipe'),
-  bob1: AUTHORS[:bob].posts.create!(title: 'titlo', body: 'her pio recipe'),
-  bob2: AUTHORS[:bob].posts.create!(title: 'titlu', body: 'our piu recipe'),
+  ax1: USERS[:a].posts.create!(title: 'ax1', body: 'my recipe '),
+  ax2: USERS[:a].posts.create!(title: 'ax2', body: 'your recipe'),
+  bx1: USERS[:b].posts.create!(title: 'bx1', body: 'her recipe'),
+  bx2: USERS[:b].posts.create!(title: 'bx2', body: 'our recipe'),
+  ab1: USERS[:a].posts.create!(title: 'ab1', reviewer: USERS[:b], body: 'their recipe'),
 }.freeze

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: searchable-by
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.5.7
 platform: ruby
 authors:
 - Dimitrij Denissenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-05-10 00:00:00.000000000 Z
+date: 2021-01-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -108,6 +108,34 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -139,7 +167,12 @@ files:
 - Rakefile
 - lib/searchable-by.rb
 - lib/searchable_by.rb
+- lib/searchable_by/column.rb
+- lib/searchable_by/concern.rb
+- lib/searchable_by/config.rb
+- lib/searchable_by/util.rb
 - searchable-by.gemspec
+- spec/searchable_by/util_spec.rb
 - spec/searchable_by_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/bsm/sortable-by
@@ -154,17 +187,18 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.4'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.4
 signing_key: 
 specification_version: 4
 summary: Generate search scopes
 test_files:
+- spec/searchable_by/util_spec.rb
 - spec/searchable_by_spec.rb
 - spec/spec_helper.rb