Last modified: 2014-06-28 23:24:27 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T59538, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 57538 - VisualEditor: Make fakeIme QUnit testing more rigorous
VisualEditor: Make fakeIme QUnit testing more rigorous
Status: ASSIGNED
Product: VisualEditor
Classification: Unclassified
Technical Debt (Other open bugs)
unspecified
All All
: High enhancement
: ---
Assigned To: Editing team bugs – take if you're interested!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-11-25 13:51 UTC by D Chan
Modified: 2014-06-28 23:24 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description D Chan 2013-11-25 13:51:13 UTC
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.

2. Some tests are disabled because they fail.

3. Only plain text in a single blank paragraph is tested.

4. 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.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links