Test coverage of the ApiVisualEditor class, which loads the editor via the API, is quite low. ApiVisualEditorTest currently covers just the static methods.
We should add a basic test to cover ApiVisualEditor::execute, the function that loads the editor. This is fairly straightforward if we update the the ApiVisualEditorTest class to extend ApiTestCase.
Doing this should make it easier to add more fine-tuned tests later.
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Add basic test for ApiVisualEditor::execute | mediawiki/extensions/VisualEditor | master | +20 -2 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T284512 [EPIC] Empower others to develop new, and extend existing, editing experiences | |||
| Open | None | T299620 Implement automated testing for core editing functionality | |||
| Resolved | • Esanders | T299791 Implement basic integration test for loading the editor via ApiVisualEditor |
Change 756040 had a related patch set uploaded (by Tchanders; author: Tchanders):
[mediawiki/extensions/VisualEditor@master] Add basic test for ApiVisualEditor::execute
Change 756040 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Add basic test for ApiVisualEditor::execute