Skip to content

Refactor QA layers#141

Merged
kmolan merged 8 commits into
mainfrom
feature/refactor-QA-layers
Jul 14, 2026
Merged

Refactor QA layers#141
kmolan merged 8 commits into
mainfrom
feature/refactor-QA-layers

Conversation

@kmolan

@kmolan kmolan commented Jul 14, 2026

Copy link
Copy Markdown
Owner

What & why

Why:

The workspace has six overlapping QA layers — doctests (~112), tests/ (~24 files / ~6k lines / ~300 tests in 13 binaries), 15 examples/, criterion benches/ with hand-maintained .md accuracy tables, the tools/oracle golden-fixture crate, and the tools/embedded-smoke QEMU firmware — plus a demo surface split across two places: crates/multicalc/examples/ (headless, ships in the published tarball) and showcase/viz/ (the multicalc-viz live-Rerun crate). The layers themselves are each justified, but four problems make the setup confusing and expensive to scale.

What:

  1. De-deplicate test configs using a new crate multicalc-testkit
  2. embedded-smoke uses multicalc-testkit
  3. Consolidate all tests into one binary
  4. Create a single demos/ crate multicalc-demos. Move existing showcases and all examples into it.
  5. Update all documentation.

Checklist

  • CHANGELOG.md updated under ## [Unreleased] (required for behavior-facing changes)
  • cargo test + cargo clippy --all-targets clean locally
  • New public APIs have a doc example
  • No unwrap/expect/panic on library paths (typed errors instead)

@kmolan

kmolan commented Jul 14, 2026

Copy link
Copy Markdown
Owner Author

@rtmongold for visibility. I streamlined all the tests and QA layers to be more scalable. You will need to rebase on top for the RISC-V changes. Let me know if there's any confusions, sorry for the code churn!

@kmolan kmolan merged commit f8db4ac into main Jul 14, 2026
15 checks passed
@kmolan kmolan deleted the feature/refactor-QA-layers branch July 14, 2026 18:05
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