Add per-user home sections & unified Audible cache

Introduce per-user configurable home page sections and a unified Audible cache/category model. Adds Prisma models (UserHomeSection, AudibleCacheCategory) and migrations to create tables and remove legacy popular/new_release flags; updates schema.prisma accordingly. Add API routes for user home sections, live Audible categories, and category-based audiobook listing, and refactor popular/new-releases/covers routes to read from AudibleCacheCategory. Frontend: new HomeSection component, HomeSectionConfigModal, useHomeSections hook, and homepage changes to render dynamic sections plus image fallback to a placeholder SVG. Also add placeholder_cover.svg and tests for home sections and the audible refresh processor.
2026-06-03 04:40:09 +00:00 · 2026-03-05 11:30:39 -05:00
parent 248bd5359c
commit cc8e106a2b
40 changed files with 2582 additions and 655 deletions
@@ -68,6 +68,12 @@ describe('Audiobooks browse routes', () => {
  });

  it('returns popular audiobooks with cached cover URLs', async () => {
+    // Mock AudibleCacheCategory query (popular route now queries category table)
+    prismaMock.audibleCacheCategory.findMany.mockResolvedValueOnce([
+      { asin: 'ASIN', rank: 1 },
+    ]);
+    prismaMock.audibleCacheCategory.count.mockResolvedValueOnce(1);
+    // Mock AudibleCache metadata fetch
    prismaMock.audibleCache.findMany.mockResolvedValueOnce([
      {
        asin: 'ASIN',
@@ -84,7 +90,6 @@ describe('Audiobooks browse routes', () => {
        lastSyncedAt: new Date(),
      },
    ]);
-    prismaMock.audibleCache.count.mockResolvedValueOnce(1);
    enrichMock.mockResolvedValueOnce([{ asin: 'ASIN', coverArtUrl: '/api/cache/thumbnails/asin.jpg' }]);

    const { GET } = await import('@/app/api/audiobooks/popular/route');
@@ -106,8 +111,9 @@ describe('Audiobooks browse routes', () => {
  });

  it('returns new release audiobooks', async () => {
-    prismaMock.audibleCache.findMany.mockResolvedValueOnce([]);
-    prismaMock.audibleCache.count.mockResolvedValueOnce(0);
+    // Mock AudibleCacheCategory query (new-releases route now queries category table)
+    prismaMock.audibleCacheCategory.findMany.mockResolvedValueOnce([]);
+    prismaMock.audibleCacheCategory.count.mockResolvedValueOnce(0);

    const { GET } = await import('@/app/api/audiobooks/new-releases/route');
    const response = await GET({ nextUrl: new URL('http://app/api/audiobooks/new-releases?page=1&limit=1') } as any);
@@ -118,6 +124,12 @@ describe('Audiobooks browse routes', () => {
  });

  it('enriches new releases and uses cached cover URLs', async () => {
+    // Mock AudibleCacheCategory query
+    prismaMock.audibleCacheCategory.findMany.mockResolvedValueOnce([
+      { asin: 'ASIN', rank: 1 },
+    ]);
+    prismaMock.audibleCacheCategory.count.mockResolvedValueOnce(1);
+    // Mock AudibleCache metadata fetch
    prismaMock.audibleCache.findMany.mockResolvedValueOnce([
      {
        asin: 'ASIN',
@@ -134,7 +146,6 @@ describe('Audiobooks browse routes', () => {
        lastSyncedAt: new Date('2024-01-02'),
      },
    ]);
-    prismaMock.audibleCache.count.mockResolvedValueOnce(1);
    currentUserMock.mockReturnValue({ sub: 'user-1' });
    enrichMock.mockResolvedValueOnce([{ asin: 'ASIN', available: true }]);

@@ -155,7 +166,7 @@ describe('Audiobooks browse routes', () => {
  });

  it('returns 500 when new releases query fails', async () => {
-    prismaMock.audibleCache.findMany.mockRejectedValueOnce(new Error('db down'));
+    prismaMock.audibleCacheCategory.findMany.mockRejectedValueOnce(new Error('db down'));

    const { GET } = await import('@/app/api/audiobooks/new-releases/route');
    const response = await GET({ nextUrl: new URL('http://app/api/audiobooks/new-releases?page=1&limit=1') } as any);
@@ -209,6 +220,11 @@ describe('Audiobooks browse routes', () => {
  });

  it('returns cached covers for login', async () => {
+    // Mock AudibleCacheCategory query (covers route now queries category table)
+    prismaMock.audibleCacheCategory.findMany.mockResolvedValueOnce([
+      { asin: 'ASIN' },
+    ]);
+    // Mock AudibleCache metadata fetch
    prismaMock.audibleCache.findMany.mockResolvedValueOnce([
      { asin: 'ASIN', title: 'Title', author: 'Author', cachedCoverPath: '/tmp/asin.jpg', coverArtUrl: null },
    ]);
@@ -221,5 +237,3 @@ describe('Audiobooks browse routes', () => {
    expect(payload.covers[0].coverUrl).toBe('/api/cache/thumbnails/asin.jpg');
  });
 });
-
-
@@ -0,0 +1,166 @@
+/**
+ * Component: Home Sections API Route Tests
+ * Documentation: documentation/features/home-sections.md
+ */
+
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import { createPrismaMock } from '../helpers/prisma';
+
+const prismaMock = createPrismaMock();
+
+vi.mock('@/lib/db', () => ({
+  prisma: prismaMock,
+}));
+
+vi.mock('@/lib/middleware/auth', () => ({
+  requireAuth: vi.fn((_req: any, handler: any) => {
+    const mockReq = {
+      user: { id: 'user-1', sub: 'user-1', role: 'user' },
+      json: async () => (globalThis as any).__testBody || {},
+    };
+    return handler(mockReq);
+  }),
+  getCurrentUser: vi.fn(() => ({ sub: 'user-1' })),
+}));
+
+vi.mock('@/lib/utils/logger', () => ({
+  RMABLogger: { create: () => ({ info: vi.fn(), error: vi.fn(), warn: vi.fn(), debug: vi.fn() }) },
+}));
+
+describe('GET /api/user/home-sections', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+    // Re-apply default mock implementations after clearAllMocks
+    prismaMock.userHomeSection.createMany.mockResolvedValue({ count: 0 });
+    prismaMock.userHomeSection.deleteMany.mockResolvedValue({ count: 0 });
+    prismaMock.$transaction.mockImplementation(async (fn: any) => fn(prismaMock));
+  });
+
+  it('returns default sections for new user', async () => {
+    // ensureDefaultSections check: no existing sections
+    prismaMock.userHomeSection.findMany
+      .mockResolvedValueOnce([]) // ensureDefaultSections
+      .mockResolvedValueOnce([ // actual fetch after defaults created
+        { id: '1', sectionType: 'popular', categoryId: null, categoryName: null, sortOrder: 0 },
+        { id: '2', sectionType: 'new_releases', categoryId: null, categoryName: null, sortOrder: 1 },
+      ]);
+    prismaMock.scheduledJob.findFirst.mockResolvedValueOnce(null);
+
+    const { GET } = await import('@/app/api/user/home-sections/route');
+    const request = new Request('http://localhost/api/user/home-sections');
+    const response = await GET(request as any);
+    const data = await response.json();
+
+    expect(data.success).toBe(true);
+    expect(data.sections).toHaveLength(2);
+    expect(data.sections[0].sectionType).toBe('popular');
+    expect(data.sections[1].sectionType).toBe('new_releases');
+  });
+
+  it('returns existing sections without creating defaults', async () => {
+    prismaMock.userHomeSection.findMany
+      .mockResolvedValueOnce([{ id: '1' }]) // has existing
+      .mockResolvedValueOnce([
+        { id: '1', sectionType: 'category', categoryId: '123', categoryName: 'Sci-Fi', sortOrder: 0 },
+      ]);
+    prismaMock.scheduledJob.findFirst.mockResolvedValueOnce({
+      nextRun: new Date('2026-03-05T00:00:00Z'),
+    });
+
+    const { GET } = await import('@/app/api/user/home-sections/route');
+    const request = new Request('http://localhost/api/user/home-sections');
+    const response = await GET(request as any);
+    const data = await response.json();
+
+    expect(data.success).toBe(true);
+    expect(data.sections).toHaveLength(1);
+    expect(data.sections[0].categoryName).toBe('Sci-Fi');
+    expect(data.nextRefresh).toBe('2026-03-05T00:00:00.000Z');
+    expect(prismaMock.userHomeSection.createMany).not.toHaveBeenCalled();
+  });
+});
+
+describe('PUT /api/user/home-sections', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+    prismaMock.userHomeSection.createMany.mockResolvedValue({ count: 0 });
+    prismaMock.userHomeSection.deleteMany.mockResolvedValue({ count: 0 });
+    prismaMock.$transaction.mockImplementation(async (fn: any) => fn(prismaMock));
+  });
+
+  it('saves new section configuration', async () => {
+    (globalThis as any).__testBody = {
+      sections: [
+        { sectionType: 'new_releases', sortOrder: 0 },
+        { sectionType: 'popular', sortOrder: 1 },
+        { sectionType: 'category', categoryId: '123', categoryName: 'Sci-Fi', sortOrder: 2 },
+      ],
+    };
+
+    prismaMock.userHomeSection.findMany.mockResolvedValueOnce([
+      { id: '1', sectionType: 'new_releases', categoryId: null, categoryName: null, sortOrder: 0 },
+      { id: '2', sectionType: 'popular', categoryId: null, categoryName: null, sortOrder: 1 },
+      { id: '3', sectionType: 'category', categoryId: '123', categoryName: 'Sci-Fi', sortOrder: 2 },
+    ]);
+
+    const { PUT } = await import('@/app/api/user/home-sections/route');
+    const request = new Request('http://localhost/api/user/home-sections', { method: 'PUT' });
+    const response = await PUT(request as any);
+    const data = await response.json();
+
+    expect(data.success).toBe(true);
+    expect(data.sections).toHaveLength(3);
+    expect(prismaMock.userHomeSection.deleteMany).toHaveBeenCalledWith({
+      where: { userId: 'user-1' },
+    });
+    expect(prismaMock.userHomeSection.createMany).toHaveBeenCalled();
+  });
+
+  it('rejects more than 10 sections', async () => {
+    (globalThis as any).__testBody = {
+      sections: Array.from({ length: 11 }, (_, i) => ({
+        sectionType: 'category',
+        categoryId: `cat-${i}`,
+        categoryName: `Cat ${i}`,
+        sortOrder: i,
+      })),
+    };
+
+    const { PUT } = await import('@/app/api/user/home-sections/route');
+    const request = new Request('http://localhost/api/user/home-sections', { method: 'PUT' });
+    const response = await PUT(request as any);
+
+    expect(response.status).toBe(400);
+  });
+
+  it('rejects duplicate sections', async () => {
+    (globalThis as any).__testBody = {
+      sections: [
+        { sectionType: 'popular', sortOrder: 0 },
+        { sectionType: 'popular', sortOrder: 1 },
+      ],
+    };
+
+    const { PUT } = await import('@/app/api/user/home-sections/route');
+    const request = new Request('http://localhost/api/user/home-sections', { method: 'PUT' });
+    const response = await PUT(request as any);
+
+    expect(response.status).toBe(400);
+    const data = await response.json();
+    expect(data.message).toContain('Duplicate');
+  });
+
+  it('rejects category section without categoryId', async () => {
+    (globalThis as any).__testBody = {
+      sections: [{ sectionType: 'category', sortOrder: 0 }],
+    };
+
+    const { PUT } = await import('@/app/api/user/home-sections/route');
+    const request = new Request('http://localhost/api/user/home-sections', { method: 'PUT' });
+    const response = await PUT(request as any);
+
+    expect(response.status).toBe(400);
+    const data = await response.json();
+    expect(data.message).toContain('categoryId');
+  });
+});