From 2a010c38f3d22e352c8816efbaaff0a4d116c0d7 Mon Sep 17 00:00:00 2001 From: Antoine Guenet Date: Tue, 19 Feb 2019 13:40:26 +0000 Subject: [PATCH] [FIX] web_editor: remove broken and unnecessary test The next test was identical to the removed one, with the exception that it inserts a character at the end, which really is the behavior we want to test. The removed test was broken because it expected the range on the parent, which is anyway equivalent in this case. This is confirmed by the fact that inserting a character at either ranges indeed inserts the character where it is expected to do so. --- .../static/tests/wysiwyg_keyboard_tests.js | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/addons/web_editor/static/tests/wysiwyg_keyboard_tests.js b/addons/web_editor/static/tests/wysiwyg_keyboard_tests.js index 14f06ec2e9d..e8696a802c5 100644 --- a/addons/web_editor/static/tests/wysiwyg_keyboard_tests.js +++ b/addons/web_editor/static/tests/wysiwyg_keyboard_tests.js @@ -3092,22 +3092,6 @@ var keyboardTestsBackspace = [{ start: "p:contents()[0]->4", }, }, - { - name: "in complex-dom (empty-td (td before) -> 2x SHIFT-ENTER): 3x BACKSPACE", - content: '

dom not to edit




', - steps: [{ - start: 'p:eq(1)->3', - key: 'BACKSPACE', - }, { - key: 'BACKSPACE', - }, { - key: 'BACKSPACE', - }], - test: { - content: '

dom not to edit


', - start: "p:eq(1):contents()[0]->0", - }, - }, { name: "in complex-dom (empty-td (td before) -> 2x SHIFT-ENTER): 3x BACKSPACE -> 'a'", content: '

dom not to edit




',