Allow user searching on multiple adapters

powerhome · May 10, 2024 · d095bc6 · d095bc6
1 parent 97c8473
commit d095bc6
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/audiences/app/models/audiences/users_search.rb b/audiences/app/models/audiences/users_search.rb
@@ -27,7 +27,14 @@ def users
   private
 
     def result
-      @result ||= @scope.where("data LIKE ?", "%#{@query}%")
+      @result ||= @scope.where("#{data_attribute_query} LIKE ?", "%#{@query}%")
+    end
+
+    def data_attribute_query
+      case @scope.connection.adapter_name
+      when "postgresql" then "CAST(data AS TEXT)"
+      else "CAST(data AS CHAR)"
+      end
     end
   end
 end
diff --git a/audiences/spec/models/audiences/user_search_spec.rb b/audiences/spec/models/audiences/user_search_spec.rb
@@ -4,8 +4,8 @@
 
 RSpec.describe Audiences::UsersSearch do
   it "searches through any serialized data attribute" do
-    john_doe = Audiences::ExternalUser.create(user_id: 123, data: { name: "John Doe" })
-    frank_doe = Audiences::ExternalUser.create(user_id: 321, data: { name: "Frank Doe", territory: "Philadelphia" })
+    john_doe = Audiences::ExternalUser.create(user_id: 123, data: { displayName: "John Doe" })
+    frank_doe = Audiences::ExternalUser.create(user_id: 321, data: { displayName: "Frank Doe", territory: "Philadelphia" })
 
     john_search = Audiences::UsersSearch.new(query: "John")
     phila_search = Audiences::UsersSearch.new(query: "Phila")