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

Update ToolsTab tests to require JSON fields and adjust default value generation for optional types

Browse source
This commit is contained in:
olaservo 2025-09-27 21:17:17 -07:00
commit a11ee99818
1 changed files with 48 additions and 19 deletions
Download patch file Download diff file Expand all files Collapse all files

25
client/src/components/__tests__/ToolsTab.test.tsx
View file

@ -644,6 +644,7 @@ describe("ToolsTab", () => {
description: "Tool with JSON parameters",
inputSchema: {
type: "object" as const,
required: ["config", "data"], // Make them required so they render as form fields
properties: {
config: {
type: "object" as const,
@ -693,15 +694,16 @@ describe("ToolsTab", () => {
callTool: mockCallTool,
});
// Find JSON editor textareas
// Find JSON editor textareas (should have one for each required field: config and data)
const textareas = screen.getAllByRole("textbox");
expect(textareas.length).toBe(2);
// Enter valid JSON in the first textarea
// Enter valid JSON in each textarea
fireEvent.change(textareas[0], {
target: {
value:
'{ "config": { "setting": "value" }, "data": ["item1", "item2"] }',
},
target: { value: '{ "setting": "value" }' },
});
fireEvent.change(textareas[1], {
target: { value: '["item1", "item2"]' },
});
// Wait for debounced updates
@ -799,9 +801,16 @@ describe("ToolsTab", () => {
});
const textareas = screen.getAllByRole("textbox");
expect(textareas.length).toBe(2);
// Clear the textarea (empty JSON should be valid)
fireEvent.change(textareas[0], { target: { value: "" } });
// Clear both textareas (empty JSON should be valid)
fireEvent.change(textareas[0], { target: { value: "{}" } });
fireEvent.change(textareas[1], { target: { value: "[]" } });
// Wait for debounced updates
await act(async () => {
await new Promise((resolve) => setTimeout(resolve, 350));
});
// Try to run the tool
const runButton = screen.getByRole("button", { name: /run tool/i });