From e219f323cb7c3f88b74b2908ed00b42c3467a2b4 Mon Sep 17 00:00:00 2001 From: Rob Findley Date: Mon, 2 Jun 2025 15:42:28 +0000 Subject: [PATCH] internal/mcp: add missing testenv.NeedsExec Unbreak our WASM tests by correctly marking a test as testenv.NeedsExec. Change-Id: Ib976657e9b09acc6fc563f6fc26d6066fd64acf5 Reviewed-on: https://go-review.googlesource.com/c/tools/+/678035 Reviewed-by: Alan Donovan Auto-Submit: Robert Findley LUCI-TryBot-Result: Go LUCI --- mcp/cmd_test.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/mcp/cmd_test.go b/mcp/cmd_test.go index e4f33dd..79e89bf 100644 --- a/mcp/cmd_test.go +++ b/mcp/cmd_test.go @@ -13,6 +13,7 @@ import ( "github.com/google/go-cmp/cmp" "github.com/modelcontextprotocol/go-sdk/mcp" + "golang.org/x/tools/internal/testenv" ) const runAsServer = "_MCP_RUN_AS_SERVER" @@ -38,6 +39,8 @@ func runServer() { } func TestCmdTransport(t *testing.T) { + testenv.NeedsExec(t) + ctx, cancel := context.WithCancel(context.Background()) defer cancel()