-
Notifications
You must be signed in to change notification settings - Fork 0
feat(testing): add MCP and CLI test harnesses #80
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: p3-17/testing/fixtures
Are you sure you want to change the base?
Conversation
cb7305d to
563d1cc
Compare
6622398 to
9806a74
Compare
563d1cc to
d5b1567
Compare
9806a74 to
d96c800
Compare
d96c800 to
e9a7e76
Compare
d5b1567 to
a57045a
Compare
a57045a to
4d06f54
Compare
e9a7e76 to
119f070
Compare
Greptile Summary
|
| Filename | Overview |
|---|---|
packages/testing/src/index.ts |
New entry point exporting all testing utilities with comprehensive JSDoc documentation |
packages/testing/src/mcp-harness.ts |
New MCP test harness providing tool invocation, search, and fixture loading capabilities |
packages/testing/src/cli-harness.ts |
New CLI test harness for executing commands and capturing stdout/stderr/exit codes |
Confidence score: 4/5
- This PR is safe to merge with high confidence as it implements well-structured testing infrastructure
- Score reflects comprehensive test coverage and follows established patterns, but deducted one point due to test files being written before implementation (TDD approach means some tests may initially fail)
- Pay close attention to
packages/testing/src/__tests__/fixtures.test.tswhich references fixture files that may not exist yet
Sequence Diagram
sequenceDiagram
participant User
4d06f54 to
a33da83
Compare
119f070 to
0161544
Compare
|
Reviewed the CLI harness concerns: stdin is closed immediately and signal-based exits are mapped to 128+signal; tests passed in the pre-push run. |
|
Verified CLI harness now closes stdin to allow EOF and maps signal termination to non-zero exit codes (128+signal). Restacked/checked and submitted with gt. |
a33da83 to
cddc577
Compare
0161544 to
12987e1
Compare
|
Restacked after downstack update (formatRelative test stabilization); no additional changes in this PR. |
12987e1 to
b8a6bab
Compare
cddc577 to
ac76953
Compare
|
CLI harness already closes stdin immediately and maps signal terminations to non-zero (128+signal) exit codes. Resubmitted after restack. |

Add transport harnesses for integration testing:
Co-Authored-By: Claude Opus 4.5 noreply@anthropic.com
Resolves #47