Skip to content

test: close zone-map coverage loop (inspector pane + Rust interop)#122

Merged
dfa1 merged 2 commits into
mainfrom
test/inspector-stats-pane
Jun 21, 2026
Merged

test: close zone-map coverage loop (inspector pane + Rust interop)#122
dfa1 merged 2 commits into
mainfrom
test/inspector-stats-pane

Conversation

@dfa1

@dfa1 dfa1 commented Jun 21, 2026

Copy link
Copy Markdown
Owner

Now that the writer emits vortex.stats zone-maps, the rich Java fixture's
I64 column carries a per-chunk stats layout, so the inspector's
per-chunk-stats detail pane (runStatsLoad / decodeStatsLayout /
zoneStatsAnchor) renders and is asserted. Closes the coverage gap that
first surfaced the missing writer capability.

VortexInspectorTui line coverage 66.9% -> 75.8%, branch 55.5% -> 64.8%.

Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com

dfa1 and others added 2 commits June 21, 2026 15:21
Now that the writer emits vortex.stats zone-maps, the rich Java fixture's
I64 column carries a per-chunk stats layout, so the inspector's
per-chunk-stats detail pane (runStatsLoad / decodeStatsLayout /
zoneStatsAnchor) renders and is asserted. Closes the coverage gap that
first surfaced the missing writer capability.

VortexInspectorTui line coverage 66.9% -> 75.8%, branch 55.5% -> 64.8%.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Multi-chunk file written with enableZoneMaps (one zone per chunk) is read
back via the Rust JNI reader; all values round-trip, proving the
Java-emitted vortex.stats layout is Rust-compatible.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@dfa1 dfa1 changed the title test(cli): cover inspector per-chunk-stats pane via zone-mapped fixture test: close zone-map coverage loop (inspector pane + Rust interop) Jun 21, 2026
@dfa1 dfa1 merged commit f2d7435 into main Jun 21, 2026
6 checks passed
@dfa1 dfa1 deleted the test/inspector-stats-pane branch June 21, 2026 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant