fix(eval): bump pre-push-gate-governance bats count 50->51 after ce9d42f8

Commit ce9d42f8 (fix(pre-push): keep fast two-pass advisory scoped) added
a 51st BATS test ("pre-push-gate.sh --fast two-pass keeps pass 2 fast and
skips evals for non-eval changes") but did not update the contract-canary
expectation in evals/agentops-core/pre-push-gate-governance.json, which
still pinned "1..50". Result: agentops-core.pre-push-gate-governance
canary failed locally with:

  pre-push-bats-suite: stdout does not contain "1..50"

bats tests/scripts/pre-push-gate.bats reports 1..51 with all tests
passing, so the count is correct; the eval pin just needs to track it.

Validated:

- bats tests/scripts/pre-push-gate.bats -> 51/51 pass, "1..51" first line
- bash scripts/pre-push-gate.sh --fast -> Pass 1 PASSED, Pass 2 PASSED
- contract canaries gate flips from FAIL -> ok
This commit is contained in:
Boden Fuller
2026-05-07 22:46:12 -04:00
parent c587b361e4
commit bd104270aa
@@ -40,7 +40,7 @@
},
"expectations": [
{"type": "exit_code", "value": 0},
{"type": "stdout_contains", "value": "1..50"},
{"type": "stdout_contains", "value": "1..51"},
{"type": "stdout_contains", "value": "ok 6 pre-push-gate.sh passes when no Go changes"},
{"type": "stdout_contains", "value": "ok 21 pre-push-gate.sh clears GIT env for skill CLI snippets"},
{"type": "stdout_contains", "value": "ok 22 pre-push-gate.sh clears GIT env for CLI docs parity"},