PolicyStat / ckeditor-spell-check-plugin-js-dev-edge

User interface enhancements and beta features for http://ckeditor-spellcheck.nanospell.com . This repo is for nanospell developers and trusted partners to modify and fork plugin.js
Other
0 stars 0 forks source link

Fix the backwards events tests #44

Closed caffodian closed 7 years ago

caffodian commented 7 years ago

Stumbled into this when working on #39 - our event observer actually fires backwards (and our tests catch things in reverse order) because the event priority is wrong. Kinda.

caffodian commented 7 years ago

The problem to now look into is...if we fix the order (by ensuring the observer catches the events before the actual handlers), how much of the test breaks (because it assumes the wrong order)