[wp-trac] [WordPress Trac] #55652: Test tool and unit test improvements for 6.1

WordPress Trac noreply at wordpress.org
Thu Sep 1 22:54:34 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:"54058" 54058]:
 {{{
 #!CommitTicketReference repository="" revision="54058"
 Tests: Explicitly mark empty REST API tests as not performing any
 assertions.

 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.

 As these tests are intentionally empty, they were previously marked as
 skipped, so that they are not reported as risky.

 This commit aims to further reduce noise in the test suite and effectively
 ignores these empty tests altogether, which seems like a more appropriate
 option at this time.

 The `@doesNotPerformAssertions` annotation 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], [53921].

 See #40538, #41463, #55652.
 }}}

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/55652#comment:135>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list