fix: add bats coverage for --backend <value> space-separated form (#825)
All checks were successful
ci/woodpecker/push/ci Pipeline was successful
ci/woodpecker/push/nomad-validate Pipeline was successful
ci/woodpecker/pr/ci Pipeline was successful
ci/woodpecker/pr/nomad-validate Pipeline was successful
ci/woodpecker/pr/secret-scan Pipeline was successful
ci/woodpecker/pr/smoke-init Pipeline was successful
All checks were successful
ci/woodpecker/push/ci Pipeline was successful
ci/woodpecker/push/nomad-validate Pipeline was successful
ci/woodpecker/pr/ci Pipeline was successful
ci/woodpecker/pr/nomad-validate Pipeline was successful
ci/woodpecker/pr/secret-scan Pipeline was successful
ci/woodpecker/pr/smoke-init Pipeline was successful
The bin/disinto flag loop has separate cases for `--backend value` (space-separated) and `--backend=value`; a regression in either would silently route to the docker default path. Per the "stub-first dispatch" lesson, silent misrouting during a migration is the worst failure mode — covering both forms closes that gap. Also triggers a retry of the smoke-init pipeline step, which hit a known Forgejo branch-indexing flake on pipeline #913 (same flake cleared on retry for PR #829 pipelines #906 → #908); unrelated to the nomad-validate changes, which went all-green in #913. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
parent
e5c41dd502
commit
14c67f36e6
1 changed files with 13 additions and 0 deletions
|
|
@ -78,6 +78,19 @@ setup_file() {
|
||||||
[[ "$output" == *"── Dry-run: intended actions ────"* ]]
|
[[ "$output" == *"── Dry-run: intended actions ────"* ]]
|
||||||
}
|
}
|
||||||
|
|
||||||
|
# ── Flag syntax: --flag=value vs --flag value ────────────────────────────────
|
||||||
|
|
||||||
|
# Both forms must work. The bin/disinto flag loop has separate cases for
|
||||||
|
# `--backend value` and `--backend=value`; a regression in either would
|
||||||
|
# silently route to the docker default, which is the worst failure mode
|
||||||
|
# for a mid-migration dispatcher ("loud-failing stub" lesson from S0.4).
|
||||||
|
@test "disinto init --backend nomad (space-separated) dispatches to nomad" {
|
||||||
|
run "$DISINTO_BIN" init placeholder/repo --backend nomad --dry-run
|
||||||
|
[ "$status" -eq 0 ]
|
||||||
|
[[ "$output" == *"nomad backend: default"* ]]
|
||||||
|
[[ "$output" == *"[dry-run] Step 1/9: install nomad + vault binaries"* ]]
|
||||||
|
}
|
||||||
|
|
||||||
# ── Flag validation ──────────────────────────────────────────────────────────
|
# ── Flag validation ──────────────────────────────────────────────────────────
|
||||||
|
|
||||||
@test "--backend=bogus is rejected with a clear error" {
|
@test "--backend=bogus is rejected with a clear error" {
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue