address comments: create default test config

Author: blva

tests/integration/helpers.ts

@@ -5,6 +5,7 @@ import { UserConfig } from "../../src/config.js";
 import { McpError } from "@modelcontextprotocol/sdk/types.js";
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { Session } from "../../src/session.js";
+import { config } from "../../src/config.js";
 
 interface ParameterInfo {
     name: string;
@@ -19,16 +20,16 @@ export interface IntegrationTest {
     mcpClient: () => Client;
     mcpServer: () => Server;
 }
+export const defaultTestConfig: UserConfig = {
+    ...config,
+    telemetry: "disabled",
+};
 
 export function setupIntegrationTest(getUserConfig: () => UserConfig): IntegrationTest {
     let mcpClient: Client | undefined;
     let mcpServer: Server | undefined;
-    let oldDoNotTrackValue: string | undefined;
 
     beforeAll(async () => {
-        // GET DO_NOT_TRACK value
-        oldDoNotTrackValue = process.env.DO_NOT_TRACK;
-        process.env.DO_NOT_TRACK = "1";
         const userConfig = getUserConfig();
         const clientTransport = new InMemoryTransport();
         const serverTransport = new InMemoryTransport();
@@ -79,13 +80,6 @@ export function setupIntegrationTest(getUserConfig: () => UserConfig): Integrati
 
         await mcpServer?.close();
         mcpServer = undefined;
-
-        // Reset DO_NOT_TRACK value
-        if (oldDoNotTrackValue !== undefined) {
-            process.env.DO_NOT_TRACK = oldDoNotTrackValue;
-        } else {
-            delete process.env.DO_NOT_TRACK;
-        }
     });
 
     const getMcpClient = () => {

tests/integration/server.test.ts

@@ -1,5 +1,4 @@
-import { expectDefined, setupIntegrationTest } from "./helpers.js";
-import { config } from "../../src/config.js";
+import { defaultTestConfig, expectDefined, setupIntegrationTest } from "./helpers.js";
 import { describeWithMongoDB } from "./tools/mongodb/mongodbHelpers.js";
 
 describe("Server integration test", () => {
@@ -16,15 +15,15 @@ describe("Server integration test", () => {
             });
         },
         () => ({
-            ...config,
+            ...defaultTestConfig,
             apiClientId: undefined,
             apiClientSecret: undefined,
         })
     );
 
     describe("with atlas", () => {
         const integration = setupIntegrationTest(() => ({
-            ...config,
+            ...defaultTestConfig,
             apiClientId: "test",
             apiClientSecret: "test",
         }));
@@ -59,7 +58,7 @@ describe("Server integration test", () => {
 
     describe("with read-only mode", () => {
         const integration = setupIntegrationTest(() => ({
-            ...config,
+            ...defaultTestConfig,
             readOnly: true,
             apiClientId: "test",
             apiClientSecret: "test",

tests/integration/tools/atlas/atlasHelpers.ts

@@ -1,15 +1,14 @@
 import { ObjectId } from "mongodb";
 import { Group } from "../../../../src/common/atlas/openapi.js";
 import { ApiClient } from "../../../../src/common/atlas/apiClient.js";
-import { setupIntegrationTest, IntegrationTest } from "../../helpers.js";
-import { config } from "../../../../src/config.js";
+import { setupIntegrationTest, IntegrationTest, defaultTestConfig } from "../../helpers.js";
 
 export type IntegrationTestFunction = (integration: IntegrationTest) => void;
 
 export function describeWithAtlas(name: string, fn: IntegrationTestFunction) {
     const testDefinition = () => {
         const integration = setupIntegrationTest(() => ({
-            ...config,
+            ...defaultTestConfig,
             apiClientId: process.env.MDB_MCP_API_CLIENT_ID,
             apiClientSecret: process.env.MDB_MCP_API_CLIENT_SECRET,
         }));

tests/integration/tools/mongodb/mongodbHelpers.ts

@@ -2,8 +2,8 @@ import { MongoCluster } from "mongodb-runner";
 import path from "path";
 import fs from "fs/promises";
 import { MongoClient, ObjectId } from "mongodb";
-import { getResponseContent, IntegrationTest, setupIntegrationTest } from "../../helpers.js";
-import { config, UserConfig } from "../../../../src/config.js";
+import { getResponseContent, IntegrationTest, setupIntegrationTest, defaultTestConfig } from "../../helpers.js";
+import { UserConfig } from "../../../../src/config.js";
 
 interface MongoDBIntegrationTest {
     mongoClient: () => MongoClient;
@@ -14,7 +14,7 @@ interface MongoDBIntegrationTest {
 export function describeWithMongoDB(
     name: string,
     fn: (integration: IntegrationTest & MongoDBIntegrationTest & { connectMcpClient: () => Promise<void> }) => void,
-    getUserConfig: (mdbIntegration: MongoDBIntegrationTest) => UserConfig = () => config,
+    getUserConfig: (mdbIntegration: MongoDBIntegrationTest) => UserConfig = () => defaultTestConfig,
     describeFn = describe
 ) {
     describeFn(name, () => {