From b89634b3cc28ed1a18a349188444f86403d22790 Mon Sep 17 00:00:00 2001 From: Eduardo Filho Date: Tue, 29 Jul 2025 10:38:54 -0400 Subject: [PATCH 1/2] fix bugzillaNotificationEmails field name --- src/formatters/monitor.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/formatters/monitor.js b/src/formatters/monitor.js index 38c92b84f..a55d005e3 100644 --- a/src/formatters/monitor.js +++ b/src/formatters/monitor.js @@ -18,7 +18,7 @@ export function formatMonitor(monitor) { change_detection_technique, change_detection_args = [], } = monitor; - const notificationEmails = + const bugzillaNotificationEmails = Array.isArray(bugzilla_notification_emails) && bugzilla_notification_emails.length > 0 ? bugzilla_notification_emails @@ -26,7 +26,7 @@ export function formatMonitor(monitor) { const json = { alert: alert ? "Enabled" : "Disabled", platforms: platforms.length ? platforms : [], - notificationEmails, + bugzillaNotificationEmails, lowerIsBetter: lower_is_better === undefined ? null : !!lower_is_better, changeDetectionTechnique: change_detection_technique || null, changeDetectionArgs: change_detection_args, From b917465750a3daff8758f31860f005c95615b29e Mon Sep 17 00:00:00 2001 From: Eduardo Filho Date: Tue, 29 Jul 2025 12:29:55 -0400 Subject: [PATCH 2/2] Fix tests --- tests/formatters.monitor.test.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/formatters.monitor.test.js b/tests/formatters.monitor.test.js index 829c09083..712b7d88d 100644 --- a/tests/formatters.monitor.test.js +++ b/tests/formatters.monitor.test.js @@ -31,7 +31,7 @@ describe("formatMonitor", () => { '"platforms": [\n "Windows",\n "Linux"\n ]' ); expect(result).toContain( - '"notificationEmails": [\n "a@b.com",\n "c@d.com"\n ]' + '"bugzillaNotificationEmails": [\n "a@b.com",\n "c@d.com"\n ]' ); expect(result).toContain('"lowerIsBetter": true'); expect(result).toContain('"changeDetectionTechnique": "cdf-squared"'); @@ -46,7 +46,7 @@ describe("formatMonitor", () => { const result = formatMonitor(monitor); expect(result).toContain('"alert": "Disabled"'); expect(result).toContain('"platforms": []'); - expect(result).toContain('"notificationEmails": []'); + expect(result).toContain('"bugzillaNotificationEmails": []'); expect(result).toContain('"lowerIsBetter": null'); expect(result).toContain('"changeDetectionTechnique": null'); expect(result).toContain('"changeDetectionArgs": []');