-
Notifications
You must be signed in to change notification settings - Fork 354
Add e2e tests #354
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: main
Are you sure you want to change the base?
Add e2e tests #354
Conversation
…move console log collection for now
@@ -22,12 +22,13 @@ | |||
"cli" | |||
], | |||
"scripts": { | |||
"clean": "rimraf ./node_modules ./client/node_modules ./server/node_modules ./dist ./client/dist ./server/dist ./package-lock.json && npm install", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added this since I was running into some complications with needing to do this manually, I can remove it before merging.
@@ -49,21 +58,28 @@ | |||
"zod": "^3.23.8" | |||
}, | |||
"devDependencies": { | |||
"@modelcontextprotocol/server-everything": "2025.4.25", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some of these dependency updates are probably from a previous attempt on same branch - main ones to pay attention to are importing server-everything and adding Playwright to dev dependences.
Motivation and Context
Opening as a draft for feedback on this general approach. There's a few configs that need to get adjusted or cleaned up here and obviously the oauth part is still mostly
// TODO
.Will plan to update to include Streamable Http once we can import the latest changes from the published package.
How Has This Been Tested?
To run these tests I've been doing the following:
In one terminal, run Inspector locally
In another terminal, run one of the transport-specific test commands, eg:
Breaking Changes
None
Types of changes
Checklist