Skip to content

test(v3): add real rig YAML fixtures from maDisplayTools (for #91/#89)#92

Merged
mbreiser merged 1 commit into
mainfrom
chore/rig-fixtures-for-91
Jun 2, 2026
Merged

test(v3): add real rig YAML fixtures from maDisplayTools (for #91/#89)#92
mbreiser merged 1 commit into
mainfrom
chore/rig-fixtures-for-91

Conversation

@mbreiser
Copy link
Copy Markdown
Contributor

@mbreiser mbreiser commented Jun 2, 2026

Ready test inputs for the rig-aware plugin work (#91) and the imported-plugin-name bug (#89).

Three rig configs copied verbatim from maDisplayTools (main @ 4592cea, configs/rigs/) into tests/fixtures/rigs/ — not hand-written:

File Why
test_rig_1.yaml Richest — backlight + camera enabled with full config + type strings (happy path).
example_rig.yaml Schema variation — no type, com_port, "BIAS" casing (exercises the tolerant rig→class mapping).
test_rig_2.yaml Minimal — all plugins disabled ("nothing enabled" path).

Plus a README.md documenting provenance + intent. Fixtures only — inert inputs, no suite reads them; npm test green (arena 10 · v2 137 · v3 592).

🤖 Generated with Claude Code

Verbatim copies of three rig configs from maDisplayTools (main @ 4592cea,
configs/rigs/) under tests/fixtures/rigs/, as ready test inputs for the
rig-aware plugin work (#91) and the imported-plugin-name bug (#89):

- test_rig_1.yaml  — richest: backlight + camera enabled with full config + type
- example_rig.yaml — schema variation: no type, com_port, "BIAS" casing
- test_rig_2.yaml  — minimal: all plugins disabled

README documents provenance + why each is included. Fixtures only — inert test
inputs, do not affect any suite (npm test green: arena 10 · v2 137 · v3 592).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mbreiser mbreiser merged commit fb7e940 into main Jun 2, 2026
1 check passed
@mbreiser mbreiser deleted the chore/rig-fixtures-for-91 branch June 2, 2026 10:52
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