8
0
Quellcode durchsuchen

Tests: Forced resizer redraw is no longer required, so the code is removed.

Probably thanks to adjusting the size of the editor / editable.
Marek Lewandowski vor 5 Jahren
Ursprung
Commit
712ff770f2
1 geänderte Dateien mit 5 neuen und 24 gelöschten Zeilen
  1. 5 24
      packages/ckeditor5-widget/tests/widgetresize.js

+ 5 - 24
packages/ckeditor5-widget/tests/widgetresize.js

@@ -101,25 +101,19 @@ describe( 'WidgetResize', () => {
 	} );
 
 	describe( 'mouse listeners', () => {
-		let resizer;
-
 		beforeEach( () => {
-			resizer = createResizer();
+			createResizer();
 		} );
 
 		it( 'doesnt break when called with unexpected element', async () => {
 			const unrelatedElement = document.createElement( 'div' );
 
-			resizer.redraw(); // @todo this shouldn't be necessary.
-
 			editor.plugins.get( WidgetResize )._mouseDownListener( {}, {
 				target: unrelatedElement
 			} );
 		} );
 
 		it( 'passes new width to the options.onCommit()', async () => {
-			resizer.redraw(); // @todo this shouldn't be necessary.
-
 			const usedResizer = 'top-right';
 			const domParts = getWidgetDomParts( widget, usedResizer );
 			const initialPointerPosition = getElementCenterPoint( domParts.widget, usedResizer );
@@ -152,10 +146,9 @@ describe( 'WidgetResize', () => {
 	} );
 
 	it( 'nothing bad happens if activeResizer got unset', async () => {
-		const resizer = createResizer( {
+		createResizer( {
 			isCentered: () => true
 		} );
-		resizer.redraw(); // @todo this shouldn't be necessary.
 
 		const usedResizer = 'top-right';
 		const domParts = getWidgetDomParts( widget, usedResizer );
@@ -172,17 +165,13 @@ describe( 'WidgetResize', () => {
 	} );
 
 	describe( '_proposeNewSize()', () => {
-		let resizer;
-
 		beforeEach( async () => {
-			resizer = createResizer( {
+			createResizer( {
 				isCentered: () => true
 			} );
 		} );
 
 		it( 'assumes a centered image if no isCentered option is provided', async () => {
-			resizer.redraw(); // @todo this shouldn't be necessary.
-
 			const usedResizer = 'top-right';
 			const domParts = getWidgetDomParts( widget, usedResizer, editor.editing.view );
 			const initialPointerPosition = getElementCenterPoint( domParts.widget, usedResizer );
@@ -204,15 +193,11 @@ describe( 'WidgetResize', () => {
 		} );
 	} );
 	describe( 'Integration (pixels)', () => {
-		let resizer;
-
 		beforeEach( async () => {
-			resizer = createResizer();
+			createResizer();
 		} );
 
 		it( 'properly sets the state for subsequent resizes', async () => {
-			resizer.redraw(); // @todo this shouldn't be necessary.
-
 			const usedResizer = 'top-right';
 			const domParts = getWidgetDomParts( widget, usedResizer, editor.editing.view );
 			const initialPointerPosition = getElementCenterPoint( domParts.widget, usedResizer );
@@ -244,17 +229,13 @@ describe( 'WidgetResize', () => {
 	} );
 
 	describe( 'Integration (percents)', () => {
-		let resizer;
-
 		beforeEach( async () => {
-			resizer = createResizer( {
+			createResizer( {
 				unit: undefined
 			} );
 		} );
 
 		it( 'properly sets the state for subsequent resizes', async () => {
-			resizer.redraw(); // @todo this shouldn't be necessary.
-
 			const usedResizer = 'top-right';
 			const domParts = getWidgetDomParts( widget, usedResizer, editor.editing.view );
 			const initialPointerPosition = getElementCenterPoint( domParts.widget, usedResizer );