fix: fix test

Author: blva

tests/unit/helpers/connectionOptions.test.ts

@@ -69,18 +69,17 @@ describe("Connection Options", () => {
             expect(result).toContain("appName=TestApp--test-device-id--unknown");
         });
 
-        it("should use deviceId utility when deviceId is not provided", async () => {
+        it("should use deviceId as unknown when deviceId is not provided", async () => {
             const connectionString = "mongodb://localhost:27017";
             const result = await setAppNameParamIfMissing({
                 connectionString,
                 components: {
                     appName: "TestApp",
                     clientName: "TestClient",
-                    deviceId: DeviceIdService.getInstance().getDeviceId(),
                 },
             });
 
-            expect(result).toContain("appName=TestApp--test-device-id--TestClient");
+            expect(result).toContain("appName=TestApp--unknown--TestClient");
         });
 
         it("should preserve other query parameters", async () => {

tests/unit/helpers/deviceId.test.ts

@@ -72,23 +72,6 @@ describe("Device ID Helper", () => {
             expect(MockGetDeviceId).toHaveBeenCalledTimes(1); // Still only called once
         });
 
-        it("should return cached device ID without triggering calculation", async () => {
-            const mockDeviceId = "test-device-id-123";
-            MockGetDeviceId.mockResolvedValue(mockDeviceId);
-
-            // Initialize after mocking
-            DeviceIdService.init(testLogger);
-
-            const deviceId = DeviceIdService.getInstance();
-
-            // First call to populate cache
-            await deviceId.getDeviceId();
-
-            // Get cached value without triggering calculation
-            const cachedValue = await deviceId.getDeviceId();
-            expect(cachedValue).toBe(mockDeviceId);
-        });
-
         it("should allow aborting calculation", async () => {
             MockGetDeviceId.mockImplementation((options) => {
                 // Simulate a long-running operation that can be aborted
@@ -145,6 +128,7 @@ describe("Device ID Helper", () => {
             const result = await deviceId.getDeviceId();
 
             expect(result).toBe("device-id");
+            expect(MockGetDeviceId).toHaveBeenCalledTimes(1);
         });
 
         it("should handle timeout error callback", async () => {