test: adds tier-ordering and border accent tests for tracked user items

Author: wgordon17

tests/components/dashboard/IssuesTab.test.tsx

@@ -456,6 +456,25 @@ describe("IssuesTab — left border accent in 'all' scope", () => {
     expect(listitem?.className).not.toContain("border-l-primary");
   });
 
+  it("adds border-l-primary to tracked user issue in monitored repo in 'all' scope", () => {
+    const issues = [
+      makeIssue({ id: 1, title: "Bot issue", repoFullName: "org/monitored", surfacedBy: ["tracked-bot[bot]"] }),
+    ];
+    setTabFilter("issues", "scope", "all");
+    setAllExpanded("issues", ["org/monitored"], true);
+
+    const { container } = render(() => (
+      <IssuesTab
+        issues={issues}
+        userLogin="me"
+        monitoredRepos={[{ owner: "org", name: "monitored", fullName: "org/monitored" }]}
+      />
+    ));
+
+    const listitem = container.querySelector('[role="listitem"]');
+    expect(listitem?.className).toContain("border-l-primary");
+  });
+
   it("does not add border-l-2 in default 'involves_me' scope", () => {
     const issues = [
       makeIssue({ id: 1, title: "My issue", repoFullName: "org/repo", surfacedBy: ["me"] }),

tests/components/dashboard/PullRequestsTab.test.tsx

@@ -406,6 +406,25 @@ describe("PullRequestsTab — left border accent in 'all' scope", () => {
     expect(listitem?.className).not.toContain("border-l-primary");
   });
 
+  it("adds border-l-primary to tracked user PR in monitored repo in 'all' scope", () => {
+    const prs = [
+      makePullRequest({ id: 1, title: "Bot PR", repoFullName: "org/monitored", surfacedBy: ["tracked-bot[bot]"] }),
+    ];
+    setTabFilter("pullRequests", "scope", "all");
+    setAllExpanded("pullRequests", ["org/monitored"], true);
+
+    const { container } = render(() => (
+      <PullRequestsTab
+        pullRequests={prs}
+        userLogin="me"
+        monitoredRepos={[{ owner: "org", name: "monitored", fullName: "org/monitored" }]}
+      />
+    ));
+
+    const listitem = container.querySelector('[role="listitem"]');
+    expect(listitem?.className).toContain("border-l-primary");
+  });
+
   it("does not add border-l-primary in default 'involves_me' scope", () => {
     const prs = [
       makePullRequest({ id: 1, title: "My PR", repoFullName: "org/repo", surfacedBy: ["me"] }),

tests/lib/grouping.test.ts

@@ -63,7 +63,7 @@ describe("isUserInvolved", () => {
   const base = { repoFullName: "org/repo", userLogin: "author", assigneeLogins: [] as string[] };
   const monitored = new Set(["org/monitored"]);
 
-  it("returns true when surfacedBy includes user", () => {
+  it("returns true when surfacedBy is non-empty (main user)", () => {
     expect(isUserInvolved({ ...base, surfacedBy: ["me"] }, "me", monitored)).toBe(true);
   });
 
@@ -75,6 +75,10 @@ describe("isUserInvolved", () => {
     expect(isUserInvolved({ ...base, surfacedBy: ["bot1[bot]", "bot2"] }, "me", monitored)).toBe(true);
   });
 
+  it("returns true for monitored repo item with surfacedBy (tier 1 before tier 2)", () => {
+    expect(isUserInvolved({ ...base, repoFullName: "org/monitored", surfacedBy: ["tracked-bot[bot]"] }, "me", monitored)).toBe(true);
+  });
+
   it("returns true for non-monitored item with no surfacedBy (fetched via involves:{user})", () => {
     expect(isUserInvolved(base, "me", monitored)).toBe(true);
   });