Skip to content

Conversation

mclenhard
Copy link

Adds new e2e test that loads an MCP client, which in turn runs the server and processes the actual tool call. Afterwards, it then grades the response for correctness.

note: I'm the package author

@mclenhard mclenhard changed the title Feat/add e2e test and evals feat add e2e test and evals May 5, 2025
@mclenhard mclenhard changed the title feat add e2e test and evals feat: add e2e test and evals May 5, 2025
@mclenhard
Copy link
Author

Looks like the linter is failing, I'll get this updated later tonight or tomorrow morning

@mclenhard
Copy link
Author

Just fixed the lint errors

@mclenhard
Copy link
Author

Hey @jiachengxu! Any feedback ?

@christianh814 christianh814 requested a review from jiachengxu May 29, 2025 15:02
@jiachengxu
Copy link
Member

hi @mclenhard,

Thank you so much for your contribution! We are still evaluating different test frameworks for the mcp server, and will come back to this PR later when we decide which framework we are going to use. Thank you for write the test library for the open source community and contribution to this repo!

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.

2 participants