Remove unnecessary workaround from tests (after moving button focus e…

…lsewhere).

packages/ckeditor5-highlight/tests/highlightui.js

@@ -179,7 +179,6 @@ describe( 'HighlightUI', () => {
 			it( 'should change button on execute option', () => {
 				command.value = 'yellowMarker';
 				validateButton( 0 );
-				button.render();
 
 				buttons[ 5 ].fire( 'execute' );
 				command.value = 'greenPen';
@@ -193,7 +192,6 @@ describe( 'HighlightUI', () => {
 			it( 'should focus view after command execution', () => {
 				const focusSpy = testUtils.sinon.spy( editor.editing.view, 'focus' );
 
-				button.render();
 				button.commandName = 'highlight';
 				button.fire( 'execute' );
 

packages/ckeditor5-image/tests/imageresize/imageresizebuttons.js

@@ -142,7 +142,6 @@ describe( 'ImageResizeButtons', () => {
 			const commandSpy = sinon.spy( command, 'execute' );
 			const resizeBy50Percent = dropdownView.listView.items._items[ 1 ].children._items[ 0 ];
 
-			dropdownView.render();
 			command.isEnabled = true;
 
 			resizeBy50Percent.fire( 'execute' );

packages/ckeditor5-image/tests/imagestyle/imagestyleui.js

@@ -427,7 +427,6 @@ describe( 'ImageStyleUI', () => {
 				const commandSpy = sinon.spy( editor, 'execute' );
 
 				for ( const { view, buttonView } of dropdowns ) {
-					buttonView.render();
 					buttonView.fire( 'execute' );
 
 					sinon.assert.notCalled( commandSpy );
@@ -469,7 +468,6 @@ describe( 'ImageStyleUI', () => {
 				const focusSpy = sinon.stub( editor.editing.view, 'focus' );
 
 				for ( const { buttonView, config, view } of dropdowns ) {
-					buttonView.render();
 					buttonView.fire( 'execute' );
 
 					expect( view.isOpen ).to.be.false;

packages/ckeditor5-list/tests/listproperties/listpropertiesui.js

@@ -262,7 +262,6 @@ describe( 'ListPropertiesUI', () => {
 				it( 'should close the drop-down when any button gets executed', () => {
 					const spy = sinon.spy();
 
-					bulletedListDropdown.render();
 					bulletedListDropdown.on( 'execute', spy );
 					listPropertiesView.fire( 'execute' );
 
@@ -284,7 +283,6 @@ describe( 'ListPropertiesUI', () => {
 					beforeEach( () => {
 						// "circle"
 						styleButtonView = stylesView.children.get( 1 );
-						bulletedListDropdown.render();
 
 						sinon.spy( editor, 'execute' );
 						sinon.spy( editor.editing.view, 'focus' );
@@ -563,7 +561,6 @@ describe( 'ListPropertiesUI', () => {
 				it( 'should close the drop-down when any button gets executed', () => {
 					const spy = sinon.spy();
 
-					numberedListDropdown.render();
 					numberedListDropdown.on( 'execute', spy );
 					listPropertiesView.fire( 'execute' );
 
@@ -585,7 +582,6 @@ describe( 'ListPropertiesUI', () => {
 					beforeEach( () => {
 						// "decimal-leading-zero""
 						styleButtonView = stylesView.children.get( 1 );
-						numberedListDropdown.render();
 
 						sinon.spy( editor, 'execute' );
 						sinon.spy( editor.editing.view, 'focus' );

packages/ckeditor5-style/tests/styleui.js

@@ -98,7 +98,6 @@ describe( 'StyleUI', () => {
 
 				testUtils.sinon.stub( editor, 'execute' );
 
-				dropdown.render();
 				dropdown.isOpen = true;
 
 				dropdown.panelView.children.first.fire( new EventInfo( buttonMock, 'execute' ) );
@@ -153,7 +152,6 @@ describe( 'StyleUI', () => {
 						}
 					};
 
-					dropdown.render();
 					dropdown.on( 'execute', spy );
 
 					panel.fire( new EventInfo( buttonMock, 'execute' ) );
@@ -168,7 +166,6 @@ describe( 'StyleUI', () => {
 						}
 					};
 
-					dropdown.render();
 					panel.fire( new EventInfo( buttonMock, 'execute' ) );
 
 					sinon.assert.calledOnceWithExactly( commandExecuteStub, 'style', { styleName: 'foo' } );

packages/ckeditor5-table/tests/tableui.js

@@ -82,7 +82,7 @@ describe( 'TableUI', () => {
 
 			tableSizeView.rows = 2;
 			tableSizeView.columns = 7;
-			insertTable.render();
+
 			insertTable.fire( 'execute' );
 
 			sinon.assert.calledOnce( executeSpy );
@@ -193,7 +193,6 @@ describe( 'TableUI', () => {
 		it( 'should focus view after command execution', () => {
 			const focusSpy = testUtils.sinon.spy( editor.editing.view, 'focus' );
 
-			dropdown.render();
 			dropdown.listView.items.get( 2 ).children.last.fire( 'execute' );
 
 			sinon.assert.calledOnce( focusSpy );
@@ -202,7 +201,6 @@ describe( 'TableUI', () => {
 		it( 'should not focus view after using a switchbutton', () => {
 			const focusSpy = testUtils.sinon.spy( editor.editing.view, 'focus' );
 
-			dropdown.render();
 			dropdown.listView.items.first.children.last.fire( 'execute' );
 
 			sinon.assert.notCalled( focusSpy );
@@ -340,7 +338,6 @@ describe( 'TableUI', () => {
 		it( 'should focus view after command execution', () => {
 			const focusSpy = testUtils.sinon.spy( editor.editing.view, 'focus' );
 
-			dropdown.render();
 			dropdown.listView.items.get( 2 ).children.first.fire( 'execute' );
 
 			sinon.assert.calledOnce( focusSpy );
@@ -349,7 +346,6 @@ describe( 'TableUI', () => {
 		it( 'should not focus view after using a switchbutton', () => {
 			const focusSpy = testUtils.sinon.spy( editor.editing.view, 'focus' );
 
-			dropdown.render();
 			dropdown.listView.items.first.children.last.fire( 'execute' );
 
 			sinon.assert.notCalled( focusSpy );
@@ -539,7 +535,6 @@ describe( 'TableUI', () => {
 		it( 'should focus view after command execution', () => {
 			const focusSpy = testUtils.sinon.spy( editor.editing.view, 'focus' );
 
-			dropdown.render();
 			dropdown.listView.items.first.children.first.fire( 'execute' );
 
 			sinon.assert.calledOnce( focusSpy );
@@ -548,7 +543,6 @@ describe( 'TableUI', () => {
 		it( 'executes command when it\'s executed', () => {
 			const spy = sinon.stub( editor, 'execute' );
 
-			dropdown.render();
 			dropdown.listView.items.first.children.first.fire( 'execute' );
 
 			expect( spy.calledOnce ).to.be.true;