-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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 for ticket #58392 #4580
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good to me, except that we need a few docblocks in the provided unit tests.
You can take wpPublishPost as an example.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the updates @Hug0-Drelon! I've left some more thoughts in this review 🙂
Co-authored-by: Colin Stewart <79332690+costdev@users.noreply.github.com>
Thanks for the insight @costdev! Nice to see how standardization is going 🙂 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the updates @Hug0-Drelon! I think that's all from me on this one!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the update!
committed in https://core.trac.wordpress.org/changeset/56043 |
Trac ticket: https://core.trac.wordpress.org/ticket/58392
This PR adds unit tests to the previously proposed diff.