Code coverage for the content-providers/MwApiContentProvider.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%
Code coverage for the content-providers/MwApiContentProvider.php is 0% and unit tests should be written to cover all it's code.
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Declined | None | T208761 [Epic] Improve PHPUnit coverage in MobileFrontend to 50% | |||
Resolved | • pmiazga | T212666 PHPUnit: Cover methods in content-providers/MwApiContentProvider.php with unit tests |
Change 481841 had a related patch set uploaded (by D3r1ck01; owner: Derick Alangi):
[mediawiki/extensions/MobileFrontend@master] tests: Add PHPUnit tests for MwApiContentProviderTest::class
@Jdlrobson, last but one additional code path has now been covered, we're at 100% 🎉🎉🎉🎉🎉🎉🎉
Change 481841 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] tests: Add PHPUnit tests for MwApiContentProviderTest::class
Change 486492 had a related patch set uploaded (by Pmiazga; owner: Pmiazga):
[mediawiki/extensions/MobileFrontend@master] Hygiene: simplify the if statement
Change 486492 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Hygiene: simplify the if statement
Thank you very much for the recent patch :) That folder coverage really looks good :)