Comprehensive unit tests and docs for LHN item presence

[OlimpiaZurek]

Problem LHN items have a historical record of rendering inconsistencies & regressions that lead us to a recurring question of “Why is this in my LHN?”. There are multiple factors determining which reports should be displayed including report’s type, participants, status, specific conditions such as being pinned, having violations, or being archived which makes it easy to accidentally break this pipeline.

Solution Implement a comprehensive unit test suite & docs around shouldReportBeInOptionList that focuses on clearly defined scenarios for report visibility. This will help us increase confidence and make sure on the CI level that no regressions in this area make it to the production environment which is not something we can guarantee ourselves right now. Here are some sample scenarios we can assert:

Should show the report

• is expense report with outstanding violations and the report is not reimbursed
• is currently viewing
• has valid draft comment in the report
• Is a self DM report
• is marked it as pinned
• is in focus mode and the report is unread
• is in default mode and the report is archived

slack thread: https://callstack-hq.slack.com/archives/C05LX9D6E07/p1727967763482779

[melvin-bot[bot]]

Triggered auto assignment to @RachCHopkins (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

[RachCHopkins]

Expensify slack thread for convenience: https://expensify.slack.com/archives/C05LX9D6E07/p1727967763482779

[melvin-bot[bot]]

Eep! 4 days overdue now. Issues have feelings too...

[melvin-bot[bot]]

Still overdue 6 days?! Let's take care of this!

[melvin-bot[bot]]

8 days overdue is a lot. Should this be a Weekly issue? If so, feel free to change it!

[melvin-bot[bot]]

12 days overdue now... This issue's end is nigh!

[melvin-bot[bot]]

This issue has not been updated in over 14 days. eroding to Weekly issue.