fix lint

Author: nirinchev

tests/integration/common/apiClient.test.ts

@@ -2,6 +2,7 @@ import { afterEach, beforeEach, describe, expect, it } from "vitest";
 import type { AccessToken } from "../../../src/common/atlas/apiClient.js";
 import { ApiClient } from "../../../src/common/atlas/apiClient.js";
 import { HTTPServerProxyTestSetup } from "../fixtures/httpsServerProxyTest.js";
+import { NullLogger } from "../../../src/common/logger.js";
 
 describe("ApiClient integration test", () => {
     describe(`atlas API proxy integration`, () => {
@@ -14,14 +15,17 @@ describe("ApiClient integration test", () => {
             await proxyTestSetup.listen();
 
             process.env.HTTP_PROXY = `https://localhost:${proxyTestSetup.httpsProxyPort}/`;
-            apiClient = new ApiClient({
-                baseUrl: `https://localhost:${proxyTestSetup.httpsServerPort}/`,
-                credentials: {
-                    clientId: "test-client-id",
-                    clientSecret: "test-client-secret",
+            apiClient = new ApiClient(
+                {
+                    baseUrl: `https://localhost:${proxyTestSetup.httpsServerPort}/`,
+                    credentials: {
+                        clientId: "test-client-id",
+                        clientSecret: "test-client-secret",
+                    },
+                    userAgent: "test-user-agent",
                 },
-                userAgent: "test-user-agent",
-            });
+                new NullLogger()
+            );
         });
 
         function withToken(accessToken: string, expired: boolean) {

tests/integration/telemetry.test.ts

@@ -4,6 +4,7 @@ import { Session } from "../../src/common/session.js";
 import { config } from "../../src/common/config.js";
 import nodeMachineId from "node-machine-id";
 import { describe, expect, it } from "vitest";
+import { CompositeLogger } from "../../src/common/logger.js";
 
 describe("Telemetry", () => {
     it("should resolve the actual machine ID", async () => {
@@ -14,6 +15,7 @@ describe("Telemetry", () => {
         const telemetry = Telemetry.create(
             new Session({
                 apiBaseUrl: "",
+                logger: new CompositeLogger(),
             }),
             config
         );

tests/unit/common/apiClient.test.ts

@@ -1,6 +1,7 @@
 import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import { ApiClient } from "../../../src/common/atlas/apiClient.js";
 import { CommonProperties, TelemetryEvent, TelemetryResult } from "../../../src/telemetry/types.js";
+import { NullLogger } from "../../../src/common/logger.js";
 
 describe("ApiClient", () => {
     let apiClient: ApiClient;
@@ -26,14 +27,17 @@ describe("ApiClient", () => {
     ];
 
     beforeEach(() => {
-        apiClient = new ApiClient({
-            baseUrl: "https://api.test.com",
-            credentials: {
-                clientId: "test-client-id",
-                clientSecret: "test-client-secret",
+        apiClient = new ApiClient(
+            {
+                baseUrl: "https://api.test.com",
+                credentials: {
+                    clientId: "test-client-id",
+                    clientSecret: "test-client-secret",
+                },
+                userAgent: "test-user-agent",
             },
-            userAgent: "test-user-agent",
-        });
+            new NullLogger()
+        );
 
         // @ts-expect-error accessing private property for testing
         apiClient.getAccessToken = vi.fn().mockResolvedValue("mockToken");

tests/unit/common/session.test.ts

@@ -2,6 +2,7 @@ import { beforeEach, describe, expect, it, vi } from "vitest";
 import { NodeDriverServiceProvider } from "@mongosh/service-provider-node-driver";
 import { Session } from "../../../src/common/session.js";
 import { config } from "../../../src/common/config.js";
+import { CompositeLogger } from "../../../src/common/logger.js";
 
 vi.mock("@mongosh/service-provider-node-driver");
 const MockNodeDriverServiceProvider = vi.mocked(NodeDriverServiceProvider);
@@ -12,6 +13,7 @@ describe("Session", () => {
         session = new Session({
             apiClientId: "test-client-id",
             apiBaseUrl: "https://api.test.com",
+            logger: new CompositeLogger(),
         });
 
         MockNodeDriverServiceProvider.connect = vi.fn().mockResolvedValue({} as unknown as NodeDriverServiceProvider);