Skip to content

refactor(mcp): extract tool test modules - #532

Merged
xxx merged 1 commit into
masterfrom
refactor/mcp-tool-test-modules
Aug 26, 2026
Merged

refactor(mcp): extract tool test modules#532
xxx merged 1 commit into
masterfrom
refactor/mcp-tool-test-modules

Conversation

@xxx

@xxx xxx commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • extract the shared MCP tool test platform into tools/testutil.rs
  • extract the tools unit tests into tools/tests.rs
  • reduce tools.rs from 2,267 lines to 503 without changing production behavior

Validation

  • cargo fmt --all --check
  • cargo clippy --workspace --all-targets --locked -- -D warnings
  • cargo test --workspace --locked
  • verified the extracted modules preserve the original token stream apart from formatting

Review

Reviewed against origin/master for repository standards and the behavior-preserving refactor request. No findings.

@xxx
xxx merged commit 0ac69ff into master Aug 26, 2026
17 checks passed
@xxx
xxx deleted the refactor/mcp-tool-test-modules branch August 26, 2026 00:31
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