Page MenuHomePhabricator

VisualEditor: Make fakeIme QUnit testing more rigorous
Open, MediumPublicFeature

Description

The fakeIme tests pass IME-like event sequences into the CE Surface and check that it behaves reasonably. Currently the tests are too weak, in that they don't detect certain problematic behaviour:

  1. Calls to the simulated preventDefault do nothing at all.
  1. Some tests are disabled because they fail.
  1. Only plain text in a single blank paragraph is tested.
  1. In real usage, changing the CE surface content/selection during input will close the candidate window and commits the candidate text. The tests currently do not check for this. The test data does not show which pieces of text are candidates, since there is no way to detect this from Javascript, but we can infer that at least any text which is subsequenctly changed was a candidate.

Version: unspecified
Severity: enhancement

Details

Reference
bz57538

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 2:15 AM
bzimport added a project: Technical-Debt.
bzimport set Reference to bz57538.
Jdforrester-WMF lowered the priority of this task from High to Medium.Jan 15 2015, 12:20 AM
Jdforrester-WMF set Security to None.
Aklapper changed the subtype of this task from "Task" to "Feature Request".Feb 4 2022, 12:23 PM
Aklapper removed a subscriber: TrevorParscal.