MM-59444 Add tests for actions/remote/command

jwilander commented 2 weeks ago

Summary

Add unit tests for actions/remote/command. Coverage increases from x% to 95.06%.

This was my first time using GitHub Copilot Chat to help write tests. As a result it took me longer than usual to write these but they are more thorough than some of the other tests I've written.

Ticket Link

https://mattermost.atlassian.net/browse/MM-59444

Checklist

[x] Added or updated unit tests (required for all new features)
[ ] Has UI changes
[ ] Includes text changes and localization file updates
[ ] Have tested against the 5 core themes to ensure consistency between them.
[ ] Have run E2E tests by adding label E2E iOS tests for PR.

Device Information

This PR was tested on: n/a

Screenshots

n/a

Release Note

NONE

yasserfaraazkhan commented 2 weeks ago

I like that there is check for error messages and for the functions being called with certain params. 🎉

jwilander commented 2 weeks ago

so copilot huh? definitely better test cases, good coverage.

In your opinion, should we leverage this copilot thing more? is it worth it even if takes you a bit longer ?

I'm going to play with it some more but it has potential to be faster. It rarely generates perfect code but with the right set of prompts it seems to get 80% of the way there

mattermost / mattermost-mobile