Merge pull request #321 from shoaib050326/codex/issue-161-category-filter

[codex] Add category filtering to raffle search

Author: Otaiki1

backend/src/api/rest/search/search.controller.spec.ts

@@ -1,4 +1,32 @@
 import { SearchController } from './search.controller';
+import { SearchService } from '../../../services/search.service';
+
+describe('SearchController', () => {
+  let controller: SearchController;
+  let searchService: { search: jest.Mock };
+
+  beforeEach(() => {
+    searchService = {
+      search: jest.fn(),
+    };
+
+    controller = new SearchController(searchService as unknown as SearchService);
+  });
+
+  it('passes a trimmed category filter to the search service', async () => {
+    searchService.search.mockResolvedValue([]);
+
+    await controller.search('raffle', '  Art  ');
+
+    expect(searchService.search).toHaveBeenCalledWith('raffle', 'Art');
+  });
+
+  it('treats an empty category as no filter', async () => {
+    searchService.search.mockResolvedValue([]);
+
+    await controller.search('raffle', '   ');
+
+    expect(searchService.search).toHaveBeenCalledWith('raffle', undefined);
 
 describe('SearchController', () => {
   it('forwards q, limit, and offset and returns the service total', async () => {

backend/src/services/metadata.service.spec.ts

@@ -0,0 +1,57 @@
+import { MetadataService } from './metadata.service';
+import { SUPABASE_CLIENT } from './supabase.provider';
+
+describe('MetadataService', () => {
+  let service: MetadataService;
+  let queryBuilder: {
+    select: jest.Mock;
+    or: jest.Mock;
+    ilike: jest.Mock;
+    limit: jest.Mock;
+    upsert: jest.Mock;
+  };
+  let client: { from: jest.Mock };
+
+  beforeEach(() => {
+    queryBuilder = {
+      select: jest.fn().mockReturnThis(),
+      or: jest.fn().mockReturnThis(),
+      ilike: jest.fn().mockReturnThis(),
+      limit: jest.fn(),
+      upsert: jest.fn(),
+    };
+
+    client = {
+      from: jest.fn().mockReturnValue(queryBuilder),
+    };
+
+    service = new MetadataService(client as any);
+  });
+
+  it('applies category filtering case-insensitively', async () => {
+    queryBuilder.limit.mockResolvedValue({ data: [], error: null });
+
+    await service.searchMetadata('raffle', 'Art');
+
+    expect(queryBuilder.ilike).toHaveBeenCalledWith('category', 'Art');
+  });
+
+  it('trims category values before upserting metadata', async () => {
+    const selectBuilder = {
+      single: jest.fn().mockResolvedValue({
+        data: { raffle_id: 1, category: 'Art' },
+        error: null,
+      }),
+    };
+    queryBuilder.upsert.mockReturnValue({
+      select: jest.fn().mockReturnValue(selectBuilder),
+    });
+
+    await service.upsertMetadata(1, { category: '  Art  ' });
+
+    expect(queryBuilder.upsert).toHaveBeenCalledWith(
+      expect.objectContaining({ category: 'Art' }),
+      expect.any(Object),
+    );
+  });
+});

backend/src/services/metadata.service.ts

@@ -111,6 +111,7 @@ export class MetadataService {
     const row = {
       raffle_id: raffleId,
       ...payload,
+      category: payload.category?.trim() || null,
       updated_at: new Date().toISOString(),
     };