Skip to content
This repository was archived by the owner on May 21, 2026. It is now read-only.

test: add converter binary smoke test#33

Merged
N283T merged 1 commit into
mainfrom
test/converter-smoke
Apr 26, 2026
Merged

test: add converter binary smoke test#33
N283T merged 1 commit into
mainfrom
test/converter-smoke

Conversation

@N283T

@N283T N283T commented Apr 26, 2026

Copy link
Copy Markdown
Owner

Summary

Closes #30.

A new `zig build converter-smoke` step runs the installed converter binary on `test_data/frame0.trr` doing a full `trr_to_xtc → xtc_to_trr` round-trip. This exercises the full `std.process.Init` plumbing (gpa, io, args, arena) end-to-end — previously the converter binary was built but never run as part of CI.

CI runs it after validation tests.

Test plan

Closes #30.

A new 'zig build converter-smoke' step runs the installed
converter binary trr_to_xtc -> xtc_to_trr on test_data/frame0.trr,
exercising the full std.process.Init plumbing
(gpa, io, args, arena) end-to-end. CI runs it after validation.
@N283T N283T force-pushed the test/converter-smoke branch from 0b81320 to c559f53 Compare April 26, 2026 08:45
@N283T N283T merged commit f9d8e5b into main Apr 26, 2026
2 checks passed
@N283T N283T deleted the test/converter-smoke branch April 26, 2026 08:46
@N283T N283T mentioned this pull request Apr 26, 2026
2 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test: add converter smoke test that exercises std.process.Init

1 participant