Nicolas Bayet
3fa7894106
1) Replace editor function that change the editor internal structure
with a new mechansim that does it automatically by observing dom
mutations. Welcome wysiwyg.withDomMutations.
2) Add a mechanism to reuse the active editor execution context
when a wysiwyg.execCommand or wysiwyg.withDomMutations is called
inside itself.
Example:
await wysiwyg.execCommand(async ()=> {
...
await wysiwyg.execCommand(()=>{...})
await wysiwyg.withDomMutations(()=>{...})
})
closes odoo/odoo#63237
X-original-commit: fe7056ccb450c9767693671e8d980a6aebacd6ab
Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>
68 lines
2.2 KiB
JavaScript
68 lines
2.2 KiB
JavaScript
odoo.define('web_editor.wysiwyg.test', function (require) {
|
|
"use strict";
|
|
|
|
var Wysiwyg = require('web_editor.wysiwyg');
|
|
|
|
QUnit.module('web_editor', {}, function () {
|
|
|
|
QUnit.module('web_editor.wysiwyg');
|
|
|
|
QUnit.test('should automatically get the context when calling execCommand inside execCommand', async function (assert) {
|
|
assert.expect(1);
|
|
|
|
const div = document.createElement('div');
|
|
|
|
const wysiwyg = new Wysiwyg(undefined, { location: [div, 'append'] });
|
|
await wysiwyg.start();
|
|
const calls = [];
|
|
|
|
await wysiwyg.execCommand(async () => {
|
|
calls.push(1);
|
|
await wysiwyg.execCommand(() => {
|
|
calls.push(2);
|
|
});
|
|
});
|
|
await wysiwyg.execCommand(() => {
|
|
calls.push(3);
|
|
});
|
|
|
|
assert.deepEqual(calls, [1, 2, 3], "all command should have been called in the proper order");
|
|
});
|
|
QUnit.test('should automatically get the context when calling execCommand (without await) inside execCommand', async function (assert) {
|
|
assert.expect(1);
|
|
|
|
const div = document.createElement('div');
|
|
|
|
const wysiwyg = new Wysiwyg(undefined, { location: [div, 'append'] });
|
|
await wysiwyg.start();
|
|
const calls = [];
|
|
|
|
const promises = [];
|
|
|
|
promises.push(wysiwyg.execCommand(async () => {
|
|
calls.push(1);
|
|
await wysiwyg.execCommand(() => {
|
|
calls.push(2);
|
|
});
|
|
await wysiwyg.execCommand(() => {
|
|
calls.push(4);
|
|
});
|
|
}));
|
|
|
|
// Another execCommand can occurs before the previous promise finish.
|
|
// For example, in case there is a click handler that call
|
|
// wysiwyg.execCommand or wysiwyg.withDomMutations while another
|
|
// wysiwyg.execCommand or wysiwyg.withDomMutations is not finished.
|
|
promises.push(wysiwyg.execCommand(async () => {
|
|
await new Promise(resolve => setTimeout(resolve, 0));
|
|
calls.push(3);
|
|
}));
|
|
|
|
await Promise.all(promises);
|
|
|
|
assert.deepEqual(calls, [1, 2, 4, 3], "all command should have been called in the proper order");
|
|
});
|
|
});
|
|
|
|
});
|