#56782 closed task (blessed) (fixed)
PHPUnit tests: Add/improve @covers annotations for WP 6.2
Reported by: | pbearne | Owned by: | hellofromTonya |
---|---|---|---|
Milestone: | 6.2 | Priority: | normal |
Severity: | normal | Version: | 4.8 |
Component: | Build/Test Tools | Keywords: | has-patch php8 has-unit-tests |
Focuses: | coding-standards | Cc: |
Description (last modified by )
Previously:
- #39265 (up to 6.1)
Continuation of ongoing work to add and/or improve the PHPUnit tests @covers
annotations for more accurate coverage reporting.
Change History (14)
#4
@
2 years ago
- Summary changed from PHPUnit tests: Add/improvement @covers annotations for WP 6.2 to PHPUnit tests: Add/improve @covers annotations for WP 6.2
This ticket was mentioned in PR #1255 on WordPress/wordpress-develop by @pbearne.
2 years ago
#7
- Keywords has-unit-tests added
the ongoing covers ticket
I am still rechecking all the files so expect up[date to the pull request
Trac:
https://core.trac.wordpress.org/ticket/56782
https://core.trac.wordpress.org/ticket/39265
This ticket was mentioned in Slack in #core by sergey. View the logs.
22 months ago
#9
@
21 months ago
- Status changed from assigned to reviewing
There are new commits in the PR that are ready for review and commit. I'll be working on these during the remainder of the 6.2 cycle.
Triage folks:
This Trac ticket is for tests only. Please leave it in the 6.2 milestone. Thanks :)
This ticket was mentioned in Slack in #core by costdev. View the logs.
21 months ago
This ticket was mentioned in Slack in #core by mukeshpanchal27. View the logs.
21 months ago
#12
@
21 months ago
- Resolution set to fixed
- Status changed from reviewing to closed
Sadly other things took my attention from this. Closing this ticket as fixed for the 6.2 cycle. Work will continue once 6.3-alpha opens.
@spacedmonkey commented on PR #1255:
14 months ago
#14
@pbearne Can I recommend breaking this into smaller PR. Or at least classes should be in new PR. This PR is really hard to review and harder to commit / revert.
In 54710: