HtmlEditor: Rename isStringInsert & restore mock in tests

Author: Ruslan Farkhutdinov

packages/devextreme/js/__internal/ui/html_editor/matchers/m_wordLists.ts

@@ -1,3 +1,5 @@
+import { isString } from '@js/core/utils/type';
+
 function getListType(matches) {
   const prefix = matches[1];
   return prefix.match(/\S+\./) ? 'ordered' : 'bullet';
@@ -21,10 +23,6 @@ function removeNewLineChar(operations) {
   newLineOperation.insert = newLineOperation.insert.trim();
 }
 
-function isTextOp(op): boolean {
-  return typeof op === 'string';
-}
-
 const getMatcher = (quill) => {
   const Delta = quill.import('delta');
   const msStyleAttributeName = quill.MS_LIST_DATA_KEY;
@@ -34,7 +32,7 @@ const getMatcher = (quill) => {
 
     const insertOperation = ops[0];
 
-    if (!isTextOp(insertOperation.insert)) {
+    if (!isString(insertOperation.insert)) {
       return delta;
     }
 

packages/devextreme/testing/tests/DevExpress.ui.widgets.htmlEditor/htmlEditorParts/multilineIntegration.tests.js

@@ -70,22 +70,23 @@ export default function() {
                 test(`editor should not throw error for ${className} class with allowSoftLineBreak (T1292587)`, function(assert) {
                     const spy = sinon.spy(console, 'error');
                     const value = `<p class="${className}"><br><br></p>`;
-                    const $element = $('#htmlEditor');
-                    $element.dxHtmlEditor({
-                        value
-                    }).dxHtmlEditor('instance');
 
-                    assert.strictEqual(spy.called, false, 'No console error was thrown');
-                    spy.restore();
+                    try {
+                        $('#htmlEditor').dxHtmlEditor({ value }).dxHtmlEditor('instance');
+
+                        assert.strictEqual(spy.called, false, 'No console error was thrown');
+                    } finally {
+                        spy.restore();
+                    }
                 });
 
                 test(`editor should preserve double <br> in list paragraph for ${className} class (T1292587)`, function(assert) {
                     const value = `<p class="${className}"><br><br></p>`;
-                    const $element = $('#htmlEditor');
-                    $element.dxHtmlEditor({
+                    const htmlEditor = $('#htmlEditor').dxHtmlEditor({
                         value
                     }).dxHtmlEditor('instance');
-                    const markup = $element.find(`.${CONTENT_CLASS}`).html();
+
+                    const markup = htmlEditor.option('value');
                     const $markup = $('<div>').html(markup);
 
                     assert.strictEqual($markup.find('br').length, 2, 'Two soft breaks preserved');