From 258945d4d0d8ff2d16f9ee055074bbe71738ef65 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Sat, 14 Mar 2026 01:30:10 +0000
Subject: [PATCH] test: share status issue assertion helpers

---
 extensions/test-utils/status-issues.ts        | 10 ++++++++++
 extensions/zalo/src/status-issues.test.ts     | 10 +++++-----
 extensions/zalouser/src/status-issues.test.ts | 10 +++++-----
 3 files changed, 20 insertions(+), 10 deletions(-)
 create mode 100644 extensions/test-utils/status-issues.ts

diff --git a/extensions/test-utils/status-issues.ts b/extensions/test-utils/status-issues.ts
new file mode 100644
index 00000000000..7de3c6bcd55
--- /dev/null
+++ b/extensions/test-utils/status-issues.ts
@@ -0,0 +1,10 @@
+import { expect } from "vitest";
+
+export function expectOpenDmPolicyConfigIssue<TAccount>(params: {
+  collectIssues: (accounts: TAccount[]) => Array<{ kind?: string }>;
+  account: TAccount;
+}) {
+  const issues = params.collectIssues([params.account]);
+  expect(issues).toHaveLength(1);
+  expect(issues[0]?.kind).toBe("config");
+}
diff --git a/extensions/zalo/src/status-issues.test.ts b/extensions/zalo/src/status-issues.test.ts
index 1326ba13604..581a0dfe916 100644
--- a/extensions/zalo/src/status-issues.test.ts
+++ b/extensions/zalo/src/status-issues.test.ts
@@ -1,18 +1,18 @@
 import { describe, expect, it } from "vitest";
+import { expectOpenDmPolicyConfigIssue } from "../../test-utils/status-issues.js";
 import { collectZaloStatusIssues } from "./status-issues.js";
 
 describe("collectZaloStatusIssues", () => {
   it("warns when dmPolicy is open", () => {
-    const issues = collectZaloStatusIssues([
-      {
+    expectOpenDmPolicyConfigIssue({
+      collectIssues: collectZaloStatusIssues,
+      account: {
         accountId: "default",
         enabled: true,
         configured: true,
         dmPolicy: "open",
       },
-    ]);
-    expect(issues).toHaveLength(1);
-    expect(issues[0]?.kind).toBe("config");
+    });
   });
 
   it("skips unconfigured accounts", () => {
diff --git a/extensions/zalouser/src/status-issues.test.ts b/extensions/zalouser/src/status-issues.test.ts
index 73f7277b2b9..c1e142c88e8 100644
--- a/extensions/zalouser/src/status-issues.test.ts
+++ b/extensions/zalouser/src/status-issues.test.ts
@@ -1,4 +1,5 @@
 import { describe, expect, it } from "vitest";
+import { expectOpenDmPolicyConfigIssue } from "../../test-utils/status-issues.js";
 import { collectZalouserStatusIssues } from "./status-issues.js";
 
 describe("collectZalouserStatusIssues", () => {
@@ -17,16 +18,15 @@ describe("collectZalouserStatusIssues", () => {
   });
 
   it("warns when dmPolicy is open", () => {
-    const issues = collectZalouserStatusIssues([
-      {
+    expectOpenDmPolicyConfigIssue({
+      collectIssues: collectZalouserStatusIssues,
+      account: {
         accountId: "default",
         enabled: true,
         configured: true,
         dmPolicy: "open",
       },
-    ]);
-    expect(issues).toHaveLength(1);
-    expect(issues[0]?.kind).toBe("config");
+    });
   });
 
   it("skips disabled accounts", () => {