diff options
| author | Craig Jennings <c@cjennings.net> | 2026-07-20 23:33:02 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-07-20 23:33:02 -0500 |
| commit | cf211cd8dd8dd0ff18673c88a3eef2c63549d6d5 (patch) | |
| tree | 4051e651241dd4b13d1d57fc70ead7d4099b4ad1 /tests/installer-steps | |
| parent | 5ce5d6903b8178e3502f52fcc2723e9cc44766d5 (diff) | |
| download | archsetup-cf211cd8dd8dd0ff18673c88a3eef2c63549d6d5.tar.gz archsetup-cf211cd8dd8dd0ff18673c88a3eef2c63549d6d5.zip | |
fix(test): count a diagnose miss as a failed net scenario
The scenario_diagnose_expect else-branch printed its FAIL line but never forced a non-zero subshell exit. A scenario whose only failure was the diagnose check counted as passed, so the run printed "all scenarios passed" and exited 0 over a real net-doctor diagnosis regression.
The miss now sets a per-scenario rc that carries to the subshell exit. The fix and assert still run first, since the repair result is worth having either way.
The new harness drives the real script with stubbed ssh/rsync/jq and a controllable fake scenario, pinning all four check outcomes.
Diffstat (limited to 'tests/installer-steps')
0 files changed, 0 insertions, 0 deletions
