Skip to content

Fix everything-server-based tests#756

Merged
chemicL merged 1 commit intomainfrom
everything-server-tests-fix
Jan 20, 2026
Merged

Fix everything-server-based tests#756
chemicL merged 1 commit intomainfrom
everything-server-tests-fix

Conversation

@chemicL
Copy link
Member

@chemicL chemicL commented Jan 19, 2026

Use a fixed npx @modelcontextprotocol/server-everything version.
Replace tzolov/mcp-everything-server Docker image with node:lts-alpine + npx, handle HTTP 202, fix test assertions

Motivation and Context

The recent rollout of everything-server broke our tests which take the latest version from the node registry.
This PR unifies the everything-server usage - the Testcontainers Docker setup uses the same version as the STDIO npx-based tests and no longer relies on tzolov/mcp-everything-server.

Unfortunately, there are more problems with the latest version of everything-server (@modelcontextprotocol/server-everything@2026.1.14) so the one used for tests is 2025.12.18.

How Has This Been Tested?

Automatic test suite.

Breaking Changes

No.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

No.

…e + npx, handle HTTP 202, fix test assertions

Signed-off-by: Dariusz Jędrzejczyk <dariusz.jedrzejczyk@broadcom.com>
@chemicL chemicL added this to the 0.18.0 milestone Jan 19, 2026
@chemicL chemicL added the type/testing Addresses tests only label Jan 19, 2026
@chemicL chemicL merged commit af65356 into main Jan 20, 2026
1 check passed
@chemicL chemicL deleted the everything-server-tests-fix branch January 20, 2026 15:09
tzolov pushed a commit that referenced this pull request Jan 22, 2026
* Pin `npx @modelcontextprotocol/server-everything` version to `2025.12.18`.
* Replace `tzolov/mcp-everything-server` Docker image with `node:lts-alpine` + `npx` command.
* Handle HTTP 202 special case.
* Fix test assertions.

The recent rollout of everything-server broke integration tests which take the latest version from the node registry.
This PR unifies the everything-server usage - the Testcontainers Docker setup uses the same version as the STDIO npx-based tests and no longer relies on tzolov/mcp-everything-server.

Signed-off-by: Dariusz Jędrzejczyk <dariusz.jedrzejczyk@broadcom.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/testing Addresses tests only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments