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.

Sign up for GitHub

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

Jump to bottom

Add utils/post_list unit tests #8265

Merged
merged 1 commit into from
Oct 18, 2024
Merged

Add utils/post_list unit tests #8265

merged 1 commit into from
Oct 18, 2024

Conversation

enahum
Copy link
Contributor

@enahum enahum commented Oct 17, 2024 •
edited
Loading

Summary

Adds code coverage for utils/post_list from 0% to 92.81%

Ticket Link

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

Checklist

  • Added or updated unit tests (required for all new features)

Release Note

NONE

@enahum enahum added the 2: Dev Review Requires review by a core commiter label Oct 17, 2024
yasserfaraazkhan
yasserfaraazkhan approved these changes Oct 18, 2024
View reviewed changes
Copy link
Contributor

@yasserfaraazkhan yasserfaraazkhan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@enahum enahum merged commit 418f43c into main Oct 18, 2024
20 checks passed
@enahum enahum deleted the utils/post_list branch October 18, 2024 04:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yasserfaraazkhan yasserfaraazkhan yasserfaraazkhan approved these changes

@jwilander jwilander jwilander approved these changes

Assignees
No one assigned
Labels
2: Dev Review Requires review by a core commiter release-note-none
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@enahum @jwilander @yasserfaraazkhan @mm-cloud-bot