carlos / platform Public

test: the store usage assertion never learned about `scan`

CI has been red on both scan branches since df947ec, and the local
"five root-package tests fail for an environmental reason" note is
exactly what hid it: the sixth failure in that package looked like more
of the same and was never read.

It is not environmental. `carlos store` grew a `scan` sub-verb and its
usage line grew with it; the test still asserted the four-verb string.
The assertion was right to exist — it is the one place that notices the
usage block and the dispatch table drifting apart — it just needed the
verb that was added.

The second assertion in the same test DOES fail environmentally, on
this machine only, because the checkout sits under /home/paulca and
`pick()` finds a real ~/.carlos/config. CI passes it. That is the
distinction the note should have drawn and did not.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PEkJh1ESqiA84quAHJV4K6

Paul Campbell pushed by paul@keymail.dev 1 month ago 93e21c18f6dbe45cd38b90bb661e4b309b7a664c parent 39e9b0d

1 file changed, +1 −1

store_test.go +1 −1

diff --git a/store_test.go b/store_test.go

index b7b2decb0..2cfad43a8 100644

--- a/store_test.go

+++ b/store_test.go

@@ -521,7 +521,7 @@ func TestStoreVerbDispatchAndRefusals(t *testing.T) {

if err == nil {

t.Fatalf("carlos %v exited 0:\n%s", args, out)

}

- if !strings.Contains(out, "usage: carlos store <create|status|grant|rotate>") {

+ if !strings.Contains(out, "usage: carlos store <create|status|grant|rotate|scan>") {

t.Fatalf("carlos %v = %q, want the store usage block", args, out)

}