PR feedback

Author: nirinchev

tests/integration/tools/mongodb/metadata/connect.test.ts

@@ -27,18 +27,18 @@ describe("Connect tool", () => {
     });
 
     it("should have correct metadata", async () => {
-        const tools = await client.listTools();
-        const connectTool = tools.tools.find((tool) => tool.name === "connect");
+        const { tools } = await client.listTools();
+        const connectTool = tools.find((tool) => tool.name === "connect")!;
         expect(connectTool).toBeDefined();
-        expect(connectTool!.description).toBe("Connect to a MongoDB instance");
-        expect(connectTool!.inputSchema.type).toBe("object");
-        expect(connectTool!.inputSchema.properties).toBeDefined();
+        expect(connectTool.description).toBe("Connect to a MongoDB instance");
+        expect(connectTool.inputSchema.type).toBe("object");
+        expect(connectTool.inputSchema.properties).toBeDefined();
 
-        const propertyNames = Object.keys(connectTool!.inputSchema.properties!);
+        const propertyNames = Object.keys(connectTool.inputSchema.properties!);
         expect(propertyNames).toHaveLength(1);
         expect(propertyNames[0]).toBe("connectionStringOrClusterName");
 
-        const connectionStringOrClusterNameProp = connectTool!.inputSchema.properties![propertyNames[0]] as {
+        const connectionStringOrClusterNameProp = connectTool.inputSchema.properties![propertyNames[0]] as {
             type: string;
             description: string;
         };

tests/integration/tools/mongodb/metadata/listDatabases.test.ts

@@ -27,14 +27,14 @@ describe("listDatabases tool", () => {
     });
 
     it("should have correct metadata", async () => {
-        const tools = await client.listTools();
-        const listDatabases = tools.tools.find((tool) => tool.name === "list-databases");
+        const { tools } = await client.listTools();
+        const listDatabases = tools.find((tool) => tool.name === "list-databases")!;
         expect(listDatabases).toBeDefined();
-        expect(listDatabases!.description).toBe("List all databases for a MongoDB connection");
-        expect(listDatabases!.inputSchema.type).toBe("object");
-        expect(listDatabases!.inputSchema.properties).toBeDefined();
+        expect(listDatabases.description).toBe("List all databases for a MongoDB connection");
+        expect(listDatabases.inputSchema.type).toBe("object");
+        expect(listDatabases.inputSchema.properties).toBeDefined();
 
-        const propertyNames = Object.keys(listDatabases!.inputSchema.properties!);
+        const propertyNames = Object.keys(listDatabases.inputSchema.properties!);
         expect(propertyNames).toHaveLength(0);
     });