Page MenuHomePhabricator

Increase test coverage for non-View files with 0% coverage
Closed, ResolvedPublic1 Estimated Story Points

Description

According to https://www.mediawiki.org/wiki/Reading/Web/Projects/Invest_in_the_MobileFrontend_%26_MinervaNeue_frontend_architecture/Progress?useskin=vector several modules have no unit tests.

Let's add some.

Acceptance criteria

  • mobile.startup/context.js has unit tests (hopefully 100% coverage)
  • mobile.startup/icons.js has unit tests (hopefully 100% coverage)

Sign off steps

Event Timeline

Jdlrobson set the point value for this task to 1.

Change 472069 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] Add tests for context.js

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

Change 472070 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] Add test coverage for the icons helper functions

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

Change 472071 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] Release new dist files

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

Change 472071 abandoned by Jdlrobson:
Release new dist files

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

Change 472069 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Add tests for context.js

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

Change 472070 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Add test coverage for the icons helper functions

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

100% test coverage for icons.js and context.js!

nray removed nray as the assignee of this task.
nray subscribed.