diff --git a/test/labels-triggers.test.sh b/test/labels-triggers.test.sh index c941e67..14847c5 100644 --- a/test/labels-triggers.test.sh +++ b/test/labels-triggers.test.sh @@ -81,9 +81,13 @@ check "the trigger dispatches the sweep caller by name" 0 \ grep -F '/dispatches' "$REUSABLE" check "...never bootstrapping" 0 'bootstrap: "no"' \ grep -F 'bootstrap' "$REUSABLE" -# shellcheck disable=SC2016 # $1 expands in the nested bash, not here -check "the trigger dispatch is never silenced with || true" 1 "" \ - bash -c 'grep -F "gh workflow run" "$1" | grep -qF "|| true"' _ "$REUSABLE" +# The never-silenced invariant moved to test/labels-dispatch.test.sh, where it +# is BEHAVIOURAL: a curl that dies at the transport must fail the extracted +# step, plus a code-aware no-`|| true` guard on the step itself. The check that +# lived here grepped the `gh workflow run` line #205 removed, so after the port +# it passed on every implementation including one that swallows a failed POST — +# a green assertion whose name claimed an invariant its implementation could +# not observe (@codex-reviewer-andresmgsl, #213 review). check "the sweep caller filename input defaults to labels-sweep.yml" 0 \ "default: labels-sweep.yml" grep -F 'default: labels-sweep.yml' "$REUSABLE" # the dogfood callers wear the split: the event caller names its deviant