From cad1c95405c47a98a5d55cdca1229e1f78308fe6 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Fri, 13 Mar 2026 22:26:03 +0000
Subject: [PATCH] test: dedupe inline action skip assertions

---
 ...ine-actions.skip-when-config-empty.test.ts | 63 ++++++++++---------
 1 file changed, 32 insertions(+), 31 deletions(-)

diff --git a/src/auto-reply/reply/get-reply-inline-actions.skip-when-config-empty.test.ts b/src/auto-reply/reply/get-reply-inline-actions.skip-when-config-empty.test.ts
index 51351f05de8..36b5910ecae 100644
--- a/src/auto-reply/reply/get-reply-inline-actions.skip-when-config-empty.test.ts
+++ b/src/auto-reply/reply/get-reply-inline-actions.skip-when-config-empty.test.ts
@@ -84,6 +84,19 @@ const createHandleInlineActionsInput = (params: {
   };
 };
 
+async function expectInlineActionSkipped(params: {
+  ctx: ReturnType<typeof buildTestCtx>;
+  typing: TypingController;
+  cleanedBody: string;
+  command?: Partial<HandleInlineActionsInput["command"]>;
+  overrides?: Partial<Omit<HandleInlineActionsInput, "ctx" | "sessionCtx" | "typing" | "command">>;
+}) {
+  const result = await handleInlineActions(createHandleInlineActionsInput(params));
+  expect(result).toEqual({ kind: "reply", reply: undefined });
+  expect(params.typing.cleanup).toHaveBeenCalled();
+  expect(handleCommandsMock).not.toHaveBeenCalled();
+}
+
 describe("handleInlineActions", () => {
   beforeEach(() => {
     handleCommandsMock.mockReset();
@@ -97,18 +110,12 @@ describe("handleInlineActions", () => {
       To: "whatsapp:+123",
       Body: "hi",
     });
-    const result = await handleInlineActions(
-      createHandleInlineActionsInput({
-        ctx,
-        typing,
-        cleanedBody: "hi",
-        command: { to: "whatsapp:+123" },
-      }),
-    );
-
-    expect(result).toEqual({ kind: "reply", reply: undefined });
-    expect(typing.cleanup).toHaveBeenCalled();
-    expect(handleCommandsMock).not.toHaveBeenCalled();
+    await expectInlineActionSkipped({
+      ctx,
+      typing,
+      cleanedBody: "hi",
+      command: { to: "whatsapp:+123" },
+    });
   });
 
   it("forwards agentDir into handleCommands", async () => {
@@ -163,25 +170,19 @@ describe("handleInlineActions", () => {
       MessageSid: "41",
     });
 
-    const result = await handleInlineActions(
-      createHandleInlineActionsInput({
-        ctx,
-        typing,
-        cleanedBody: "old queued message",
-        command: {
-          rawBodyNormalized: "old queued message",
-          commandBodyNormalized: "old queued message",
-        },
-        overrides: {
-          sessionEntry,
-          sessionStore,
-        },
-      }),
-    );
-
-    expect(result).toEqual({ kind: "reply", reply: undefined });
-    expect(typing.cleanup).toHaveBeenCalled();
-    expect(handleCommandsMock).not.toHaveBeenCalled();
+    await expectInlineActionSkipped({
+      ctx,
+      typing,
+      cleanedBody: "old queued message",
+      command: {
+        rawBodyNormalized: "old queued message",
+        commandBodyNormalized: "old queued message",
+      },
+      overrides: {
+        sessionEntry,
+        sessionStore,
+      },
+    });
   });
 
   it("clears /stop cutoff when a newer message arrives", async () => {