Move the YAML node script tests into scripts/tests #122

Merged
Claude merged 1 commit from chore/yaml-test-folder into feat/yaml-nodes 2026-10-15 16:05:54 +00:00
Collaborator

YN-5. The two test files and the fixtures move to scripts/tests/ (with git mv). CI runs python3 -m unittest discover -s scripts/tests -t scripts; the -t scripts puts the generator on the import path with no extra code. build_yaml_nodes.sh stays put and now reads scripts/tests/fixtures/, which stay outside nodes/ so they never join the real-node uniqueness checks.

The fixtures are now block-style YAML. The S3 fixture's relay gets a toggle bind from its latching switch, and a new test generates it and asserts the output contains ToggleOnChange, so CI covers that path and not just that it compiles.

Tested: 26 Python tests pass locally with none skipped (was 25), and all three fixtures and the sample still build.

YN-5. The two test files and the fixtures move to `scripts/tests/` (with `git mv`). CI runs `python3 -m unittest discover -s scripts/tests -t scripts`; the `-t scripts` puts the generator on the import path with no extra code. `build_yaml_nodes.sh` stays put and now reads `scripts/tests/fixtures/`, which stay outside `nodes/` so they never join the real-node uniqueness checks. The fixtures are now block-style YAML. The S3 fixture's relay gets a `toggle` bind from its latching switch, and a new test generates it and asserts the output contains `ToggleOnChange`, so CI covers that path and not just that it compiles. Tested: 26 Python tests pass locally with none skipped (was 25), and all three fixtures and the sample still build.
Move the YAML node script tests into scripts/tests
All checks were successful
Build and test / build (pull_request) Successful in 5m8s
Build and test / release (pull_request) Has been skipped
5b7a36f344
Claude merged commit b3e41087ce into feat/yaml-nodes 2026-10-15 16:05:54 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
rob/CampervanControlSystems!122
No description provided.