Add unit test cases for performance.ts
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:
- PR title:
test: <Jira ticket> unit test forfilename``, ex.test: MM-59413 unit test for app/actions/remote/apps.ts - Indicate test coverage improvement, ex.
From 2% to 96% - Attached a screenshot of test coverage result.
- 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 Would like to work on it !
@mattermod Would like to work on it
@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
Already solved, see file https://github.com/mattermost/mattermost-mobile/blob/main/app/actions/remote/performance.test.ts