View.js has already been migrated to webpack's control via T203100, but its associated tests proved to be challenging to migrate due to tight coupling with dependencies ( e.g. jQuery) and a lot of stubbing in its place. It was thus decided to leave its test file as is and migrate it in a future card (this one).
To review the work already done on this, see (and feel free to abandon): https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/MobileFrontend/+/458883/
A discussion needs to be had on how much stubbing / spying we feel comfortable with on this.
- test_View.js is migrated to tests/node-qunit/mobile.startup and can be run from headless qunit
- Revise/move/remove any tests that don't test View's responsibility. Note: some tests feel like they test the functionality of mfExtend more than they test View's code
Note: Adding additional test coverage is out of scope and will be added separately.
Sign off steps
- Measure and record code coverage on the wiki page