Change Details

Currently CtaDrawer.js and Drawer.js have no test coverage and this makes me a sad panda. This task aims to make this 100%. = Precursors: [x] Ensure T207805 has happened before beginning on this change. Tests for tests/qunit/mobile.references/ReferencesDrawer.test.js do exist, and we'll want to ensure the initial figure for code coverage [[ https://www.mediawiki.org/wiki/Reading/Web/Projects/Invest_in_the_MobileFrontend_%26_MinervaNeue_frontend_architecture/Progress | on the wiki page ]] is accurate before starting this work. = Developer notes The Drawer and CtaDrawer are mostly logic-less templates. They do not contain gateways and are unlikely to need any kind of refactoring. = Sign off steps [] Record [[ https://www.mediawiki.org/wiki/Reading/Web/Projects/Invest_in_the_MobileFrontend_%26_MinervaNeue_frontend_architecture/Progress?useskin=vector#Impact_on_bytes_shipped | progress ]] Coverage should be 100%. If not, and there is a reason, please make a note on the table. If coverage is not 100% push this back to needs more work to find out why.

Currently CtaDrawer.js and Drawer.js have no test coverage and this makes me a sad panda. This task aims to make this 100%. = Precursors: [x] Ensure T207805 has happened before beginning on this change. Tests for tests/qunit/mobile.references/ReferencesDrawer.test.js do exist, and we'll want to ensure the initial figure for code coverage [[ https://www.mediawiki.org/wiki/Reading/Web/Projects/Invest_in_the_MobileFrontend_%26_MinervaNeue_frontend_architecture/Progress | on the wiki page ]] is accurate before starting this work. = Developer notes The Drawer and CtaDrawer are mostly logic-less templates. They do not contain gateways and are unlikely to need any kind of refactoring. = Sign off steps [x] Record [[ https://www.mediawiki.org/wiki/Reading/Web/Projects/Invest_in_the_MobileFrontend_%26_MinervaNeue_frontend_architecture/Progress?useskin=vector#Impact_on_bytes_shipped | progress ]] Coverage should be 100%. If not, and there is a reason, please make a note on the table. If coverage is not 100% push this back to needs more work to find out why.

Currently CtaDrawer.js and Drawer.js have no test coverage and this makes me a sad panda. This task aims to make this 100%. = Precursors: [x] Ensure T207805 has happened before beginning on this change. Tests for tests/qunit/mobile.references/ReferencesDrawer.test.js do exist, and we'll want to ensure the initial figure for code coverage [[ https://www.mediawiki.org/wiki/Reading/Web/Projects/Invest_in_the_MobileFrontend_%26_MinervaNeue_frontend_architecture/Progress | on the wiki page ]] is accurate before starting this work. = Developer notes The Drawer and CtaDrawer are mostly logic-less templates. They do not contain gateways and are unlikely to need any kind of refactoring. = Sign off steps [x] Record [[ https://www.mediawiki.org/wiki/Reading/Web/Projects/Invest_in_the_MobileFrontend_%26_MinervaNeue_frontend_architecture/Progress?useskin=vector#Impact_on_bytes_shipped | progress ]] Coverage should be 100%. If not, and there is a reason, please make a note on the table. If coverage is not 100% push this back to needs more work to find out why.