[wp-trac] [WordPress Trac] #54177: Add visibility to test class methods

WordPress Trac noreply at wordpress.org
Thu Nov 4 13:15:47 UTC 2021


#54177: Add visibility to test class methods
--------------------------------------+-------------------------------
 Reporter:  costdev                   |       Owner:  hellofromTonya
     Type:  enhancement               |      Status:  accepted
 Priority:  normal                    |   Milestone:  5.9
Component:  Build/Test Tools          |     Version:
 Severity:  normal                    |  Resolution:
 Keywords:  has-patch has-unit-tests  |     Focuses:  coding-standards
--------------------------------------+-------------------------------

Comment (by hellofromTonya):

 In [changeset:"52009" 52009]:
 {{{
 #!CommitTicketReference repository="" revision="52009"
 Coding Standards: Add `public` visibility to methods in
 `tests/phpunit/includes/`.

 This commit adds the `public` visibility keyword to each method which did
 not have an explicit visibility keyword.

 Why `public`?

 With no visibility previously declared, these methods are implicitly
 `public` and available for use. As these are part of the WordPress testing
 framework (for Core and extenders), changing them to anything else would
 be a backwards-compatibility break.

 Props costdev, jrf, hellofromTonya.
 See #54177.
 }}}

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


More information about the wp-trac mailing list