s1d3sw1ped/robinhood-agentic-mcp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

docs: live MCP tests are out of scope, not deferred

Browse Source 
No //go:build live smoke. Tests stop at rhntest and env-token Login.
This commit is contained in:
Justin Harms
2026-09-01 10:56:58 -05:00
parent 6600aa0eff
commit 4dd586eea1
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
docs/superpowers/specs/2026-09-01-robinhood-agentic-mcp-design.md
+3 -2
View File
@@ -23,6 +23,7 @@ A standalone Go module at `/fast/projects/golang/robinhood-agentic-mcp` that:
- Codegen from `tools/list`. - Codegen from `tools/list`.
- Proxying or re-hosting the MCP as a server. - Proxying or re-hosting the MCP as a server.
- Banking / credit-card MCP (`agent.robinhood.com` non-trading endpoints). - Banking / credit-card MCP (`agent.robinhood.com` non-trading endpoints).
- Live MCP or browser-OAuth tests (`//go:build live` smokes included). Tests stop at `rhntest` and env-token `Login`.
## Operator contract ## Operator contract
@@ -363,7 +364,7 @@ Required tests:
- `ToolError` is `errors.As`-able. - `ToolError` is `errors.As`-able.
- Coverage: frozen `testdata/tools.json` from live `tools/list` must have a method for every name. - Coverage: frozen `testdata/tools.json` from live `tools/list` must have a method for every name.
No live-MCP or browser-OAuth in default `go test`. An optional `//go:build live` smoke is out of v1. No live-MCP or browser-OAuth tests, including `//go:build live` smokes. Not deferred — not in this library.
## Package map ## Package map
@@ -378,7 +379,7 @@ No live-MCP or browser-OAuth in default `go test`. An optional `//go:build live`
## Success criteria ## Success criteria
- `go test ./...` passes with no live network (httptest mock only). - `go test ./...` passes with no live network (httptest mock only). There is no live-MCP test target.
- Typed methods work with an injected `Caller` **and** round-trip through `rhntest`. - Typed methods work with an injected `Caller` **and** round-trip through `rhntest`.
- `Connect`/`Login` accept `Config.Name` / `Config.Version`; empty uses library defaults. - `Connect`/`Login` accept `Config.Name` / `Config.Version`; empty uses library defaults.
- `Login` with `ROBINHOOD_ACCESS_TOKEN` writes `0600` tokens. - `Login` with `ROBINHOOD_ACCESS_TOKEN` writes `0600` tokens.