[HOLD for payment 2023-12-20] [HOLD for payment 2023-12-12] [Tracking] Use measureFunction from reassure to cover performance testing of important functions

[mountiny]

Coming from here https://expensify.slack.com/archives/C05LX9D6E07/p1698193710179789

We have set up the reassure in the App with new option to use measureFunction to test performance of methods. Lets use this to cover the most crucial method in the app to ensure changes do not introduce big regressions in these methods which might already take long and are called often.

Let's focus on methods in ReportActionsUtils, SidebarUtils (getOptionData specifically) and ReportUtils

• getLastVisibleAction [NoQA] test(reassure): get last visible action #30482

[mountiny]

this is being worked on

[mountiny]

Making a steady progress 🚀

[melvin-bot]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.7-4 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• [NoQA] add perf tests for ReportActionsUtils #32093
• [NoQA] chore: add perf tests for SidebarUtils #30909

If no regressions arise, payment will be issued on 2023-12-12. 🎊

After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

• External issue reporter
• Contributor that fixed the issue
• Contributor+ that helped on the issue and/or PR

[mountiny]

@OlimpiaZurek @adhorodyski This is all done, right?

[OlimpiaZurek]

Hey @mountiny perf tests for methods from SidebarUtils and ReportActionUtils have already been done. Now I’m working on adding tests to ReportUtils. This will be the last part of this scope. PR should be ready by the EOD.

[OlimpiaZurek]

@mountiny PR is ready for review

[melvin-bot]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.11-25 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• Add perf tests for ReportUtils #32637

If no regressions arise, payment will be issued on 2023-12-20. 🎊

After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

• External issue reporter
• Contributor that fixed the issue
• Contributor+ that helped on the issue and/or PR

[mountiny]

Seems like we are all done here!