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 #45 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 authored Jan 7, 2020
2 parents f89a01a + 39ecc45 commit 0ee879e
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions tests/alignmentui.js
Original file line number Diff line number Diff line change
Expand Up @@ -260,9 +260,9 @@ describe( 'Alignment UI', () => {
} );

describe( 'config', () => {
beforeEach( () => {
element = document.createElement( 'div' );
document.body.appendChild( element );
beforeEach( async () => {
// Clean up the editor created in main test suite hook.
await editor.destroy();

return ClassicTestEditor
.create( element, {
Expand Down Expand Up @@ -291,9 +291,9 @@ describe( 'Alignment UI', () => {
expect( dropdown.buttonView.icon ).to.equal( alignLeftIcon );
} );

it( 'should have default icon set (RTL content)', () => {
const element = document.createElement( 'div' );
document.body.appendChild( element );
it( 'should have default icon set (RTL content)', async () => {
// Clean up the editor created in main test suite hook.
await editor.destroy();

return ClassicTestEditor
.create( element, {
Expand Down

0 comments on commit 0ee879e

Please sign in to comment.