chore: lint and test fix

Author: blva

tests/integration/tools/atlas/atlasHelpers.ts

@@ -15,7 +15,7 @@ export function describeWithAtlas(name: string, fn: IntegrationTestFunction): Su
         }));
 
         describe(name, () => {
-            fn(integration);
+            fn(clea);
         });
     };
 

tests/integration/transports/stdio.test.ts

@@ -2,6 +2,8 @@ import { describe, expect, it, beforeAll, afterAll } from "vitest";
 import { StdioClientTransport } from "@modelcontextprotocol/sdk/client/stdio.js";
 import { Client } from "@modelcontextprotocol/sdk/client/index.js";
 import { describeWithMongoDB } from "../tools/mongodb/mongodbHelpers.js";
+import { DeviceIdService } from "../../../src/helpers/deviceId.js";
+import { CompositeLogger } from "../../../src/common/logger.js";
 
 describeWithMongoDB("StdioRunner", (integration) => {
     describe("client connects successfully", () => {
@@ -20,6 +22,7 @@ describeWithMongoDB("StdioRunner", (integration) => {
                 name: "test",
                 version: "0.0.0",
             });
+            DeviceIdService.init(new CompositeLogger());
             await client.connect(transport);
         });
 

tests/integration/transports/streamableHttp.test.ts

@@ -3,6 +3,7 @@ import { Client } from "@modelcontextprotocol/sdk/client/index.js";
 import { StreamableHTTPClientTransport } from "@modelcontextprotocol/sdk/client/streamableHttp.js";
 import { describe, expect, it, beforeAll, afterAll } from "vitest";
 import { config } from "../../../src/common/config.js";
+import { DeviceIdService } from "../../../src/helpers/deviceId.js";
 
 describe("StreamableHttpRunner", () => {
     let runner: StreamableHttpRunner;
@@ -15,6 +16,7 @@ describe("StreamableHttpRunner", () => {
         config.telemetry = "disabled";
         config.loggers = ["stderr"];
         runner = new StreamableHttpRunner(config);
+        DeviceIdService.init(runner.logger);
         await runner.start();
     });
 

tests/unit/helpers/deviceId.test.ts

@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-unsafe-assignment */
 import { describe, expect, it, vi, beforeEach, afterEach } from "vitest";
 import { DeviceIdService } from "../../../src/helpers/deviceId.js";
 import { getDeviceId } from "@mongodb-js/device-id";
@@ -184,9 +183,6 @@ describe("Device ID Helper", () => {
         });
 
         it("should handle machine ID generation failure", async () => {
-            // Mock getMachineId to throw an error
-            mockGetMachineId = vi.fn().mockRejectedValue(new Error("Machine ID generation failed"));
-
             MockGetDeviceId.mockRejectedValue(new Error("Device ID failed"));
 
             // Initialize after mocking

tests/unit/resources/common/debug.test.ts

@@ -6,6 +6,7 @@ import { config } from "../../../../src/common/config.js";
 import { CompositeLogger } from "../../../../src/common/logger.js";
 import { ConnectionManager } from "../../../../src/common/connectionManager.js";
 import { ExportsManager } from "../../../../src/common/exportsManager.js";
+import { DeviceIdService } from "../../../../src/helpers/deviceId.js";
 
 describe("debug resource", () => {
     const logger = new CompositeLogger();
@@ -21,6 +22,7 @@ describe("debug resource", () => {
 
     beforeEach(() => {
         debugResource = new DebugResource(session, config, telemetry);
+        DeviceIdService.init(logger);
     });
 
     it("should be connected when a connected event happens", () => {