Page MenuHomePhabricator

Remove VisualEditor browser test inside MobileFrontend
Closed, ResolvedPublic0 Estimated Story Points

Description

There is a browser test tests/browser/features/editor_ve.feature which checks integration of VisualEditor with MobileFrontend (ie. you can switch between modes, go back to save screen and edit a page using VE). It is skipped in all environments (on every commit, on beta cluster).

It's worth noting that the browser test seems a little outdated and css selectors need updating for this to pass again.

A/C

  • Remove the tests mentioned above

Event Timeline

@Esanders @Jdforrester-WMF any preference on what we should do? After a brief discussion we are proposing to remove it, but out of courtesy we would like to check with your team.

bmansurov renamed this task from VisualEditor browser test inside MobileFrontend are skipped to Remove VisualEditor browser test inside MobileFrontend.Dec 14 2016, 2:28 PM
bmansurov updated the task description. (Show Details)
MtDu subscribed.

I'll clean this up. :)

Change 332082 had a related patch set uploaded (by MtDu):
Remove VisualEditor browser test inside MobileFrontend

https://gerrit.wikimedia.org/r/332082

Does anyone have time to give this a review and close this up? :)

Change 332082 merged by jenkins-bot:
Remove VisualEditor browser test inside MobileFrontend

https://gerrit.wikimedia.org/r/332082

Jdforrester-WMF set the point value for this task to 0.