Skip to content

Commit a95af7b

Browse files
committed
disable telemetry for all tests
1 parent a06c460 commit a95af7b

File tree

2 files changed

+12
-10
lines changed

2 files changed

+12
-10
lines changed

tests/integration/helpers.ts

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,12 @@ export interface IntegrationTest {
2323
export function setupIntegrationTest(getUserConfig: () => UserConfig): IntegrationTest {
2424
let mcpClient: Client | undefined;
2525
let mcpServer: Server | undefined;
26+
let oldDoNotTrackValue: string | undefined;
2627

2728
beforeAll(async () => {
29+
// GET DO_NOT_TRACK value
30+
oldDoNotTrackValue = process.env.DO_NOT_TRACK;
31+
process.env.DO_NOT_TRACK = "1";
2832
const userConfig = getUserConfig();
2933
const clientTransport = new InMemoryTransport();
3034
const serverTransport = new InMemoryTransport();
@@ -51,25 +55,18 @@ export function setupIntegrationTest(getUserConfig: () => UserConfig): Integrati
5155
apiClientSecret: userConfig.apiClientSecret,
5256
});
5357

54-
userConfig.telemetry = "disabled";
5558
mcpServer = new Server({
5659
session,
5760
userConfig,
5861
mcpServer: new McpServer({
5962
name: "test-server",
60-
version: "1.2.3",
63+
version: "5.2.3",
6164
}),
6265
});
6366
await mcpServer.connect(serverTransport);
6467
await mcpClient.connect(clientTransport);
6568
});
6669

67-
beforeEach(() => {
68-
if (mcpServer) {
69-
mcpServer.userConfig.telemetry = "disabled";
70-
}
71-
});
72-
7370
afterEach(async () => {
7471
if (mcpServer) {
7572
await mcpServer.session.close();
@@ -82,6 +79,13 @@ export function setupIntegrationTest(getUserConfig: () => UserConfig): Integrati
8279

8380
await mcpServer?.close();
8481
mcpServer = undefined;
82+
83+
// Reset DO_NOT_TRACK value
84+
if (oldDoNotTrackValue !== undefined) {
85+
process.env.DO_NOT_TRACK = oldDoNotTrackValue;
86+
} else {
87+
delete process.env.DO_NOT_TRACK;
88+
}
8589
});
8690

8791
const getMcpClient = () => {

tests/integration/tools/mongodb/mongodbHelpers.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,12 +22,10 @@ export function describeWithMongoDB(
2222
const integration = setupIntegrationTest(() => ({
2323
...getUserConfig(mdbIntegration),
2424
connectionString: mdbIntegration.connectionString(),
25-
telemetry: "disabled", // Explicitly disable telemetry
2625
}));
2726

2827
beforeEach(() => {
2928
integration.mcpServer().userConfig.connectionString = mdbIntegration.connectionString();
30-
integration.mcpServer().userConfig.telemetry = "disabled"; // Ensure telemetry stays disabled
3129
});
3230

3331
fn({

0 commit comments

Comments
 (0)