bharat_filter_engine 0.1.2 → 0.2.0

data/spec/search_builder_spec.rb

@@ -1,79 +1,78 @@
-require "spec_helper"
+# frozen_string_literal: true
 
-RSpec.describe BharatFilterEngine::SearchBuilder do
+require 'spec_helper'
 
+RSpec.describe BharatFilterEngine::SearchBuilder do
   let!(:client) do
     Client.create!(
-      name: "John Doe",
-      email: "john@example.com"
+      name: 'John Doe',
+      email: 'john@example.com'
     )
   end
 
   let!(:lead) do
     Lead.create!(
-      source: "google",
+      source: 'google',
       client: client
     )
   end
 
   let!(:sale) do
     Sale.create!(
-      stage: "qualified",
+      stage: 'qualified',
       lead: lead
     )
   end
 
-  it "supports simple search" do
-
+  it 'supports simple search' do
     result =
       described_class.new(
         scope: Sale.all,
-        allowed_columns: [
-          :stage,
-          :"lead__source"
+        allowed_columns: %i[
+          stage
+          lead__source
         ]
-      ).apply("google")
+      ).apply('google')
 
     expect(result).to contain_exactly(sale)
   end
 
-  it "supports case insensitive search" do
+  it 'supports case insensitive search' do
     result =
       described_class.new(
         scope: Sale.all,
         allowed_columns: [:stage]
-      ).apply("QUAL")
+      ).apply('QUAL')
 
     expect(result).to contain_exactly(sale)
   end
 
-  it "supports nested field search" do
-
+  it 'supports nested field search' do
     result =
       described_class.new(
         scope: Sale.all,
         allowed_columns: [
-          :"lead__client__name"
+          :lead__client__name
         ]
       ).apply(
-        "lead__client__name=John"
+        'lead__client__name=John'
       )
 
     expect(result).to contain_exactly(sale)
   end
 
-  it "supports multi-level association search" do
+  it 'supports multi-level association search' do
     organization = Organization.create!(
-      name: "Acme"
+      name: 'Acme'
     )
 
     client_two = Client.create!(
-      name: "Jane",
+      name: 'Jane',
       organization: organization
     )
 
     lead_two = Lead.create!(
-      source: "referral",
+      source: 'referral',
       client: client_two
     )
 
@@ -85,56 +84,76 @@ RSpec.describe BharatFilterEngine::SearchBuilder do
       described_class.new(
         scope: Sale.all,
         allowed_columns: [
-          :"lead__client__organization__name"
+          :lead__client__organization__name
         ]
       ).apply(
-        "lead__client__organization__name=Acme"
+        'lead__client__organization__name=Acme'
       )
 
     expect(result).to contain_exactly(sale_two)
   end
 
-  it "supports OR search" do
+  it 'returns each record once when searching through a has_many association' do
+    matching_client = Client.create!(name: 'John1')
+    Lead.create!(source: 'google', client: matching_client)
+    Lead.create!(source: 'google', client: matching_client)
+    Client.create!(name: 'John2').tap do |other_client|
+      Lead.create!(source: 'meta', client: other_client)
+    end
 
+    result =
+      described_class.new(
+        scope: Client.all,
+        allowed_columns: [:leads__source]
+      ).apply(
+        'leads__source=google'
+      )
+
+    expect(result).to contain_exactly(client, matching_client)
+    expect(result.count).to eq(2)
+    expect(result.order(:id).limit(1).pluck(:id)).to eq([client.id])
+  end
+
+  it 'supports OR search' do
     result =
       described_class.new(
         scope: Sale.all,
-        allowed_columns: [
-          :stage,
-          :"lead__source"
+        allowed_columns: %i[
+          stage
+          lead__source
         ]
       ).apply(
-        "stage=qualified|lead__source=google"
+        'stage=qualified|lead__source=google'
       )
 
     expect(result).to contain_exactly(sale)
   end
 
-  it "supports combined AND and OR conditions" do
+  it 'supports combined AND and OR conditions' do
     sale_one = Sale.create!(
-      stage: "qualified",
-      approval_status: "approved"
+      stage: 'qualified',
+      approval_status: 'approved'
     )
 
     sale_two = Sale.create!(
-      stage: "qualified",
-      approval_status: "pending"
+      stage: 'qualified',
+      approval_status: 'pending'
     )
 
     Sale.create!(
-      stage: "new",
-      approval_status: "approved"
+      stage: 'new',
+      approval_status: 'approved'
     )
 
     result =
       described_class.new(
         scope: Sale.all,
-        allowed_columns: [
-          :stage,
-          :approval_status
+        allowed_columns: %i[
+          stage
+          approval_status
         ]
       ).apply(
-        "stage=qualified&approval_status=approved|approval_status=pending"
+        'stage=qualified&approval_status=approved|approval_status=pending'
       )
 
     expect(result).to contain_exactly(
@@ -143,15 +162,176 @@ RSpec.describe BharatFilterEngine::SearchBuilder do
     )
   end
 
-  it "returns no records when the searched field is not configured" do
+  it 'returns no records when the searched field is not configured' do
+    result =
+      described_class.new(
+        scope: Sale.all,
+        allowed_columns: [:stage]
+      ).apply(
+        'unknown=value'
+      )
+
+    expect(result).to be_empty
+  end
+
+  it 'returns no records when the searched model column is not allowed' do
+    result =
+      described_class.new(
+        scope: Sale.all,
+        allowed_columns: [:stage]
+      ).apply(
+        'approval_status=approved'
+      )
+
+    expect(result).to be_empty
+  end
+
+  it 'returns no records when the searched association column is not allowed' do
+    result =
+      described_class.new(
+        scope: Sale.all,
+        allowed_columns: [:stage]
+      ).apply(
+        'lead__client__email=john@example.com'
+      )
+
+    expect(result).to be_empty
+  end
+
+  it 'supports searching the id column as text' do
+    result =
+      described_class.new(
+        scope: Sale.all,
+        allowed_columns: [:id]
+      ).apply(
+        "id=#{sale.id}"
+      )
+
+    expect(result).to contain_exactly(sale)
+  end
+
+  it "handles a value that itself contains an '=' sign" do
+    sale_with_url = Sale.create!(
+      stage: 'https://example.com?x=1'
+    )
+
+    result =
+      described_class.new(
+        scope: Sale.all,
+        allowed_columns: [:stage]
+      ).apply(
+        'stage=https://example.com?x=1'
+      )
+
+    expect(result).to contain_exactly(sale_with_url)
+  end
+
+  it 'skips a whitespace-only expression value' do
     result =
       described_class.new(
         scope: Sale.all,
         allowed_columns: [:stage]
       ).apply(
-        "unknown=value"
+        'stage=   '
       )
 
+    # every expression in the (only) AND group is blank/skipped, so
+    # the group can never match anything
     expect(result).to be_empty
   end
+
+  it 'returns the scope unchanged when allowed_columns is empty' do
+    result =
+      described_class.new(
+        scope: Sale.all,
+        allowed_columns: []
+      ).apply('anything')
+
+    expect(result).to contain_exactly(sale)
+  end
+
+  it 'raises when none of the allowed_columns can be resolved' do
+    expect do
+      described_class.new(
+        scope: Sale.all,
+        allowed_columns: %i[
+          totally_not_a_column
+          also__not__real
+        ]
+      )
+    end.to raise_error(BharatFilterEngine::InvalidConfigurationError)
+  end
+
+  it 'does not raise when only some allowed_columns are invalid' do
+    expect do
+      described_class.new(
+        scope: Sale.all,
+        allowed_columns: %i[
+          stage
+          totally_not_a_column
+        ]
+      )
+    end.not_to raise_error
+  end
+
+  it 'reuses a single AssociationResolver across a whole search' do
+    allow(BharatFilterEngine::AssociationResolver)
+      .to receive(:new)
+      .and_call_original
+
+    described_class.new(
+      scope: Sale.all,
+      allowed_columns: %i[
+        lead__source
+        lead__client__name
+      ]
+    ).apply(
+      'lead__source=google&lead__client__name=John'
+    )
+
+    expect(BharatFilterEngine::AssociationResolver)
+      .to have_received(:new)
+      .once
+  end
+
+  describe 'LIKE wildcard escaping' do
+    it 'treats a literal underscore as a literal character, not a wildcard' do
+      match = Sale.create!(stage: 'sale_2026')
+      Sale.create!(stage: 'saleX2026')
+
+      result =
+        described_class.new(
+          scope: Sale.all,
+          allowed_columns: [:stage]
+        ).apply('sale_2026')
+
+      expect(result).to contain_exactly(match)
+    end
+
+    it 'treats a literal percent sign as a literal character, not a wildcard' do
+      match = Sale.create!(stage: '50%off')
+      Sale.create!(stage: '50xoff')
+
+      result =
+        described_class.new(
+          scope: Sale.all,
+          allowed_columns: [:stage]
+        ).apply('50%off')
+
+      expect(result).to contain_exactly(match)
+    end
+
+    it 'escapes wildcards in field-based search too' do
+      match = Sale.create!(stage: 'sale_2026')
+      Sale.create!(stage: 'saleX2026')
+
+      result =
+        described_class.new(
+          scope: Sale.all,
+          allowed_columns: [:stage]
+        ).apply('stage=sale_2026')
+
+      expect(result).to contain_exactly(match)
+    end
+  end
 end

data/spec/spec_helper.rb

@@ -1,14 +1,15 @@
 # frozen_string_literal: true
 
-require "bundler/setup"
-require "active_record"
-require "database_cleaner/active_record"
-require "bharat_filter_engine"
-require "sqlite3"
+require 'logger'
+require 'bundler/setup'
+require 'active_record'
+require 'database_cleaner/active_record'
+require 'bharat_filter_engine'
+require 'sqlite3'
 
 ActiveRecord::Base.establish_connection(
-  adapter: "sqlite3",
-  database: ":memory:"
+  adapter: 'sqlite3',
+  database: ':memory:'
 )
 
 ActiveRecord::Schema.define do
@@ -26,6 +27,7 @@ ActiveRecord::Schema.define do
     t.string :source
     t.string :status
     t.references :client
+    t.integer :referrer_id
     t.datetime :created_at
   end
 
@@ -41,29 +43,26 @@ ActiveRecord::Schema.define do
   end
 end
 
-
 class Organization < ActiveRecord::Base
   has_many :clients
 end
 
-
 class Client < ActiveRecord::Base
   belongs_to :organization
   has_many :leads
+  has_many :referred_leads, class_name: 'Lead', foreign_key: :referrer_id
 end
 
-
 class Lead < ActiveRecord::Base
   belongs_to :client
+  belongs_to :referrer, class_name: 'Client', optional: true
   has_many :sales
 end
 
-
 class Sale < ActiveRecord::Base
   belongs_to :lead
 end
 
-
 RSpec.configure do |config|
   config.before(:suite) do
     DatabaseCleaner.clean_with(:truncation)

metadata

@@ -1,17 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: bharat_filter_engine
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.2.0
 platform: ruby
 authors:
 - Jayesh Rathore
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-09-03 00:00:00.000000000 Z
+date: 2026-11-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: activerecord
+  name: activemodel
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -25,7 +25,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '6.1'
 - !ruby/object:Gem::Dependency
-  name: activemodel
+  name: activerecord
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -53,7 +53,9 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '6.1'
 description: Configurable Rails ActiveRecord filtering engine supporting string, array,
-  boolean, numeric, date-range, nested association and AND/OR search filters.
+  boolean, numeric (with gte/lte/between ranges), date-range, presence, negate/exclude,
+  nested association, and AND/OR search filters, plus dynamic filter-value lookups
+  for dropdowns/autocomplete.
 email:
 - jayeshrathore7024@gmail.com
 executables: []
@@ -67,25 +69,31 @@ files:
 - Rakefile
 - lib/bharat_filter_engine.rb
 - lib/bharat_filter_engine/association_resolver.rb
+- lib/bharat_filter_engine/column_helper.rb
 - lib/bharat_filter_engine/dynamic_filter_values.rb
 - lib/bharat_filter_engine/engine.rb
 - lib/bharat_filter_engine/errors.rb
+- lib/bharat_filter_engine/join_tracker.rb
 - lib/bharat_filter_engine/rule_normalizer.rb
 - lib/bharat_filter_engine/search_builder.rb
 - lib/bharat_filter_engine/version.rb
 - spec/association_resolver_spec.rb
 - spec/dynamic_filter_values_spec.rb
 - spec/engine_spec.rb
+- spec/join_tracker_spec.rb
 - spec/rule_normalizer_spec.rb
+- spec/same_table_join_spec.rb
 - spec/search_builder_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/jayesh70240/bharat-filter-engine
 licenses:
 - MIT
 metadata:
+  homepage_uri: https://github.com/jayesh70240/bharat-filter-engine
   source_code_uri: https://github.com/jayesh70240/bharat-filter-engine
   changelog_uri: https://github.com/jayesh70240/bharat-filter-engine/blob/main/CHANGELOG.md
   bug_tracker_uri: https://github.com/jayesh70240/bharat-filter-engine/issues
+  rubygems_mfa_required: 'true'
 post_install_message:
 rdoc_options: []
 require_paths:
@@ -101,7 +109,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.3
+rubygems_version: 3.5.22
 signing_key:
 specification_version: 4
 summary: A generic filtering engine for Rails ActiveRecord