Dragan Spiridonov 9e314ff383 fix(tests): unblock full unit suite and drop force-exit hack (#396) ...

The full `npx vitest run tests/unit/` suite was hanging >22 minutes
since v3.9.1 due to a futex deadlock in `@ruvector/router` native NAPI
calls during HNSW VectorDb operations. v3.9.5 already addressed the
root cause by flipping `useNativeHNSW` default from true to false
(commit 3a1d9636). This commit closes the loop by:

1. Fixing the one fallout test introduced by that flag flip:
   `tests/unit/cli/commands/ruvector-commands.test.ts:110` was setting
   `useNativeHNSW: false` to test the (modified) marker, but false is
   now the default, so the marker never appeared. Flipped to `true`.

2. Removing the `tests/setup-worker.ts` `process.exit(0)` force-exit
   hack. It was added as a band-aid when fork workers hung on lingering
   native handles. Since v3.9.3 added the `dispose()` contract on HNSW
   backends and v3.9.5 disabled native HNSW by default, workers now
   exit cleanly on their own. The 5-second main-process safety net in
   `tests/global-teardown.ts` is intentionally kept as a last resort.

Verification (full `tests/unit/`, 505 files):
- Before this PR (with hack):    7m26s, 1 failed,  16308 passed
- After  this PR (no hack):      6m21s, 0 failed,  16309 passed

The 1m05s saved comes from the per-file 2s force-exit delays the hack
was adding, even though the timer was unref'd.

Closes #396

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

2026-04-06 15:15:04 +00:00

917 B

Raw Permalink Blame History

View Raw