From c25f56c96e6ca4fd83dcd0aea7ad205dc384387d Mon Sep 17 00:00:00 2001
From: Jinjing <6427696+AmethystLiang@users.noreply.github.com>
Date: Thu, 25 Jun 2026 01:18:17 -0700
Subject: [PATCH] Improve timeline items assertions in work-item-details test

Extract timelineItems to a local variable and add an explicit null check to prevent TypeScript compilation errors when calling the .at method.
---
 src/main/github/work-item-details.test.ts | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/src/main/github/work-item-details.test.ts b/src/main/github/work-item-details.test.ts
index 5343c00813f..a36ac97e0fa 100644
--- a/src/main/github/work-item-details.test.ts
+++ b/src/main/github/work-item-details.test.ts
@@ -283,9 +283,13 @@ describe('getWorkItemDetails', () => {
         call[0].includes('repos/acme/widgets/issues/923/timeline?per_page=100&page=7')
       )
     ).toBe(false)
-    expect(details?.timelineItems).toHaveLength(300)
-    expect(details?.timelineItems.at(0)).toMatchObject({ assignee: 'assignee-3-0' })
-    expect(details?.timelineItems.at(-1)).toMatchObject({ assignee: 'assignee-6-89' })
+    const timelineItems = details?.timelineItems
+    expect(timelineItems).toHaveLength(300)
+    if (!timelineItems) {
+      throw new Error('Expected timeline items to be present')
+    }
+    expect(timelineItems.at(0)).toMatchObject({ assignee: 'assignee-3-0' })
+    expect(timelineItems.at(-1)).toMatchObject({ assignee: 'assignee-6-89' })
   })
 
   it('falls back to REST + GraphQL when the collapsed issue query fails', async () => {