Merge pull request #1812 from ckeditor/i/6002

Tests: Fixed tests leaking editor instances / DOM elements. See ckeditor/ckeditor5#6002.
ckeditor · Dec 20, 2019 · fd8c9f6 · fd8c9f6
2 parents b7e2bfe + 1e8f90a
commit fd8c9f6
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/tests/view/observer/focusobserver.js b/tests/view/observer/focusobserver.js
@@ -165,6 +165,11 @@ describe( 'FocusObserver', () => {
 			observer = view.getObserver( FocusObserver );
 		} );
 
+		afterEach( () => {
+			view.destroy();
+			domRoot.remove();
+		} );
+
 		it( 'should always render document after selectionChange event', done => {
 			const selectionChangeSpy = sinon.spy();
 			const renderSpy = sinon.spy();

diff --git a/tests/view/renderer.js b/tests/view/renderer.js
@@ -2009,6 +2009,8 @@ describe( 'Renderer', () => {
 				container = document.getSelection().anchorNode;
 
 				expect( domEditable.contains( container ) ).to.be.true;
+
+				domEditable.remove();
 			} );
 
 			it( 'should bind fake selection container to view selection', () => {
@@ -3721,6 +3723,11 @@ describe( 'Renderer', () => {
 			converter = view.domConverter;
 		} );
 
+		afterEach( () => {
+			view.destroy();
+			domRoot.remove();
+		} );
+
 		it( 'should properly render unwrapped attributes #1', () => {
 			setViewData( view,
 				'<container:p>' +