goosefoot
Stream beets imports over HTTP.
Star 1
Fork 1
Watch

test(server): split multi-scenario import tests and replace sleeps with synctest #43

Merged glerp.sullen.net
targeting main from glerp.sullen.net/goosefoot:test/hygiene-server-import
Merged opened by and assigned to glerp.sullen.net
targeting main from glerp.sullen.net/goosefoot:test/hygiene-server-import

Summary #

  • Split multi-scenario mega-tests (duplicateActiveImportPath, overlappingAncestorOrDescendantImportPath, dryRunOnActiveImportPath, completedOrFailedImportReleasesPath) into single-behavior linear Arrange-Act-Assert tests with 1:1 name-to-assertion parity.
  • Replace wall-clock time.Sleep hacks in concurrent queue, SSE keepalive heartbeat, and client-disconnect tests with deterministic testing/synctest (synctest.Test / synctest.Wait) and notifying recorder synchronization.
  • Add missing coverage for StepTimeout expiration during Runner.DryRun and Runner.Import (using virtual time), non-fatal runtime hooks.Discover failure continuation, single regular-file import happy path, and dangling symlinks escaping ImportDirectory.
  • Replace abbreviated identifiers (srv, req, rec, ct, cc, conn, accel, hooksDir, mbURL, importingIdx, mu, wg, logBuf, etc.) with full descriptive names.
Commits 0