- Verify that , the CE is editable 776963
- Verify you can save the page 721024 801729 T309712
- Automate VE Toolbar actions 844470
- Verify you can insert/modify/remove table 841915
- Verify you can edit/modify/remove a link
790682790683 - Verify you can insert/modify/remove comment
811704 - Verify you can apply different heading style on a text
- Very undo and redo action works properly
- Verify you can change the effect on any text by applying different formatting options from the toolbar
- Verify you can add all kind of citations from the "Cite" menu
- Verify you can clear styling from a text
- Verify you can edit/remove any existing citation
- Check the functionality of list and indentation
- Verify you can insert/modify/remove image
- Verify you can insert/modify/remove template
- Verify you can insert/modify/remove formula
- Verify you can insert/modify/remove reference list
- Verify the Re-use existing reference works properly
- Verify you can insert special character
- Verify you can add categories to the page you are editing
- Verify you can edit different properties inside Media Settings dialog
Description
Details
| 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 | |||
| Open | None | T296187 Selenium Tests for VisualEditor | |||
| Resolved | kostajh | T284611 Selenium tests for add link feature | |||
| Resolved | zeljkofilipin | T309712 Dismiss 'Try Content Translation' popup | |||
| Open | None | T401573 Flaky VisualEditor Selenium test: Content Editable should insert a table | |||
| Open | None | T337864 Error in "Toolbar.should open special characters menu" |
Event Timeline
Change 802528 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Run only one test in Jenkins daily job
Change 801729 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Better assertion after saving the page
Change 803937 had a related patch set uploaded (by Esther Akinloose; author: Esther Akinloose):
[mediawiki/extensions/VisualEditor@master] WIP selenium: saveComplete
Change 803937 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Add EditPage.saveComplete()
Change 805385 had a related patch set uploaded (by Esther Akinloose; author: Esther Akinloose):
[mediawiki/extensions/VisualEditor@master] selenium: Do not retry tests
Change 805385 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Do not retry tests
Change 790683 abandoned by Esther Akinloose:
[mediawiki/extensions/VisualEditor@master] selenium: Link icon is clickable
Reason:
Change 811704 had a related patch set uploaded (by Esther Akinloose; author: Esther Akinloose):
[mediawiki/extensions/VisualEditor@master] WIP selenium: Comment should be added
Change 790682 abandoned by Esther Akinloose:
[mediawiki/extensions/VisualEditor@master] WIP selenium: Link should be added
Reason:
Change 811704 abandoned by Esther Akinloose:
[mediawiki/extensions/VisualEditor@master] WIP selenium: Comment should be added
Reason:
Will get back to this.
Change 841128 had a related patch set uploaded (by Esther Akinloose; author: Esther Akinloose):
[mediawiki/extensions/VisualEditor@master] selenium: Move code to beforeEach and afterEach hooks
Change 841128 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Move code to beforeEach and afterEach hooks
Change 841915 had a related patch set uploaded (by Zfilipin; author: Zfilipin):
[mediawiki/extensions/VisualEditor@master] WIP selenium: Content editable should insert a table
Change 841915 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Content editable should insert a table
Change 844470 had a related patch set uploaded (by Esther Akinloose; author: Esther Akinloose):
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open switch editor
Change 844470 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open switch editor
Change 844855 had a related patch set uploaded (by Esther Akinloose; author: Esther Akinloose):
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open page options
Change 793478 abandoned by Zfilipin:
[mediawiki/extensions/VisualEditor@master] WIP selenium: Wait 50 seconds to see in the video recording if the page is saved
Reason:
Change 862243 had a related patch set uploaded (by Esther Akinloose; author: Esther Akinloose):
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open help popup
Change 862263 had a related patch set uploaded (by Esther Akinloose; author: Esther Akinloose):
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open notices popup as soon as it loads
Change 844855 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open page options
Change 862243 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open help popup
Change 862269 had a related patch set uploaded (by Esther Akinloose; author: Esther Akinloose):
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open special characters menu
Change 862263 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open notices popup as soon as it loads
Change 862269 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open special characters menu
Change 863400 had a related patch set uploaded (by Esther Akinloose; author: Esther Akinloose):
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open insert menu
Change 863404 had a related patch set uploaded (by Esther Akinloose; author: Esther Akinloose):
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open structure options menu
Change 863405 had a related patch set uploaded (by Esther Akinloose; author: Esther Akinloose):
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open style text options
Change 863407 had a related patch set uploaded (by Esther Akinloose; author: Esther Akinloose):
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open format paragraph menu
Change 863400 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open insert menu
Change 866581 had a related patch set uploaded (by Zfilipin; author: Zfilipin):
[mediawiki/extensions/VisualEditor@master] selenium: Decrease logLevel from info to error
Change 866581 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Decrease logLevel from info to error
Change 863404 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open structure options menu
Change 863405 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open style text options
Change 863407 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Toolbar should open format paragraph menu
I believe we would have caught the errors in T332415#8828832 before the patch was merged if we verified editing behavior as a logged-out user.
I think for the Temporary accounts project, anonymous editing tests will be important to have in place to catch regressions before patches get merged.
Change 931997 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):
[mediawiki/extensions/VisualEditor@master] selenium: Disable flaky test
Change 931997 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] selenium: Disable flaky test
Change 941542 had a related patch set uploaded (by Esther Akinloose; author: Esther Akinloose):
[mediawiki/extensions/VisualEditor@master] selenium: undo, redo buttons are displayed disabled
@EAkinloose: Removing task assignee as this open task has been assigned for more than two years - see the email sent to all task assignees on 2024-04-15.
Please assign this task to yourself again if you still realistically [plan to] work on this task - it would be welcome! :)
If this task has been resolved in the meantime, or should not be worked on by anybody ("declined"), please update its task status via "Add Action… 🡒 Change Status".
Also see https://www.mediawiki.org/wiki/Bug_management/Assignee_cleanup for tips how to best manage your individual work in Phabricator. Thanks!
Note: While working in the WMDE-References-FocusArea we added Cypress e2e tests for the VisualEditor Cite extension. See
Where automation might already exist
VisualEditor
Various QUnit tests are implemented. I don't fully understand what they do or how they work, so this is just my best guess.
- Verify you can insert/modify/remove table
- Verify you can edit/modify/remove a link
- Verify you can insert/modify/remove comment
- Verify you can apply different heading style on a text
- Very undo and redo action works properly
Some of the QUnit tests already test what happens on undo, for example.
- Verify you can change the effect on any text by applying different formatting options from the toolbar
- Verify you can clear styling from a text
- Check the functionality of list and indentation
https://gerrit.wikimedia.org/r/plugins/gitiles/VisualEditor/VisualEditor/+/refs/heads/master/tests/ui/actions/ve.ui.ListAction.test.js
https://gerrit.wikimedia.org/r/plugins/gitiles/VisualEditor/VisualEditor/+/refs/heads/master/tests/ui/actions/ve.ui.IndentationAction.test.js
- Verify you can insert special character
Cite extension
Cite extension has cypress (as mentioned T296187#10272672) and qunit tests.
- Verify you can add all kind of citations from the "Cite" menu
Unsure.
- Verify you can edit/remove any existing citation
- Verify you can insert/modify/remove reference list
Viewing existing reference list: https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/extensions/Cite/+/1244b3901c0a3603770ddadf3480505d3e28cd36/tests/cypress/e2e/tests/ve-cite/veCiteIntegration.cy.js#50
- Verify the Re-use existing reference works properly
Other
- Verify you can insert/modify/remove template
TemplateWizard extension has selenium and qunit tests, but I don't think these test the TemplateWizard in VisualEditor.
- Verify you can insert/modify/remove formula
Math extension has Selenium tests, but they are very basic and don't appear to cover VisualEditor at all.
Uncertain where/if these have automation
- Automate VE Toolbar actions
- Verify you can insert/modify/remove image
- Verify you can add categories to the page you are editing
- Verify you can edit different properties inside Media Settings dialog
Change #1206926 had a related patch set uploaded (by Dom Walden; author: Dom Walden):
[mediawiki/extensions/VisualEditor@master] WIP Selenium: Inserting table
Change #1207145 had a related patch set uploaded (by Dom Walden; author: Dom Walden):
[mediawiki/extensions/VisualEditor@master] WIP Selenium: Inserting internal link
Change #1208279 had a related patch set uploaded (by Dom Walden; author: Dom Walden):
[mediawiki/extensions/VisualEditor@master] WIP Selenium: Inserting external link
Change #1208283 had a related patch set uploaded (by Dom Walden; author: Dom Walden):
[mediawiki/extensions/VisualEditor@master] WIP Selenium: Inserting comment
Change #1208295 had a related patch set uploaded (by Dom Walden; author: Dom Walden):
[mediawiki/extensions/VisualEditor@master] Selenium: Changing heading styles on text
Change #1208308 had a related patch set uploaded (by Dom Walden; author: Dom Walden):
[mediawiki/extensions/VisualEditor@master] Selenium: Changing formatting
Change #1208312 had a related patch set uploaded (by Dom Walden; author: Dom Walden):
[mediawiki/extensions/VisualEditor@master] Selenium: Inserting and indenting lists