mattermost icon indicating copy to clipboard operation
mattermost copied to clipboard

Add unit test cases for performance.ts

Open mattermod opened this issue 1 year ago • 1 comments

File: https://github.com/mattermost/mattermost-mobile/blob/main/app/actions/remote/performance.ts Acceptance criteria:

  • Should pass in CI
  • Should have Coverage of 95%

When submitting a PR:

  1. PR title: test: <Jira ticket> unit test for filename``, ex. test: MM-59413 unit test for app/actions/remote/apps.ts
  2. Indicate test coverage improvement, ex. From 2% to 96%
  3. Attached a screenshot of test coverage result.
  4. If something can’t be tested, describe and explain for reference.

Need help or would like to get in touch?

Feel free to post a message, questions and or anything at https://community.mattermost.com/core/channels/native-mobile-apps and get in touch with us – Mattermost developers and community.


If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.

JIRA: https://mattermost.atlassian.net/browse/MM-59421

mattermod avatar Jul 15 '24 13:07 mattermod

@mattermod Would like to work on it !

tasnim0tantawi avatar Aug 06 '24 13:08 tasnim0tantawi

@mattermod Would like to work on it

kubicodes avatar Feb 14 '25 10:02 kubicodes

@mattermod unfortunately I won't be able to work on this for a while. You can mark this as unassigned again, so eventually someone else can pick it up. Sorry for the inconvenience

kubicodes avatar Feb 17 '25 19:02 kubicodes

Already solved, see file https://github.com/mattermost/mattermost-mobile/blob/main/app/actions/remote/performance.test.ts

hereje avatar Oct 10 '25 06:10 hereje