Skip to content

test: add MCP tools/call end-to-end test#137

Merged
patchloom-release[bot] merged 1 commit into
mainfrom
test/mcp-tools-call-e2e
Jun 18, 2026
Merged

test: add MCP tools/call end-to-end test#137
patchloom-release[bot] merged 1 commit into
mainfrom
test/mcp-tools-call-e2e

Conversation

@SebTardif

Copy link
Copy Markdown
Contributor

Extends the managed install e2e suite to call doc_set through
the MCP server and verify the file was modified on disk. This closes
the full loop: install binary, start MCP, initialize, list tools,
call a tool, confirm the side effect.

Also bumps the --version timeout from 5s to 15s to avoid
intermittent failures on cold binary starts.

257 tests, 0 failures, 91.48% coverage.

Extend the managed install e2e suite with a test that calls doc_set
through the MCP server and verifies the file was modified on disk.
This closes the loop: install binary, start MCP, initialize, list
tools, call a tool, and confirm the side effect.

Also bumps the --version timeout from 5s to 15s to avoid intermittent
failures on cold binary starts.

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
@patchloom-release patchloom-release Bot enabled auto-merge (squash) June 18, 2026 05:45
@patchloom-release patchloom-release Bot merged commit f43353a into main Jun 18, 2026
19 checks passed
@patchloom-release patchloom-release Bot deleted the test/mcp-tools-call-e2e branch June 18, 2026 05:47
patchloom-release Bot pushed a commit that referenced this pull request Jun 18, 2026
- Add RELEASE_NOTES.md to .vscodeignore so it is not shipped in the
  extension package (it leaked into the v0.0.4 vsix)
- Update patchloomCli.test.ts count in AGENTS.md from 35 to 36 to
  reflect the tools/call e2e test added in PR #137

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant