test(helper): cover missing node row #17

Merged
ochenstarik-ui merged 1 commit from hermes/task3-b3r-missing-row-test into main 2026-08-09 10:56:13 +00:00
ochenstarik-ui commented 2026-08-09 10:51:50 +00:00 (Migrated from github.com)

Summary

Closes the sole LOW finding from the complete B-3/B-3R delivery review by restoring an explicit real-helper regression for a target Node entirely absent from the four-field nodes.tsv registry.

The contract asserts:

  • link-connect fails closed;
  • exit code is exactly 80;
  • stderr is exactly mesh.node-not-activated.

No production code changes.

Verification

  • bash -n tests/bootstrap/test-bootstrap-contract.sh
  • bash tests/bootstrap/test-bootstrap-contract.shBOOTSTRAP_CONTRACT=PASS
  • git diff --check

Physical three-server acceptance remains pending and is not affected by this test-only change.

## Summary Closes the sole LOW finding from the complete B-3/B-3R delivery review by restoring an explicit real-helper regression for a target Node entirely absent from the four-field `nodes.tsv` registry. The contract asserts: - `link-connect` fails closed; - exit code is exactly `80`; - stderr is exactly `mesh.node-not-activated`. No production code changes. ## Verification - `bash -n tests/bootstrap/test-bootstrap-contract.sh` - `bash tests/bootstrap/test-bootstrap-contract.sh` → `BOOTSTRAP_CONTRACT=PASS` - `git diff --check` Physical three-server acceptance remains pending and is not affected by this test-only change.
Sign in to join this conversation.
No reviewers
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: ochenstarik/server-monitor-manager#17
No description provided.