[wp-trac] [WordPress Trac] #57099: Fix Test Class Header Tags

WordPress Trac noreply at wordpress.org
Mon Nov 14 09:18:51 UTC 2022


#57099: Fix Test Class Header Tags
-------------------------+-----------------------------
 Reporter:  wojtekn      |      Owner:  (none)
     Type:  enhancement  |     Status:  new
 Priority:  normal       |  Milestone:  Awaiting Review
Component:  General      |    Version:
 Severity:  minor        |   Keywords:
  Focuses:               |
-------------------------+-----------------------------
 The class header of the Tests_Functions_DoEnclose class contains duplicate
 information (check @since).

 As it was proposed by antonvlasenko, it makes sense to merge these two
 docblocks into one and remove the duplicate tags (i.e., @since).

 Why?

 * Having two docblocks doesn’t make sense to me if a file contains only 1
 test class. However, it can be justified if a file contains multiple test
 classes (which is not the case here).
 * Out of 804 PHP files in the tests/phpunit folder, 497 files have 1
 docblock, and 307 files have both file and class docblocks. So, having a
 single docblock per test class/file is more common.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/57099>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list