Page MenuHomePhabricator

JsonData extension is missing @covers tags
Closed, ResolvedPublic

Description

PHPCS has identified that some of the JsonData 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 JsonData extension is missing @covers tags to pmcaaaaaaa.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.
JJMC89 renamed this task from pmcaaaaaaa to JsonData extension is missing @covers tags.Jul 1 2018, 2:27 AM
JJMC89 raised the priority of this task from High to Needs Triage.
JJMC89 updated the task description. (Show Details)
JJMC89 added a subscriber: Aklapper.

Change 471520 had a related patch set uploaded (by Zoranzoki21; owner: Zoranzoki21):
[mediawiki/extensions/JsonData@master] Add missing @covers tags in tests

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

Change 471520 abandoned by Zoranzoki21:
Add missing @covers tags in tests

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

Kizule removed Kizule as the assignee of this task.Nov 4 2018, 5:45 PM
Kizule removed projects: Patch-For-Review, User-Kizule.
Kizule edited subscribers, added: Kizule; removed: gerritbot.
Kizule unsubscribed.

Change 471563 had a related patch set uploaded (by Arcayn; owner: Arcayn):
[mediawiki/extensions/JsonData@master] Update @covers tags and reenable rule for phpcs

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

Change 471563 merged by jenkins-bot:
[mediawiki/extensions/JsonData@master] Update @covers tags and reenable rule for phpcs

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