Page MenuHomePhabricator

PHPUnit: Cover methods in content-providers/ContentProviderFactory.php with unit tests
Closed, ResolvedPublic


Code coverage for the content-providers/ContentProviderFactory.php is 0% and unit tests should be written to cover all it's code.

Acceptance Criteria

  • Cover all methods in the file class
  • Code Coverage has gone up to 100%

Event Timeline

xSavitar triaged this task as Medium priority.
xSavitar created this task.
xSavitar moved this task from Backlog to Doing [WIP] on the User-xSavitar board.

Change 475754 had a related patch set uploaded (by D3r1ck01; owner: Alangi Derick):
[mediawiki/extensions/MobileFrontend@master] Add PHPUnit tests for testing methods in ContentProviderFactory::class

@pmiazga, @Jdlrobson, I think this looks good for R1 review ;)

A gentle plead for another review considering the comments :} Thanks!

Change 475754 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Add PHPUnit tests for methods in ContentProviderFactory::class

Code Coverage has gone up to 100%

From, run against PS12 of

| Filename                                     | Old % | New %  |
| content-providers/ContentProviderFactory.php | 0     | 100.00 |
phuedx claimed this task.
phuedx updated the task description. (Show Details)

Being bold.