Files
bookhoard/cmd/server/tests
john-okeefe b3b40b77d6 fix: replace fixed sleep with proper job polling in TestWorker_ConcurrentJobs
Problem:
TestWorker_ConcurrentJobs was using a fixed 3-second sleep to wait for
concurrent scan jobs to complete. However, this wasn't sufficient time
for the watch mode to enqueue and process the jobs. When the test function
ended, Go's testing framework deleted all t.TempDir() directories,
causing the scanner to fail with 'no such file or directory' errors.

Error messages:
  Processing media file: /tmp/.../002/book0.epub
  Failed to get file info for /tmp/.../002/book0.epub: stat ...: no such file or directory

Root Cause:
The test created temporary directories and files using t.TempDir(), which
are automatically cleaned up when the test function ends. The scanner
needs time to process the files, but the test only waited 3 seconds before
checking results, causing temp dirs to be deleted mid-scan.

Solution:
Replaced the fixed 3-second sleep with proper job polling that:
1. Stores job IDs when submitting them to the worker
2. Polls job status every 100ms up to a 15-second timeout
3. Waits until all 3 jobs reach Completed or Failed status
4. Only then checks for media items in the database

This ensures the scanner has finished processing all files before the test
ends and temp dirs are cleaned up. Matches the polling pattern used in
TestWorker_DirectoryScanJob.

Files changed:
- cmd/server/tests/worker_test.go: Added job tracking and proper polling
2026-03-24 21:15:40 -04:00
..