[wp-trac] [WordPress Trac] #55652: Test tool and unit test improvements for 6.1
WordPress Trac
noreply at wordpress.org
Mon Aug 22 22:43:15 UTC 2022
#55652: Test tool and unit test improvements for 6.1
--------------------------------------+---------------------
Reporter: hellofromTonya | Owner: (none)
Type: task (blessed) | Status: new
Priority: normal | Milestone: 6.1
Component: Build/Test Tools | Version:
Severity: normal | Resolution:
Keywords: has-patch has-unit-tests | Focuses:
--------------------------------------+---------------------
Comment (by SergeyBiryukov):
In [changeset:"53921" 53921]:
{{{
#!CommitTicketReference repository="" revision="53921"
Tests: Consistently skip tests for non-implemented methods in REST API
test classes.
WordPress core test suite uses PHPUnit's
`beStrictAboutTestsThatDoNotTestAnything` option set to `true`, which
marks a test as risky when no assertions are performed.
REST API test classes have some empty tests for non-implemented methods
because these test classes extend the abstract
`WP_Test_REST_Controller_Testcase` class, which requires several methods
to be implemented that don't necessarily make sense for all REST API
routes.
Some of these empty tests were already marked as skipped, but not in a
consistent manner. Since skipping these tests is intentional for the time
being, this commit aims to bring some consistency and adjust them all to
be more accurately reported as skipped instead of risky.
The skipping can be reconsidered in the future when the tests are either
removed as unnecessary or updated to actually perform assertions related
to their behavior.
Follow-up to [40534], [41176], [41228].
Props Mte90, tomepajk, johnbillion, zieladam, SergeyBiryukov.
See #40538, #41463, #55652.
}}}
--
Ticket URL: <https://core.trac.wordpress.org/ticket/55652#comment:117>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list