Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add unit tests to app/utils/general #8133

Merged
merged 1 commit into from
Aug 8, 2024
Merged

add unit tests to app/utils/general #8133

merged 1 commit into from
Aug 8, 2024

Conversation

enahum
Copy link
Contributor

@enahum enahum commented Aug 7, 2024

Summary

Add 100% coverage for app/utils/general

image

Ticket Link

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

Checklist

  • 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.

Release Note

NONE

@enahum enahum added the 2: Dev Review Requires review by a core commiter label Aug 7, 2024
@enahum enahum requested review from larkox and jwilander August 7, 2024 08:41
@enahum enahum added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels Aug 8, 2024
@enahum enahum merged commit bde7e3e into main Aug 8, 2024
26 checks passed
@enahum enahum deleted the MM-59569 branch August 8, 2024 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request release-note-none
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants