Skip to content

[HOLD for payment 2024-06-03] [$250] Create automated tests for Workspace Profile, Members and Categories pages #40835

Description

@luacmartins

Problem

New API commands introduced as part of the Simplified Collect project don't have automated tests, which can lead to bugs and unexpected behavior when these commands are updated.

Solution

Add automated tests for the commands below:

  • Test UpdateWorkspaceDescription
  • Test AcceptJoinRequest
  • Test UpdateWorkspaceMembersRole
  • Test RequestWorkspaceOwnerChange
  • Test AddBillingCardAndRequestPolicyOwnerChange
  • Test SetWorkspaceRequiresCategory
  • Test CreateWorkspaceCategories
  • Test RenameWorkspaceCategory
  • Test SetWorkspaceCategoriesEnabled
  • Test DeleteWorkspaceCategories
Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~011452b1c44ee7036a
  • Upwork Job ID: 1782893496746893312
  • Last Price Increase: 2024-04-23
  • Automatic offers:
    • hoangzinh | Reviewer | 0
    • tienifr | Contributor | 0
Issue OwnerCurrent Issue Owner: @mallenexpensify

Activity

  1. added
    BugSomething is broken. Auto assigns a BugZero manager.
    on Apr 23, 2024
  2. self-assigned this
    on Apr 23, 2024
  3. added
    ExternalAdded to denote the issue can be worked on by a contributor
    on Apr 23, 2024
  4. melvin-bot commented on Apr 23, 2024

    @melvin-bot

    Triggered auto assignment to @mallenexpensify (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.

  5. changed the title [-]Create automated tests for Workspace Profile, Members and Categories pages[/-] [+][$250] Create automated tests for Workspace Profile, Members and Categories pages[/+] on Apr 23, 2024
  6. melvin-bot commented on Apr 23, 2024

    @melvin-bot
  7. added
    Help WantedApply this label when an issue is open to proposals by contributors
    on Apr 23, 2024
  8. removed
    Help WantedApply this label when an issue is open to proposals by contributors
    BugSomething is broken. Auto assigns a BugZero manager.
    on Apr 23, 2024
  9. melvin-bot commented on Apr 23, 2024

    @melvin-bot

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

  10. tienifr commented on Apr 24, 2024

    @tienifr
    Contributor

    Proposal

    Please re-state the problem that we are trying to solve in this issue.

    We need to add automated tests for new API commands in Workspace Profile, Members and Categories pages

    What is the root cause of that problem?

    This is new requirement, currently we don't have any automated tests for those commands

    What changes do you think we should make in order to solve the problem?

    For each command, we need to:

    1. Define the test scenarios for the command, this will be the different cases that the input params of the command function (for example this) can take. Eg. If a param is optional, we need to test both scenarios where that params is, and is not provided. And there should be the scenarios for different user's use cases as well, we can check this by testing the flow in the UI to see what possible values/actions they can take that influence the command input.

    2. Write unit tests with those scenarios, given a set of command function input, we need to check that API.write/API.read are called with a correct set of parameters and onyxData (eg. this call).

    For how to write it, we'll write Jest tests just like we did for other utils like here

    What alternative solutions did you explore? (Optional)

    NA

  11. hoangzinh commented on Apr 24, 2024

    @hoangzinh
    Contributor

    @luacmartins I am concerned that those automation tests should be added to BE repository or this Expensify/App repository?

  12. 13 remaining items

  13. mallenexpensify commented on May 22, 2024

    @mallenexpensify
    Contributor

    Thanks @hoangzinh , seems like, with the deploy freezer, we're going to have a LOT of issue come due for payment at/around the same time

  14. added
    Awaiting PaymentAuto-added when associated PR is deployed to production
    and removed on May 27, 2024
  15. changed the title [-][$250] Create automated tests for Workspace Profile, Members and Categories pages[/-] [+][HOLD for payment 2024-06-03] [$250] Create automated tests for Workspace Profile, Members and Categories pages[/+] on May 27, 2024
  16. melvin-bot commented on May 27, 2024

    @melvin-bot

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

  17. melvin-bot commented on May 27, 2024

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.75-1 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-06-03. 🎊

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

  18. mallenexpensify commented on Jun 3, 2024

    @mallenexpensify
    Contributor

    Contributor: @tienifr paid $250 via Upwork
    Contributor+: @hoangzinh paid $250 via Upwork.

    I don't think we need a regression test here. @luacmartins , comment and reopen if you disagree.

  19. luacmartins commented on Jun 3, 2024

    @luacmartins
    ContributorAuthor

    No requession test needed

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 contributor

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions