pg_search 2.3.8 → 2.4.0

data/spec/lib/pg_search/normalizer_spec.rb

@@ -7,56 +7,109 @@ describe PgSearch::Normalizer do
   describe "#add_normalization" do
     context "when config[:ignore] includes :accents" do
       context "when passed an Arel node" do
-        it "wraps the expression in unaccent()" do
+        it "returns an Arel node wrapping the expression in unaccent()" do
           config = instance_double(PgSearch::Configuration, "config", ignore: [:accents])
           node = Arel::Nodes::NamedFunction.new("foo", [Arel::Nodes.build_quoted("bar")])
 
           normalizer = described_class.new(config)
-          expect(normalizer.add_normalization(node)).to eq("regexp_replace(unaccent(foo('bar')), '[''?\\:]', '', 'g')")
+          result = normalizer.add_normalization(node)
+          expect(result).to be_an(Arel::Nodes::Node)
+          expect(result.to_sql).to eq("regexp_replace(unaccent(foo('bar')), '[''?\\:]', '', 'g')")
         end
 
         context "when a custom unaccent function is specified" do
-          it "wraps the expression in that function" do
+          it "returns an Arel node using that function" do
             allow(PgSearch).to receive(:unaccent_function).and_return("my_unaccent")
             node = Arel::Nodes::NamedFunction.new("foo", [Arel::Nodes.build_quoted("bar")])
 
             config = instance_double(PgSearch::Configuration, "config", ignore: [:accents])
 
             normalizer = described_class.new(config)
-            expect(normalizer.add_normalization(node)).to eq("regexp_replace(my_unaccent(foo('bar')), '[''?\\:]', '', 'g')")
+            result = normalizer.add_normalization(node)
+            expect(result).to be_an(Arel::Nodes::Node)
+            expect(result.to_sql).to eq("regexp_replace(my_unaccent(foo('bar')), '[''?\\:]', '', 'g')")
           end
         end
       end
 
-      context "when passed a String" do
-        it "wraps the expression in unaccent()" do
+      context "when passed a String (trusted SQL expression)" do
+        it "returns an Arel node wrapping the expression in unaccent()" do
           config = instance_double(PgSearch::Configuration, "config", ignore: [:accents])
 
           normalizer = described_class.new(config)
-          expect(normalizer.add_normalization("foo")).to eq("regexp_replace(unaccent(foo), '[''?\\:]', '', 'g')")
+          result = normalizer.add_normalization("foo")
+          expect(result).to be_an(Arel::Nodes::Node)
+          expect(result.to_sql).to eq("regexp_replace(unaccent(foo), '[''?\\:]', '', 'g')")
         end
 
         context "when a custom unaccent function is specified" do
-          it "wraps the expression in that function" do
+          it "returns an Arel node using that function" do
             allow(PgSearch).to receive(:unaccent_function).and_return("my_unaccent")
 
             config = instance_double(PgSearch::Configuration, "config", ignore: [:accents])
 
             normalizer = described_class.new(config)
-            expect(normalizer.add_normalization("foo")).to eq("regexp_replace(my_unaccent(foo), '[''?\\:]', '', 'g')")
+            result = normalizer.add_normalization("foo")
+            expect(result).to be_an(Arel::Nodes::Node)
+            expect(result.to_sql).to eq("regexp_replace(my_unaccent(foo), '[''?\\:]', '', 'g')")
           end
         end
       end
     end
 
+    it "rejects unsupported inputs instead of turning them into SQL" do
+      config = instance_double(PgSearch::Configuration, ignore: [])
+      normalizer = described_class.new(config)
+
+      [nil, Object.new, 123].each do |input|
+        expect { normalizer.add_normalization(input) }.to raise_error(TypeError)
+      end
+    end
+
     context "when config[:ignore] does not include :accents" do
-      it "passes the expression through" do
+      it "passes the expression through as an Arel-compatible object" do
         config = instance_double(PgSearch::Configuration, "config", ignore: [])
 
         normalizer = described_class.new(config)
-        expect(normalizer.add_normalization("foo")).to eq("foo")
+        result = normalizer.add_normalization("foo")
+        expect(result).to eq("foo")  # SqlLiteral is a String subclass, usable in Arel contexts
       end
     end
   end
+
+  describe "executed against PostgreSQL" do
+    with_model :Book do
+      table { |t| t.string :title }
+    end
+
+    it "strips accents and disallowed punctuation but preserves backslashes" do
+      Book.create!(title: "café's? déjà: vu\\path")
+      config = instance_double(PgSearch::Configuration, ignore: [:accents])
+      normalizer = described_class.new(config)
+      expression = normalizer.add_normalization(Book.arel_table[:title])
+
+      expect(Book.pick(expression)).to eq("cafes deja vu\\path")
+    end
+
+    it "preserves attributes when accents are not ignored" do
+      book = Book.create!(title: "café's? déjà: vu\\path")
+      config = instance_double(PgSearch::Configuration, ignore: [])
+      normalizer = described_class.new(config)
+      expression = normalizer.add_normalization(Book.arel_table[:title])
+
+      expect(Book.pick(expression)).to eq(book.title)
+    end
+
+    it "preserves quoted data rather than interpreting it as SQL" do
+      Book.create!
+      value = Arel::Nodes.build_quoted("café's? déjà: vu\\path")
+      config = instance_double(PgSearch::Configuration, ignore: [])
+      normalizer = described_class.new(config)
+
+      expect(normalizer.add_normalization(value)).to equal(value)
+      expect(Book.pick(normalizer.add_normalization(value)))
+        .to eq("café's? déjà: vu\\path")
+    end
+  end
 end
 # standard:enable RSpec/NestedGroups

data/spec/lib/pg_search_spec.rb

@@ -2,17 +2,6 @@
 
 require "spec_helper"
 
-# For Active Record 5.x, the association reflection's cache needs be cleared
-# because we're stubbing the related constants.
-if ActiveRecord::VERSION::MAJOR == 5
-  def clear_searchable_cache
-    PgSearch::Document.reflect_on_association(:searchable).clear_association_scope_cache
-  end
-else
-  def clear_searchable_cache
-  end
-end
-
 # standard:disable RSpec/NestedGroups
 describe PgSearch do
   describe ".multisearch" do
@@ -32,10 +21,7 @@ describe PgSearch do
     end
 
     context "with PgSearch.multisearch_options set to a Hash" do
-      subject do
-        clear_searchable_cache
-        described_class.multisearch(query).map(&:searchable)
-      end
+      subject { described_class.multisearch(query).map(&:searchable) }
 
       before { allow(described_class).to receive(:multisearch_options).and_return(using: :dmetaphone) }
 
@@ -57,10 +43,7 @@ describe PgSearch do
     end
 
     context "with PgSearch.multisearch_options set to a Proc" do
-      subject do
-        clear_searchable_cache
-        described_class.multisearch(query, soundalike).map(&:searchable)
-      end
+      subject { described_class.multisearch(query, soundalike).map(&:searchable) }
 
       before do
         allow(described_class).to receive(:multisearch_options) do
@@ -169,7 +152,6 @@ describe PgSearch do
 
           PgSearch::Multisearch.rebuild(SearchableSubclassModel)
 
-          clear_searchable_cache
           expect(PgSearch::Document.count).to be 1
           expect(PgSearch::Document.first.searchable.class).to be SearchableSubclassModel
           expect(PgSearch::Document.first.searchable).to eq expected
@@ -184,7 +166,6 @@ describe PgSearch do
           PgSearch::Multisearch.rebuild(SearchableSubclassModel)
           expect(PgSearch::Document.count).to be 2
 
-          clear_searchable_cache
           classes = PgSearch::Document.all.collect { |d| d.searchable.class }
           expect(classes).to include SearchableSubclassModel
           expect(classes).to include AnotherSearchableSubclassModel

metadata

@@ -1,11 +1,10 @@
 --- !ruby/object:Gem::Specification
 name: pg_search
 version: !ruby/object:Gem::Version
-  version: 2.3.8
+  version: 2.4.0
 platform: ruby
 authors:
 - Grant Hutchins
-- Case Commons, LLC
 bindir: bin
 cert_chain: []
 date: 1980-01-02 00:00:00.000000000 Z
@@ -16,33 +15,32 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.2'
+        version: '8.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.2'
+        version: '8.0'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.2'
+        version: '8.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.2'
+        version: '8.0'
 description: PgSearch builds Active Record named scopes that take advantage of PostgreSQL's
   full text search
 email:
 - gems@nertzy.com
-- casecommons-dev@googlegroups.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -63,6 +61,8 @@ files:
 - LICENSE
 - README.md
 - Rakefile
+- doc/plans/2026-09-05-arel-stack.md
+- doc/plans/2026-09-06-arel-remaining-expressions.md
 - lib/pg_search.rb
 - lib/pg_search/configuration.rb
 - lib/pg_search/configuration/association.rb
@@ -129,7 +129,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.18
+rubygems_version: 4.0.20
 specification_version: 4
 summary: PgSearch builds Active Record named scopes that take advantage of PostgreSQL's
   full text search