Rename Filterable#search to Filterable#search_text to avoid name coll…

…ision

engine/app/filters/good_job/executions_filter.rb

@@ -14,7 +14,7 @@ def filtered_query
       query = base_query
       query = query.job_class(params[:job_class]) if params[:job_class].present?
       query = query.where(queue_name: params[:queue_name]) if params[:queue_name].present?
-      query = query.search(params['query']) if params[:query].present?
+      query = query.search_text(params[:query]) if params[:query].present?
 
       if params[:state]
         case params[:state]

engine/app/filters/good_job/jobs_filter.rb

@@ -18,7 +18,7 @@ def filtered_query
 
       query = query.job_class(params[:job_class]) if params[:job_class].present?
       query = query.where(queue_name: params[:queue_name]) if params[:queue_name].present?
-      query = query.search(params['query']) if params[:query].present?
+      query = query.search_text(params[:query]) if params[:query].present?
 
       if params[:state]
         case params[:state]

lib/good_job/filterable.rb

@@ -24,12 +24,12 @@ module Filterable
       end)
 
       # Search records by text query.
-      # @!method search(query)
+      # @!method search_text(query)
       # @!scope class
       # @param query [String]
       #   Search Query
       # @return [ActiveRecord::Relation]
-      scope :search, (lambda do |query|
+      scope :search_text, (lambda do |query|
         query = query.to_s.strip
         next if query.blank?
 

spec/engine/filters/good_job/executions_filter_spec.rb

@@ -74,6 +74,16 @@
         expect(filter.records.size).to eq 1
       end
     end
+
+    context 'when filtered by search' do
+      before do
+        params[:query] = 'default'
+      end
+
+      it 'returns a limited set of results' do
+        expect(filter.records.size).to eq 2
+      end
+    end
   end
 
   %w[

spec/engine/filters/good_job/jobs_filter_spec.rb

@@ -76,6 +76,16 @@
         expect(filter.records.size).to eq 1
       end
     end
+
+    context 'when filtered by search' do
+      before do
+        params[:query] = 'DeadError'
+      end
+
+      it 'returns a limited set of results' do
+        expect(filter.records.size).to eq 1
+      end
+    end
   end
 
   %w[

spec/lib/good_job/filterable_spec.rb

@@ -7,19 +7,19 @@
 
   describe '.search' do
     it 'searches serialized params' do
-      expect(model_class.search('example_value')).to include(execution)
+      expect(model_class.search_text('example_value')).to include(execution)
     end
 
     it 'searches record id' do
-      expect(model_class.search(execution.id)).to include(execution)
+      expect(model_class.search_text(execution.id)).to include(execution)
     end
 
     it 'searches errors' do
-      expect(model_class.search('ExampleError')).to include(execution)
+      expect(model_class.search_text('ExampleError')).to include(execution)
     end
 
     it 'filters out non-matching records' do
-      expect(model_class.search('ghost')).to be_empty
+      expect(model_class.search_text('ghost')).to be_empty
     end
 
     it 'is chainable and reversible' do