Merge pull request #126 from brombomb/hardcover-api

Unified Reading Shelves & Hardcover Integration

tests/services/job-queue.service.test.ts

@@ -21,7 +21,7 @@ const processorsMock = vi.hoisted(() => ({
   processRetryMissingTorrents: vi.fn().mockResolvedValue('ok'),
   processRetryFailedImports: vi.fn().mockResolvedValue('ok'),
   processCleanupSeededTorrents: vi.fn().mockResolvedValue('ok'),
-  processSyncGoodreadsShelves: vi.fn().mockResolvedValue('ok'),
+  processSyncShelves: vi.fn().mockResolvedValue('ok'),
   // Ebook processors
   processSearchEbook: vi.fn().mockResolvedValue('ok'),
   processStartDirectDownload: vi.fn().mockResolvedValue('ok'),
@@ -116,8 +116,8 @@ vi.mock('@/lib/processors/cleanup-seeded-torrents.processor', () => ({
   processCleanupSeededTorrents: processorsMock.processCleanupSeededTorrents,
 }));
 
-vi.mock('@/lib/processors/sync-goodreads-shelves.processor', () => ({
-  processSyncGoodreadsShelves: processorsMock.processSyncGoodreadsShelves,
+vi.mock('@/lib/processors/sync-shelves.processor', () => ({
+  processSyncShelves: processorsMock.processSyncShelves,
 }));
 
 // Ebook processors
@@ -564,7 +564,7 @@ describe('JobQueueService', () => {
     expect(processorsMock.processRetryMissingTorrents).toHaveBeenCalled();
     expect(processorsMock.processRetryFailedImports).toHaveBeenCalled();
     expect(processorsMock.processCleanupSeededTorrents).toHaveBeenCalled();
-    expect(processorsMock.processSyncGoodreadsShelves).toHaveBeenCalled();
+    expect(processorsMock.processSyncShelves).toHaveBeenCalled();
   });
 
   it('returns repeatable jobs from the queue', async () => {

tests/services/scheduler.service.test.ts

@@ -18,7 +18,8 @@ const jobQueueMock = vi.hoisted(() => ({
   addRetryFailedImportsJob: vi.fn(),
   addCleanupSeededTorrentsJob: vi.fn(),
   addMonitorRssFeedsJob: vi.fn(),
-  addSyncGoodreadsShelvesJob: vi.fn(),
+  addMonitorRssFeedsJob: vi.fn(),
+  addSyncShelvesJob: vi.fn(),
 }));
 
 const configServiceMock = vi.hoisted(() => ({
@@ -63,7 +64,9 @@ describe('SchedulerService', () => {
     prismaMock.scheduledJob.findFirst.mockResolvedValue(null);
     prismaMock.scheduledJob.create.mockResolvedValue({});
     prismaMock.scheduledJob.findMany
+      .mockResolvedValueOnce([]) // cleanupDeprecatedJobs
       .mockResolvedValueOnce([
+        // scheduleAllJobs
         {
           id: 'job-1',
           name: 'Audible Data Refresh',
@@ -72,7 +75,7 @@ describe('SchedulerService', () => {
           enabled: true,
         },
       ])
-      .mockResolvedValueOnce([]);
+      .mockResolvedValue([]); // triggerOverdueJobs
 
     const { SchedulerService } = await import('@/lib/services/scheduler.service');
     const service = new SchedulerService();
@@ -289,7 +292,7 @@ describe('SchedulerService', () => {
     ['retry_failed_imports', 'addRetryFailedImportsJob'],
     ['cleanup_seeded_torrents', 'addCleanupSeededTorrentsJob'],
     ['monitor_rss_feeds', 'addMonitorRssFeedsJob'],
-    ['sync_goodreads_shelves', 'addSyncGoodreadsShelvesJob'],
+    ['sync_reading_shelves', 'addSyncShelvesJob'],
   ])('triggers %s jobs with job queue', async (type, queueMethod) => {
     prismaMock.scheduledJob.findUnique.mockResolvedValue({
       id: 'job-type',