#42731 closed enhancement (fixed)
Add inline documentation to WP_UnitTestCase
| Reported by: | anonymized_8769252 | Owned by: | johnbillion |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.1 |
| Component: | Build/Test Tools | Version: | |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | Focuses: |
Description
The WP_UnitTestCase class is large, and complex, but it does not contain much inline documentation to make it easier to understand.
Let's change that.
Attachments (2)
Change History (9)
#2
@
9 years ago
- Milestone Awaiting Review → 5.0
Looking good Frank, a couple of things:
- Can you update the patch to so that "back ported" is spelled as one word, i.e "backported" please
- I think the
//error_log( $file );comment should remain, it gives a quick heads up when debugging
#3
@
9 years ago
Thanks for the feedback, updated the patch.
Concerning the commented out //error_log( $file );, I agree that this should not be removed in a patch that adds inline documentation.
However I am convinced that it should not remain in the file, it should just be removed as part of another ticket. Leaving commented out code behind is like leaving litter on the floor of a public place.
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
The patch above contains part one, as it's easier to do this in phases.