Page MenuHomePhabricator

Set up extension tests for Parsoid repo
Open, HighPublic


In T224979: Parsoid REST Routes Extension, we have turned the Parsoid library repo into a makeshift extension, to use for testing core integration until the codebase stabilizes and can be moved into core (or maybe a proper extension). We'll need to set up extension tests for that.

Event Timeline

Tgr created this task.Jul 5 2019, 5:24 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 5 2019, 5:24 PM
hashar added a subscriber: hashar.

Should be straight forward. It is usually all about adding an entry in integration/config.git in zuul/layout.yaml

cscott added a subscriber: cscott.Jul 8 2019, 3:33 PM

We should update our developer docs as well to reflect best practices to installing Parsoid & running PHP tests. Should Parsoid be installed inside core/extensions/Parsoid and/or is setting $MW_INSTALL_PATH sufficient? Ideally composer extension-test or something would Do The Right Thing assuming proper set up (whether that's installing Parsoid inside the extension folder or setting MW_INSTALL_PATH or whatever else the Right Thing needs to be).

ssastry moved this task from Backlog to Deployment on the Parsoid-PHP board.Jul 26 2019, 12:50 AM
ssastry moved this task from Deployment to Testing / QA on the Parsoid-PHP board.Jul 30 2019, 4:56 PM
Arlolra triaged this task as High priority.Wed, Dec 4, 9:47 PM
Arlolra added a subscriber: Arlolra.

Since Parsoid/PHP is in production, a train rollout without proper integration testing could have dire effects.