orneryd / NornicDB / 37260664610

87%

main: 89%

Build Type

Pull #912

github

Committed by mm0nst3r

Commit Message

test: cover background and order-dependent paths deterministically

Coveralls failed unrelated PRs by a few hundredths of a percent because
the same lines were covered in some runs and not others:

- buildIDCaseExpression sorted map keys with a hand-written insertion
  sort whose swap ran only when map order produced unsorted keys; it
  uses sort.Strings.
- Open's startup search build summary depended on how far the build got
  before a test closed its database; summarizeSearchBuilds is a function
  of its results and is tested for every outcome.
- DB.Search and DB.HybridSearchWithOptions each retried a search that
  ran during the initial index build; one helper, searchAfterInitialBuild,
  does it for both and is tested directly.
- The HNSW maintenance loop stopped on either its stop channel or its
  context, both signalled at shutdown; context cancellation is its only
  exit.
- The server's embedder retry loop is a method whose backoff schedule
  and constructor tests can set, so every attempt, backoff and stop path
  is tested.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Pull Request Pull Request #912: test: cover background and order-dependent paths deterministically

Coverage Stats

144 of 144 new or added lines in 5 files covered. (100.0%)

11 existing lines in 3 files now uncovered.

181326 of 208198 relevant lines covered (87.09%)

1.03 hits per line