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

test(test-servers): export MCP_APP_DEMO_URI, dedupe in integration test

Browse source 
Address code-review DRY note on PR #1587: MCP_APP_DEMO_URI was defined
in both the fixture and the integration test. Export it from the fixture
and import it in the test so the two stay in lockstep.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01S3fTN8H3R8YV4yUGvZjYnX
This commit is contained in:
cliffhall 2026-07-01 19:12:44 -04:00
commit e5dbdca65c
2 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

3
clients/web/src/test/integration/mcp/mcp-app-demo.test.ts
View file

@ -7,11 +7,10 @@ import {
createTestServerInfo,
createMcpAppDemoTool,
createMcpAppDemoResource,
MCP_APP_DEMO_URI,
} from "@modelcontextprotocol/inspector-test-server";
import type { Tool } from "@modelcontextprotocol/sdk/types.js";
const MCP_APP_DEMO_URI = "ui://demo/widget.html";
/**
* Exercises the `mcp_app_demo` preset added in #1557 end-to-end: it confirms the
* `_meta` plumbing through the composable test server's tool/resource

3
test-servers/src/test-server-fixtures.ts
View file

@ -754,7 +754,8 @@ export function createArgsPrompt(
};
}
const MCP_APP_DEMO_URI = "ui://demo/widget.html";
/** Canonical URI for the {@link createMcpAppDemoResource} UI resource, referenced by {@link createMcpAppDemoTool}'s `_meta.ui.resourceUri`. Exported so tests can assert against it without redefining the literal. */
export const MCP_APP_DEMO_URI = "ui://demo/widget.html";
/**
* Minimal MCP App widget that exercises the host-side UI protocol surface