Merge pull request #49770 from c3024/correct-unread-marker-for-thread…

…s-with-unread-messages (cherry picked from commit dbcd30e) (CP triggered by thienlnam)
Expensify · Sep 27, 2024 · fb91699 · fb91699
1 parent 3c243a9
commit fb91699
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 12 deletions.
diff --git a/src/pages/home/report/ReportActionsList.tsx b/src/pages/home/report/ReportActionsList.tsx
@@ -268,7 +268,7 @@ function ReportActionsList({
         }
 
         const mostRecentReportActionCreated = sortedVisibleReportActions[0]?.created ?? '';
-        if (mostRecentReportActionCreated === unreadMarkerTime) {
+        if (mostRecentReportActionCreated <= unreadMarkerTime) {
             return;
         }
 

diff --git a/tests/perf-test/ReportActionsList.perf-test.tsx b/tests/perf-test/ReportActionsList.perf-test.tsx
@@ -81,31 +81,23 @@ beforeEach(() => {
 });
 
 function ReportActionsListWrapper() {
-    const reportActions = ReportTestUtils.getMockedSortedReportActions(500);
-    const lastVisibleActionCreated = reportActions[0].created;
-    const report = {
-        ...LHNTestUtilsModule.getFakeReport(),
-        lastVisibleActionCreated,
-        lastReadTime: lastVisibleActionCreated,
-    };
-
     return (
         <ComposeProviders components={[OnyxProvider, LocaleContextProvider, ReportAttachmentsProvider]}>
             <ReactionListContext.Provider value={mockRef}>
                 <ActionListContext.Provider value={mockRef}>
                     <ReportActionsList
                         parentReportAction={createRandomReportAction(1)}
                         parentReportActionForTransactionThread={undefined}
-                        sortedReportActions={reportActions}
-                        report={report}
+                        sortedReportActions={ReportTestUtils.getMockedSortedReportActions(500)}
+                        report={LHNTestUtilsModule.getFakeReport()}
                         onLayout={mockOnLayout}
                         onScroll={mockOnScroll}
                         onContentSizeChange={() => {}}
                         listID={1}
                         loadOlderChats={mockLoadChats}
                         loadNewerChats={mockLoadChats}
                         transactionThreadReport={LHNTestUtilsModule.getFakeReport()}
-                        reportActions={reportActions}
+                        reportActions={ReportTestUtils.getMockedSortedReportActions(500)}
                     />
                 </ActionListContext.Provider>
             </ReactionListContext.Provider>