modelcontextprotocol/java-sdk
Watch
1
0
Fork
You've already forked java-sdk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(tests): Failed to start process with command npx on Windows (#85)

Browse source 
* fix(tests): Failed to start process with command npx on Windows platform while running mvn test
This commit is contained in:
codeboyzz 2025-03-29 13:08:23 +08:00 • committed by GitHub
commit 79ec5b5ed1
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 22 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

14
mcp/src/test/java/io/modelcontextprotocol/client/StdioMcpAsyncClientTests.java
View file

@ -22,9 +22,17 @@ class StdioMcpAsyncClientTests extends AbstractMcpAsyncClientTests {
@Override @Override
protected McpClientTransport createMcpTransport() { protected McpClientTransport createMcpTransport() {
ServerParameters stdioParams = ServerParameters.builder("npx") ServerParameters stdioParams;
.args("-y", "@modelcontextprotocol/server-everything", "dir") if (System.getProperty("os.name").toLowerCase().contains("win")) {
.build(); stdioParams = ServerParameters.builder("cmd.exe")
.args("/c", "npx.cmd", "-y", "@modelcontextprotocol/server-everything", "dir")
.build();
}
else {
stdioParams = ServerParameters.builder("npx")
.args("-y", "@modelcontextprotocol/server-everything", "dir")
.build();
}
return new StdioClientTransport(stdioParams); return new StdioClientTransport(stdioParams);
} }

15
mcp/src/test/java/io/modelcontextprotocol/client/StdioMcpSyncClientTests.java
View file

@ -30,10 +30,17 @@ class StdioMcpSyncClientTests extends AbstractMcpSyncClientTests {
@Override @Override
protected McpClientTransport createMcpTransport() { protected McpClientTransport createMcpTransport() {
ServerParameters stdioParams = ServerParameters.builder("npx") ServerParameters stdioParams;
.args("-y", "@modelcontextprotocol/server-everything", "dir") if (System.getProperty("os.name").toLowerCase().contains("win")) {
.build(); stdioParams = ServerParameters.builder("cmd.exe")
.args("/c", "npx.cmd", "-y", "@modelcontextprotocol/server-everything", "dir")
.build();
}
else {
stdioParams = ServerParameters.builder("npx")
.args("-y", "@modelcontextprotocol/server-everything", "dir")
.build();
}
return new StdioClientTransport(stdioParams); return new StdioClientTransport(stdioParams);
} }