Opened 8 weeks ago
Last modified 4 weeks ago
#60705 new task (blessed)
Test tool and unit test improvements for 6.6
Reported by: | SergeyBiryukov | Owned by: | |
---|---|---|---|
Milestone: | 6.6 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Build/Test Tools | Keywords: | has-patch has-unit-tests |
Focuses: | Cc: |
Description
Previously:
- #59647 (6.5)
- #58955 (6.4)
- #57841 (6.3)
- #56793 (6.2)
- #55652 (6.1)
- #54725 (6.0)
- #53363 (5.9)
- #52625 (5.8)
- #51802 (5.7)
- #51344 (5.6)
This ticket is for various fixes and improvements in PHPUnit tests that don't have a more specific ticket, as well as general improvements to the GitHub Actions workflows that run automated testing.
Change History (4)
This ticket was mentioned in PR #5100 on WordPress/wordpress-develop by @johnbillion.
8 weeks ago
#1
- Keywords has-patch has-unit-tests added
This ticket was mentioned in PR #5849 on WordPress/wordpress-develop by @hellofromTonya.
8 weeks ago
#2
Reorganizes the hooks tests to align to coding standards and consistency within the test suite.
- [X] Moves
WP_Hooks
method tests to into separate directory<GroupName>/<className>/<methodName>.php
. - [X] Splits the function tests in
actions.php
andfilters.php
into separate test classes within thetests/hooks
/ directory. - [ ] Add assertion failure message to each instance where there are more than 1 assertion in the test method.
Trac ticket:
https://core.trac.wordpress.org/ticket/60705
https://core.trac.wordpress.org/ticket/59647
Note: See
TracTickets for help on using
tickets.
Trac ticket:
https://core.trac.wordpress.org/ticket/60705
https://core.trac.wordpress.org/ticket/59647
https://core.trac.wordpress.org/ticket/58955
This changes several instances of test skipping to test failures, and removes some unnecessary
function_exists()
checks for compat functions.If any of the checks in these test aren't satisfied then the test should fail rather than be skipped.
In addition, I removed some multisite test skipping which is redundant as it's handled by the
ms-required
andms-excluded
test group handling.Previously: