Skip to content

[HOLD for payment 2024-04-25] [$250] [Reassure] Add measureFunction test for formatSectionsFromSearchTerm #38166

Closed

Description

@mountiny

Problem

The formatSectionsFromSearchTerm is a method that is used often in the App and if its performance degrades, it can have considerable impact on the App performance.

Solution

Let's add a measure function reassure test to cover various cases of this method call similarly as we have created such tests for ReportActionUtils methods for example.


Please refer to other tests in the repository, here is a readme for reassure tests which should help you get familiar with the tool.

All new tests should be written in TS.

Please provide a proposal stating, how you could write such test and scenarios for the method such that we test the slowest execution path as well.

cc @OlimpiaZurek

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~015c574cb1d199b7bd
  • Upwork Job ID: 1767635884215123968
  • Last Price Increase: 2024-03-12
  • Automatic offers:
    • getusha | Reviewer | 0
    • brandonhenry | Contributor | 0
Issue OwnerCurrent Issue Owner: @alexpensify

Activity

  1. added
    ExternalAdded to denote the issue can be worked on by a contributor
    NewFeatureSomething to build that is a new item.
    on Mar 12, 2024
  2. self-assigned this
    on Mar 12, 2024
  3. changed the title [-][Reassure] Add measureFunction test for formatSectionsFromSearchTerm[/-] [+][$500] [Reassure] Add measureFunction test for formatSectionsFromSearchTerm[/+] on Mar 12, 2024
  4. melvin-bot commented on Mar 12, 2024

    @melvin-bot
  5. added
    Help WantedApply this label when an issue is open to proposals by contributors
    on Mar 12, 2024
  6. melvin-bot commented on Mar 12, 2024

    @melvin-bot

    Triggered auto assignment to Contributor-plus team member for initial proposal review - @getusha (External)

  7. melvin-bot commented on Mar 12, 2024

    @melvin-bot
  8. 45 remaining items

  9. changed the title [-][$250] [Reassure] Add measureFunction test for formatSectionsFromSearchTerm[/-] [+][HOLD for payment 2024-04-25] [$250] [Reassure] Add measureFunction test for formatSectionsFromSearchTerm[/+] on Apr 18, 2024
  10. melvin-bot commented on Apr 18, 2024

    @melvin-bot

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

  11. melvin-bot commented on Apr 18, 2024

    @melvin-bot

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

    If no regressions arise, payment will be issued on 2024-04-25. 🎊

    For reference, here are some details about the assignees on this issue:

  12. melvin-bot commented on Apr 18, 2024

    @melvin-bot

    BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

    • [@getusha] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
    • [@alexpensify] Link the GH issue for creating/updating the regression test once above steps have been agreed upon.
  13. mountiny commented on Apr 20, 2024

    @mountiny
    ContributorAuthor

    $250 to @getusha and to @brandonhenry

  14. alexpensify commented on Apr 25, 2024

    @alexpensify
    Contributor

    @getusha and @brandonhenry - there was an issue with Upwork. I updated the job link and need you to accept it here:

    https://www.upwork.com/jobs/~01ac085e1e841dfb81

    After you accept, I can continue with the next steps. Thanks!

  15. brandonhenry commented on Apr 26, 2024

    @brandonhenry
    Contributor

    @alexpensify proposal on upwork submitted 👍🏿

  16. alexpensify commented on Apr 26, 2024

    @alexpensify
    Contributor

    Payouts due: April 25, 2024

    Upwork job is here.

  17. alexpensify commented on Apr 26, 2024

    @alexpensify
    Contributor

    Everyone has been paid via Upwork -- closing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionDailyKSv2ExternalAdded to denote the issue can be worked on by a contributorNewFeatureSomething to build that is a new item.

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions