Handle missing backend bug report configuration

Co-authored-by: MasterAcnolo <68693319+MasterAcnolo@users.noreply.github.com>
This commit is contained in:
copilot-swe-agent[bot]
2026-09-27 18:33:26 +00:00
committed by GitHub
parent 8a11eae28e
commit e84a65b59d
2 changed files with 72 additions and 2 deletions

View File

@@ -35,10 +35,17 @@ async function sendReport(params) {
} }
} }
const response = await fetch(process.env.BUG_REPORT_URL, { const reportUrl = process.env.BUG_REPORT_URL;
const reportApiKey = process.env.BUG_REPORT_API_KEY;
if (!reportUrl || !reportApiKey) {
throw new Error("Bug report service is not configured. Missing BUG_REPORT_URL or BUG_REPORT_API_KEY on backend.");
}
const response = await fetch(reportUrl, {
method: "POST", method: "POST",
headers: { headers: {
"X-Api-Key": process.env.BUG_REPORT_API_KEY "X-Api-Key": reportApiKey
}, },
body: formData body: formData
}); });

View File

@@ -0,0 +1,63 @@
jest.mock("../../server/logger", () => ({
logger: {
info: jest.fn(),
warn: jest.fn(),
error: jest.fn(),
},
logDir: null,
}));
const { sendReport } = require("../../app/sendReport");
describe("sendReport", () => {
const originalEnv = { ...process.env };
beforeEach(() => {
jest.clearAllMocks();
process.env.BUG_REPORT_URL = "";
process.env.BUG_REPORT_API_KEY = "";
global.fetch = jest.fn();
});
afterAll(() => {
process.env = originalEnv;
});
test("throws a clear error when backend bug report config is missing", async () => {
await expect(
sendReport({
title: "Bug title",
description: "Bug description",
includeLogs: "no",
}),
).rejects.toThrow(
"Bug report service is not configured. Missing BUG_REPORT_URL or BUG_REPORT_API_KEY on backend.",
);
expect(global.fetch).not.toHaveBeenCalled();
});
test("sends request from backend when bug report config is present", async () => {
process.env.BUG_REPORT_URL = "https://example.com/report";
process.env.BUG_REPORT_API_KEY = "safe-test-key";
global.fetch.mockResolvedValue({ ok: true });
await expect(
sendReport({
title: "Bug title",
description: "Bug description",
includeLogs: "no",
}),
).resolves.toBe(true);
expect(global.fetch).toHaveBeenCalledWith(
"https://example.com/report",
expect.objectContaining({
method: "POST",
headers: {
"X-Api-Key": "safe-test-key",
},
}),
);
});
});