Skip to content

Strengthen test assertions and remove fuzz reference checks - #10465

Merged
robert3005 merged 2 commits into
developfrom
rk/test-assertion-fixes
Oct 13, 2026
Merged

robert3005 merged 2 commits into
developfrom
rk/test-assertion-fixes

Conversation

@robert3005

@robert3005 robert3005 commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

fix the assertions that have been found by test audit

extracted from #10458

@robert3005 robert3005 added the changelog/chore A trivial change label Oct 13, 2026
@robert3005 robert3005 changed the title Strengthen test assertions and fuzz reference checks Strengthen test assertions and remove fuzz reference checks Oct 13, 2026
Extract the repairs for twelve mutation-verified scenarios from the
larger test audit. Check benchmark output and vector payloads, temporal
scalar values, exact spatial distance, FFI write errors, C++ integer
values, compiler wrapper arguments, and corpus-minimizer behavior.

Give the search fuzzer an independent linear reference and compare
compressed batch results against encoded scalar lookups. Enable the
array test harness explicitly for the benchmark tests.

Validation: 136 scoped Rust tests (including two temporary fuzz controls),
7 corpus-minimizer tests, 8 CMake tests, and the C++ integer scalar case passed.
Scoped Rust formatting, Ruff lint/format, and patch whitespace checks passed.

Signed-off-by: Robert Kruszewski <github@robertk.io>
@robert3005
robert3005 force-pushed the rk/test-assertion-fixes branch from 63f6f2b to 54828b8 Compare October 13, 2026 12:50
Signed-off-by: Robert Kruszewski <github@robertk.io>
@robert3005
robert3005 enabled auto-merge (squash) October 13, 2026 12:52
@robert3005
robert3005 merged commit 2664573 into develop Oct 13, 2026
92 checks passed
@robert3005
robert3005 deleted the rk/test-assertion-fixes branch October 13, 2026 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/chore A trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants