Sfoglia il codice sorgente

Merge pull request #535 from ckeditor/i/6002

Tests: Fixed tests leaking editor instances / DOM elements. See ckeditor/ckeditor5#6002.
Kamil Piechaczek 6 anni fa
parent
commit
0636300d1f
1 ha cambiato i file con 6 aggiunte e 2 eliminazioni
  1. 6 2
      packages/ckeditor5-ui/tests/dropdown/dropdownview.js

+ 6 - 2
packages/ckeditor5-ui/tests/dropdown/dropdownview.js

@@ -186,7 +186,7 @@ describe( 'DropdownView', () => {
 
 	describe( 'render()', () => {
 		it( 'starts listening for #keystrokes coming from #element', () => {
-			view = new DropdownView( locale,
+			const view = new DropdownView( locale,
 				new ButtonView( locale ),
 				new DropdownPanelView( locale ) );
 
@@ -195,10 +195,12 @@ describe( 'DropdownView', () => {
 			view.render();
 			sinon.assert.calledOnce( spy );
 			sinon.assert.calledWithExactly( spy, view.element );
+
+			view.element.remove();
 		} );
 
 		it( 'adds #element to #focusTracker', () => {
-			view = new DropdownView( locale,
+			const view = new DropdownView( locale,
 				new ButtonView( locale ),
 				new DropdownPanelView( locale ) );
 
@@ -207,6 +209,8 @@ describe( 'DropdownView', () => {
 			view.render();
 			sinon.assert.calledOnce( spy );
 			sinon.assert.calledWithExactly( spy, view.element );
+
+			view.element.remove();
 		} );
 
 		describe( 'activates keyboard navigation for the dropdown', () => {