Skip to content

Commit d8bf3f4

Browse files
committed
fix: integration tests
1 parent 802a363 commit d8bf3f4

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

tests/helpers.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,8 @@ export async function createMcpSseClient(
2323
if (actors) {
2424
url.searchParams.append('actors', actors.join(','));
2525
}
26-
if (enableAddingActors) {
27-
url.searchParams.append('enableAddingActors', 'true');
26+
if (enableAddingActors !== undefined) {
27+
url.searchParams.append('enableAddingActors', enableAddingActors.toString());
2828
}
2929

3030
const transport = new SSEClientTransport(
@@ -59,8 +59,8 @@ export async function createMcpStreamableClient(
5959
if (actors) {
6060
url.searchParams.append('actors', actors.join(','));
6161
}
62-
if (enableAddingActors) {
63-
url.searchParams.append('enableAddingActors', 'true');
62+
if (enableAddingActors !== undefined) {
63+
url.searchParams.append('enableAddingActors', enableAddingActors.toString());
6464
}
6565

6666
const transport = new StreamableHTTPClientTransport(
@@ -94,8 +94,8 @@ export async function createMcpStdioClient(
9494
if (actors) {
9595
args.push('--actors', actors.join(','));
9696
}
97-
if (enableAddingActors) {
98-
args.push('--enable-adding-actors');
97+
if (enableAddingActors !== undefined) {
98+
args.push('--enable-adding-actors', enableAddingActors.toString());
9999
}
100100
const transport = new StdioClientTransport({
101101
command: 'node',

tests/integration/suite.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,11 +84,12 @@ export function createIntegrationTestsSuite(
8484
it('should list all default tools and default Actors', async () => {
8585
const client = await createClientFn();
8686
const tools = await client.listTools();
87-
expect(tools.tools.length).toEqual(defaultTools.length + defaults.actors.length);
87+
expect(tools.tools.length).toEqual(defaultTools.length + defaults.actors.length + addRemoveTools.length);
8888

8989
const names = getToolNames(tools);
9090
expectToolNamesToContain(names, DEFAULT_TOOL_NAMES);
9191
expectToolNamesToContain(names, DEFAULT_ACTOR_NAMES);
92+
expectToolNamesToContain(names, addRemoveTools.map((tool) => tool.tool.name));
9293
await client.close();
9394
});
9495

0 commit comments

Comments
 (0)