[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