Page MenuHomePhabricator

MultiMaps extension is missing @covers tags
Open, Needs TriagePublic

Description

PHPCS has identified that some of the MultiMaps PHPUnit tests are missing @covers tags. Those tags
are necessary for PHPUnit to accurately calculate test coverage.

See the PHPUnit documentation for how to use it.

Once the tags have been added, the MissingCovers sniff should be re-enabled in .phpcs.xml.

If you are not already familiar with PHPUnit tests and would like to learn more about this: https://www.mediawiki.org/wiki/Manual:PHP_unit_testing/Writing_unit_tests has links to some good material on the basics of testing.

Mentors: @Legoktm

Event Timeline

Vvjjkkii renamed this task from MultiMaps extension is missing @covers tags to kmcaaaaaaa.Jul 1 2018, 1:08 AM
Vvjjkkii triaged this task as High priority.
Vvjjkkii updated the task description. (Show Details)
Vvjjkkii removed a subscriber: Aklapper.
CommunityTechBot renamed this task from kmcaaaaaaa to MultiMaps extension is missing @covers tags.Jul 1 2018, 5:16 PM
CommunityTechBot raised the priority of this task from High to Needs Triage.
CommunityTechBot updated the task description. (Show Details)
CommunityTechBot added a subscriber: Aklapper.

Change 852833 had a related patch set uploaded (by Fomafix; author: Fomafix):

[mediawiki/extensions/MultiMaps@master] Add missing @covers tags to test cases

https://gerrit.wikimedia.org/r/852833

Change 852833 merged by Fomafix:

[mediawiki/extensions/MultiMaps@master] Add missing @covers tags to test cases

https://gerrit.wikimedia.org/r/852833