浏览代码

Tests: Fixed tests leaking editor instance / DOM element.

Marek Lewandowski 6 年之前
父节点
当前提交
39ae2988cf
共有 1 个文件被更改,包括 7 次插入2 次删除
  1. 7 2
      packages/ckeditor5-table/tests/converters/table-cell-refresh-post-fixer.js

+ 7 - 2
packages/ckeditor5-table/tests/converters/table-cell-refresh-post-fixer.js

@@ -17,12 +17,12 @@ import ClassicTestEditor from '@ckeditor/ckeditor5-core/tests/_utils/classictest
 import Delete from '@ckeditor/ckeditor5-typing/src/delete';
 import Delete from '@ckeditor/ckeditor5-typing/src/delete';
 
 
 describe( 'Table cell refresh post-fixer', () => {
 describe( 'Table cell refresh post-fixer', () => {
-	let editor, model, doc, root, view, refreshItemSpy;
+	let editor, model, doc, root, view, refreshItemSpy, element;
 
 
 	testUtils.createSinonSandbox();
 	testUtils.createSinonSandbox();
 
 
 	beforeEach( () => {
 	beforeEach( () => {
-		const element = document.createElement( 'div' );
+		element = document.createElement( 'div' );
 		document.body.appendChild( element );
 		document.body.appendChild( element );
 
 
 		// Most tests assume non-edge environment but we do not set `contenteditable=false` on Edge so stub `env.isEdge`.
 		// Most tests assume non-edge environment but we do not set `contenteditable=false` on Edge so stub `env.isEdge`.
@@ -53,6 +53,11 @@ describe( 'Table cell refresh post-fixer', () => {
 			} );
 			} );
 	} );
 	} );
 
 
+	afterEach( () => {
+		element.remove();
+		return editor.destroy();
+	} );
+
 	it( 'should rename <span> to <p> when adding more <paragraph> elements to the same table cell', () => {
 	it( 'should rename <span> to <p> when adding more <paragraph> elements to the same table cell', () => {
 		editor.setData( viewTable( [ [ '<p>00</p>' ] ] ) );
 		editor.setData( viewTable( [ [ '<p>00</p>' ] ] ) );