Skip to content

Commit d5587a4

Browse files
committed
fix: adjust tests
1 parent 67259e4 commit d5587a4

File tree

3 files changed

+10
-3
lines changed

3 files changed

+10
-3
lines changed

src/server.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,6 @@ export class Server {
107107
timestamp: new Date().toISOString(),
108108
source: "mdbmcp",
109109
properties: {
110-
...this.telemetry.getCommonProperties(),
111110
result: "success",
112111
duration_ms: commandDuration,
113112
component: "server",

src/telemetry/telemetry.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ export class Telemetry {
116116
await client.sendEvents(
117117
events.map((event) => ({
118118
...event,
119-
properties: { ...event.properties, ...this.getCommonProperties() },
119+
properties: { ...this.getCommonProperties(), ...event.properties },
120120
}))
121121
);
122122
return { success: true };

tests/unit/telemetry.test.ts

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,15 @@ describe("Telemetry", () => {
7171
expect(appendEvents.length).toBe(appendEventsCalls);
7272

7373
if (sendEventsCalledWith) {
74-
expect(sendEvents[0]?.[0]).toEqual(sendEventsCalledWith);
74+
expect(sendEvents[0]?.[0]).toEqual(
75+
sendEventsCalledWith.map((event) => ({
76+
...event,
77+
properties: {
78+
...telemetry.getCommonProperties(),
79+
...event.properties,
80+
},
81+
}))
82+
);
7583
}
7684

7785
if (appendEventsCalledWith) {

0 commit comments

Comments
 (0)