Page MenuHomePhabricator
Create Task
Maniphest T208987

PHPUnit: Cover all model/MobileCollection class methods
Closed, ResolvedPublic
Actions

Description

The class in model/MobileCollection.php file has no code coverage. Tests should be written to fix this.

Acceptance criteria

  • Write unit tests to cover all methods in /models/MobileCollection.php.
  • Code coverage report has gone up.

Related Objects
Search...

Event Timeline

D3r1ck01 created this task.Nov 7 2018, 7:26 PM
• Jdlrobson moved this task from To Triage to Triaged but Future on the Readers-Web-Backlog board.Nov 7 2018, 9:45 PM
• Jdlrobson triaged this task as Normal priority.
gerritbot added a subscriber: gerritbot.Nov 7 2018, 10:22 PM

Change 472341 had a related patch set uploaded (by D3r1ck01; owner: Alangi Derick):
[mediawiki/extensions/MobileFrontend@master] Add MobileCollectionTest for MobileCollection::class

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

D3r1ck01 claimed this task.Nov 7 2018, 10:24 PM
D3r1ck01 moved this task from Backlog to Doing [WIP] on the User-D3r1ck01 board.
gerritbot added a comment.Nov 8 2018, 12:36 PM

Change 472427 had a related patch set uploaded (by D3r1ck01; owner: Alangi Derick):
[mediawiki/extensions/MobileFrontend@master] Add MobilePageTest to test MobilePage::class

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

gerritbot added a comment.Nov 9 2018, 9:12 PM

Change 472341 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Add MobileCollectionTest for MobileCollection::class

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

D3r1ck01 updated the task description. (Show Details)Nov 9 2018, 9:28 PM
D3r1ck01 updated the task description. (Show Details)Nov 9 2018, 9:32 PM
gerritbot added a comment.Nov 14 2018, 11:26 AM

Change 473454 had a related patch set uploaded (by D3r1ck01; owner: Alangi Derick):
[mediawiki/extensions/MobileFrontend@master] Fix $expected and $actual values in MobileCollectionTest

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

D3r1ck01 renamed this task from PHPUnit: Cover all model/* class methods to PHPUnit: Cover all model/MobileCollection class methods.Nov 14 2018, 11:52 AM
D3r1ck01 updated the task description. (Show Details)

I've separated this task and here is the other half: T209476. Once the patches here are merged, this can be resolved. Would want to tackle these tasks in smaller bits :)

D3r1ck01 updated the task description. (Show Details)Nov 14 2018, 12:01 PM
gerritbot added a comment.Nov 14 2018, 6:08 PM

Change 473454 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Fix $expected and $actual values in MobileCollectionTest

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

• Jdlrobson moved this task from Ready for Signoff to Needs More Work on the Readers-Web-Backlog (Readers-Web-Kanbanana-Board-2018-19-Q2) board.EditedNov 14 2018, 6:16 PM

https://gerrit.wikimedia.org/r/472427 i still open and has a -1? Should that be abandoned or is that still being done?

D3r1ck01 added a comment.EditedNov 14 2018, 6:19 PM

I separated the task, so that now is tracked here: T209476. See: T208987#4745948.

phuedx closed this task as Resolved.Nov 15 2018, 5:30 PM
phuedx added a subscriber: phuedx.
+-----------------------------+-------+--------+
| Filename                    | Old % | New %  |
+-----------------------------+-------+--------+
| models/MobileCollection.php | 0     | 100.00 |
+-----------------------------+-------+--------+

🎉🎉🎉

gerritbot added a comment.Nov 15 2018, 8:15 PM

Change 472427 had a related patch set uploaded (by D3r1ck01; owner: Alangi Derick):
[mediawiki/extensions/MobileFrontend@master] Add MobilePageTest to test MobilePage::class

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

D3r1ck01 added a comment.Nov 15 2018, 8:19 PM

Ops, wrong ticket reference :(

Content licensed under Creative Commons Attribution-ShareAlike 3.0 (CC-BY-SA) unless otherwise noted; code licensed under GNU General Public License (GPL) or other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL