Merge pull request Expensify#94104 from emkhalid/fix-92431-track-workspace-admin-lhn

Fix #admin room visibility in LHN for Track Workspace onboarding

Author: lakchote

src/libs/ReportUtils.ts

@@ -9565,6 +9565,7 @@ function reasonForReportToBeInOptionList({
         isEmptyChat &&
         isChatReport(report) &&
         !isPolicyExpenseChat(report) &&
+        !isAdminRoom(report) &&
         !isTripRoom(report) &&
         !isConciergeChatReport(report) &&
         !isSystemChatReport &&

src/libs/SidebarUtils.ts

@@ -171,6 +171,7 @@ import {
     isAnnounceRoom,
     isArchivedNonExpenseReport,
     isArchivedReport,
+    isChatReport,
     isChatRoom,
     isChatThread,
     isConciergeChatReport,
@@ -335,6 +336,7 @@ function shouldDisplayReportInLHN({
         isFocused ||
         isSystemChat ||
         !!report.isPinned ||
+        (isChatReport(report) && isAdminRoom(report)) ||
         // eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
         requiresAttention ||
         (report.isOwnPolicyExpenseChat && !isReportArchived);

tests/unit/ReportUtilsTest.ts

@@ -7045,6 +7045,29 @@ describe('ReportUtils', () => {
             ).toBeTruthy();
         });
 
+        it('should return true for empty admin room even when excludeEmptyChats is true', () => {
+            const report: Report = {
+                ...createAdminRoom(1),
+                participants: buildParticipantsFromAccountIDs([1, 2]),
+                lastMessageText: '',
+                lastVisibleActionCreated: DateUtils.getDBTime(),
+            };
+
+            expect(
+                shouldReportBeInOptionList({
+                    report,
+                    chatReport: mockedChatReport,
+                    currentReportId: '',
+                    isInFocusMode: false,
+                    betas: [CONST.BETAS.DEFAULT_ROOMS],
+                    doesReportHaveViolations: false,
+                    excludeEmptyChats: true,
+                    draftComment: '',
+                    isReportArchived: undefined,
+                }),
+            ).toBeTruthy();
+        });
+
         it('should return false when the report has an unsupported type', () => {
             const report: Report = {
                 ...LHNTestUtils.getFakeReport(),