From ecfc96e0a902c35a8a5d7c6c4e2c79f257469589 Mon Sep 17 00:00:00 2001
From: "Michael B. Gale" <mbg@github.com>
Date: Mon, 12 Oct 2026 12:49:48 +0100
Subject: [PATCH] Add `getDisplayActionName` tests

---
 lib/entry-points.js       | 11 +++++++----
 src/action-common.test.ts | 18 ++++++++++++++++++
 src/status-report.ts      | 18 ++++++++++++------
 3 files changed, 37 insertions(+), 10 deletions(-)

diff --git a/lib/entry-points.js b/lib/entry-points.js
index 1cee05cfb..44d94d1e0 100644
--- a/lib/entry-points.js
+++ b/lib/entry-points.js
@@ -171409,11 +171409,14 @@ function getAddressString(address) {
 }
 
 // src/status-report.ts
+var ACTION_DISPLAY_NAMES = {
+  ["finish" /* Analyze */]: "analyze",
+  ["init-post" /* InitPost */]: "init post"
+};
 function getDisplayActionName(actionName) {
-  if (actionName === "finish" /* Analyze */) {
-    return "analyze";
-  } else if (actionName === "init-post" /* InitPost */) {
-    return "init post";
+  const displayName = ACTION_DISPLAY_NAMES[actionName];
+  if (displayName !== void 0) {
+    return displayName;
   }
   return actionName;
 }
diff --git a/src/action-common.test.ts b/src/action-common.test.ts
index fc2e0a9aa..449300a8d 100644
--- a/src/action-common.test.ts
+++ b/src/action-common.test.ts
@@ -19,6 +19,24 @@ import { getErrorMessage } from "./util";
 
 setupTests(test);
 
+for (const [key, displayName] of Object.entries(
+  statusReport.ACTION_DISPLAY_NAMES,
+)) {
+  test(`getDisplayActionName - '${displayName}' instead of '${key}'`, (t) => {
+    t.is(statusReport.getDisplayActionName(key as ActionName), displayName);
+  });
+}
+
+test("getDisplayActionName - returns action name", (t) => {
+  for (const actionName of Object.values(ActionName)) {
+    if (statusReport.ACTION_DISPLAY_NAMES[actionName]) {
+      continue;
+    }
+
+    t.is(statusReport.getDisplayActionName(actionName), actionName);
+  }
+});
+
 interface RunInActionsTestOpts {
   runFn?: () => Promise<any>;
   expectedErrorMessage?: string;
diff --git a/src/status-report.ts b/src/status-report.ts
index 3f3e1d884..fc45e82d6 100644
--- a/src/status-report.ts
+++ b/src/status-report.ts
@@ -61,15 +61,21 @@ export enum ActionName {
   UploadSarif = "upload-sarif",
 }
 
+/** A partial mapping from `ActionName` to names for actions whose display name is not the `ActionName` value. */
+export const ACTION_DISPLAY_NAMES: Partial<Record<ActionName, string>> = {
+  [ActionName.Analyze]: "analyze",
+  [ActionName.InitPost]: "init post",
+};
+
 /**
- * Maps an `ActionName` to its display name. Usually that is the same, except
- * for `ActionName.Analyze` where it is `"analyze"` instead of `"finish"`.
+ * Maps an {@link ActionName} to its display name. Usually that is the same, except
+ * for actions which have a different display name in {@link ACTION_DISPLAY_NAMES}.
  */
 export function getDisplayActionName(actionName: ActionName): string {
-  if (actionName === ActionName.Analyze) {
-    return "analyze";
-  } else if (actionName === ActionName.InitPost) {
-    return "init post";
+  const displayName = ACTION_DISPLAY_NAMES[actionName];
+
+  if (displayName !== undefined) {
+    return displayName;
   }
   return actionName;
 }