Page MenuHomePhabricator

MFA: View and Icon test coverage to 100%
Closed, DeclinedPublic


The View and Icon classes are widely used inside MobileFrontend but do not enjoy good code coverage.

File% Stmts% Branch% Funcs% LinesUncovered Line #s
Icon.js58.3333.3310058.33... 48,50,51,53,55
View.js93.1584.3888.2493.06... 99,201,202,396

Acceptance criteria

  • 100% test coverage for both or good documented reasons why this is not possible.

Sign off steps

Event Timeline

Jdlrobson created this task.
Jdlrobson renamed this task from AMC: View and Icon test coverage to 100% to MFA: View and Icon test coverage to 100%.Dec 17 2018, 4:38 PM
Jdlrobson lowered the priority of this task from High to Medium.Dec 18 2018, 12:38 AM
Jdlrobson raised the priority of this task from Medium to Needs Triage.Feb 26 2019, 7:10 PM

This appears to be the last task blocking T195478 resolution. Offhandedly, ~90% View coverage sounds great but maybe Icon's tests can be improved.

I'd take the fact this is the only subtask with a pinch of salt. We are improving coverage as part of refactoring so maybe that epic should be a subtask of this one.

In I split this into two epics. If that's helpful we could also do that here.

Not a priority any more.