Skip to content

Commit 05d9f9b

Browse files
Add test that error messages are redacted
1 parent 69b7e04 commit 05d9f9b

1 file changed

Lines changed: 20 additions & 0 deletions

File tree

extensions/ql-vscode/test/vscode-tests/no-workspace/telemetry.test.ts

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -445,6 +445,26 @@ describe("telemetry reporting", () => {
445445
});
446446
});
447447

448+
it("should redact error message contents", async () => {
449+
jest.spyOn(Config, "newTelemetryEnabled").mockReturnValue(true);
450+
await telemetryListener.initialize();
451+
452+
telemetryListener.sendError(
453+
redactableError`test message with secret information: ${42} and more ${"secret"} parts`,
454+
);
455+
456+
expect(sendTelemetryEventSpy).toHaveBeenCalledWith(
457+
"error",
458+
{
459+
message:
460+
"test message with secret information: [REDACTED] and more [REDACTED] parts",
461+
isCanary,
462+
stack: expect.any(String),
463+
},
464+
{},
465+
);
466+
});
467+
448468
async function enableTelemetry(section: string, value: boolean | undefined) {
449469
await workspace
450470
.getConfiguration(section)

0 commit comments

Comments
 (0)