Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

Commit

Permalink
Merge pull request #31 from ckeditor/i/6002
Browse files Browse the repository at this point in the history
Tests: Fixed tests leaking editor instances / DOM elements. See ckeditor/ckeditor5#6002.
  • Loading branch information
pomek committed Dec 20, 2019
2 parents d71d23a + 767295c commit 16412dd
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions tests/ckeditor.js
Expand Up @@ -45,6 +45,8 @@ describe( 'DecoupledEditor build', () => {
expect( newEditor.ui.view.element ).to.be.null;
expect( newEditor.ui.view.toolbar.element.parentElement ).to.be.null;
expect( newEditor.ui.view.editable.element.parentElement ).to.be.null;

return newEditor.destroy();
} );
} );
} );
Expand All @@ -56,6 +58,8 @@ describe( 'DecoupledEditor build', () => {
return DecoupledEditor.create( editorElement )
.then( newEditor => {
expect( newEditor.ui.view.editable.element.parentElement ).to.equal( document.body );

return newEditor.destroy();
} );
} );
} );
Expand Down

0 comments on commit 16412dd

Please sign in to comment.