Nest alertCounts object in event_report.properties (#1770)

github · Jul 11, 2023 · 863a05b · 863a05b
1 parent d6c8719
commit 863a05b
Show file tree

Hide file tree

Showing 6 changed files with 16 additions and 4 deletions.
diff --git a/lib/analyze.js b/lib/analyze.js
diff --git a/lib/analyze.js.map b/lib/analyze.js.map
diff --git a/lib/analyze.test.js b/lib/analyze.test.js
diff --git a/lib/analyze.test.js.map b/lib/analyze.test.js.map
diff --git a/src/analyze.test.ts b/src/analyze.test.ts
@@ -175,6 +175,8 @@ test("status report fields and search path setting", async (t) => {
       if (builtinStatusReport.event_reports) {
         for (const eventReport of builtinStatusReport.event_reports) {
           t.deepEqual(eventReport.event, "codeql database interpret-results");
+          t.true("properties" in eventReport);
+          t.true("alertCounts" in eventReport.properties!);
         }
       }
 
@@ -214,6 +216,8 @@ test("status report fields and search path setting", async (t) => {
       if (customStatusReport.event_reports) {
         for (const eventReport of customStatusReport.event_reports) {
           t.deepEqual(eventReport.event, "codeql database interpret-results");
+          t.true("properties" in eventReport);
+          t.true("alertCounts" in eventReport.properties!);
         }
       }
     }

diff --git a/src/analyze.ts b/src/analyze.ts
@@ -372,7 +372,9 @@ export async function runQueries(
           completed_at: endTimeInterpretResults.toISOString(),
           exit_status: "success",
           language,
-          properties: perQueryAlertCounts,
+          properties: {
+            alertCounts: perQueryAlertCounts,
+          },
         };
 
         if (statusReport["event_reports"] === undefined) {