Przeglądaj źródła

Merge pull request #222 from ckeditor/t/221

Tests: Update tests to the recent changes in widgets API. Closes #221.
Oskar Wróbel 7 lat temu
rodzic
commit
050a543d36

+ 7 - 0
packages/ckeditor5-image/tests/image.js

@@ -12,11 +12,18 @@ import { setData as setModelData } from '@ckeditor/ckeditor5-engine/src/dev-util
 import { getData as getViewData } from '@ckeditor/ckeditor5-engine/src/dev-utils/view';
 import ModelRange from '@ckeditor/ckeditor5-engine/src/model/range';
 import global from '@ckeditor/ckeditor5-utils/src/dom/global';
+import env from '@ckeditor/ckeditor5-utils/src/env';
+import testUtils from '@ckeditor/ckeditor5-core/tests/_utils/utils';
 
 describe( 'Image', () => {
 	let editorElement, model, view, editor, document, viewDocument;
 
+	testUtils.createSinonSandbox();
+
 	beforeEach( () => {
+		// Most tests assume non-edge environment but we do not set `contenteditable=false` on Edge so stub `env.isEdge`.
+		testUtils.sinon.stub( env, 'isEdge' ).get( () => false );
+
 		editorElement = global.document.createElement( 'div' );
 		global.document.body.appendChild( editorElement );
 

+ 7 - 0
packages/ckeditor5-image/tests/image/converters.js

@@ -17,11 +17,18 @@ import { upcastElementToElement } from '@ckeditor/ckeditor5-engine/src/conversio
 import ModelRange from '@ckeditor/ckeditor5-engine/src/model/range';
 import { getData as getViewData } from '@ckeditor/ckeditor5-engine/src/dev-utils/view';
 import { setData as setModelData, getData as getModelData } from '@ckeditor/ckeditor5-engine/src/dev-utils/model';
+import env from '@ckeditor/ckeditor5-utils/src/env';
+import testUtils from '@ckeditor/ckeditor5-core/tests/_utils/utils';
 
 describe( 'Image converters', () => {
 	let editor, model, document, viewDocument;
 
+	testUtils.createSinonSandbox();
+
 	beforeEach( () => {
+		// Most tests assume non-edge environment but we do not set `contenteditable=false` on Edge so stub `env.isEdge`.
+		testUtils.sinon.stub( env, 'isEdge' ).get( () => false );
+
 		return VirtualTestEditor.create()
 			.then( newEditor => {
 				editor = newEditor;

+ 7 - 0
packages/ckeditor5-image/tests/image/imageediting.js

@@ -13,11 +13,18 @@ import { getData as getModelData, setData as setModelData } from '@ckeditor/cked
 import { getData as getViewData } from '@ckeditor/ckeditor5-engine/src/dev-utils/view';
 import { isImageWidget } from '../../src/image/utils';
 import normalizeHtml from '@ckeditor/ckeditor5-utils/tests/_utils/normalizehtml';
+import env from '@ckeditor/ckeditor5-utils/src/env';
+import testUtils from '@ckeditor/ckeditor5-core/tests/_utils/utils';
 
 describe( 'ImageEditing', () => {
 	let editor, model, doc, view, viewDocument;
 
+	testUtils.createSinonSandbox();
+
 	beforeEach( () => {
+		// Most tests assume non-edge environment but we do not set `contenteditable=false` on Edge so stub `env.isEdge`.
+		testUtils.sinon.stub( env, 'isEdge' ).get( () => false );
+
 		return VirtualTestEditor
 			.create( {
 				plugins: [ ImageEditing ]

+ 8 - 0
packages/ckeditor5-image/tests/imagecaption/imagecaptionediting.js

@@ -18,10 +18,18 @@ import ModelRange from '@ckeditor/ckeditor5-engine/src/model/range';
 import { getData as getModelData, setData as setModelData } from '@ckeditor/ckeditor5-engine/src/dev-utils/model';
 import { getData as getViewData } from '@ckeditor/ckeditor5-engine/src/dev-utils/view';
 
+import env from '@ckeditor/ckeditor5-utils/src/env';
+import testUtils from '@ckeditor/ckeditor5-core/tests/_utils/utils';
+
 describe( 'ImageCaptionEditing', () => {
 	let editor, model, doc, view;
 
+	testUtils.createSinonSandbox();
+
 	beforeEach( () => {
+		// Most tests assume non-edge environment but we do not set `contenteditable=false` on Edge so stub `env.isEdge`.
+		testUtils.sinon.stub( env, 'isEdge' ).get( () => false );
+
 		return VirtualTestEditor
 			.create( {
 				plugins: [ ImageCaptionEditing, ImageEditing, UndoEditing, Paragraph ]

+ 8 - 2
packages/ckeditor5-image/tests/imagestyle/imagestyleediting.js

@@ -12,12 +12,18 @@ import { getData as getModelData, setData as setModelData } from '@ckeditor/cked
 import { getData as getViewData } from '@ckeditor/ckeditor5-engine/src/dev-utils/view';
 
 import testUtils from '@ckeditor/ckeditor5-core/tests/_utils/utils';
-
-testUtils.createSinonSandbox();
+import env from '@ckeditor/ckeditor5-utils/src/env';
 
 describe( 'ImageStyleEditing', () => {
 	let editor, model, document, viewDocument;
 
+	testUtils.createSinonSandbox( 'ImageStyleEditing' );
+
+	beforeEach( () => {
+		// Most tests assume non-edge environment but we do not set `contenteditable=false` on Edge so stub `env.isEdge`.
+		testUtils.sinon.stub( env, 'isEdge' ).get( () => false );
+	} );
+
 	afterEach( () => {
 		editor.destroy();
 	} );

+ 2 - 2
packages/ckeditor5-image/tests/imagestyle/utils.js

@@ -13,11 +13,11 @@ import rightIcon from '@ckeditor/ckeditor5-core/theme/icons/object-right.svg';
 import testUtils from '@ckeditor/ckeditor5-core/tests/_utils/utils';
 import { normalizeImageStyles } from '../../src/imagestyle/utils';
 
-testUtils.createSinonSandbox();
-
 describe( 'ImageStyle utils', () => {
 	let imageStyles;
 
+	testUtils.createSinonSandbox();
+
 	describe( 'normalizeImageStyles()', () => {
 		// Since this function is all about normalizing the config object, make sure it doesn't throw
 		// if the config is empty (which may happen e.g. if only ImageStyleUI was loaded).

+ 5 - 2
packages/ckeditor5-image/tests/imagetextalternative/ui/textalternativeformview.js

@@ -13,13 +13,16 @@ import FocusTracker from '@ckeditor/ckeditor5-utils/src/focustracker';
 import FocusCycler from '@ckeditor/ckeditor5-ui/src/focuscycler';
 import ViewCollection from '@ckeditor/ckeditor5-ui/src/viewcollection';
 import testUtils from '@ckeditor/ckeditor5-core/tests/_utils/utils';
-
-testUtils.createSinonSandbox();
+import env from '@ckeditor/ckeditor5-utils/src/env';
 
 describe( 'TextAlternativeFormView', () => {
 	let view;
 
+	testUtils.createSinonSandbox();
+
 	beforeEach( () => {
+		testUtils.sinon.stub( env, 'isEdge' ).get( () => false );
+
 		view = new TextAlternativeFormView( { t: () => {} } );
 	} );
 

+ 7 - 0
packages/ckeditor5-image/tests/imagetoolbar.js

@@ -15,11 +15,18 @@ import Paragraph from '@ckeditor/ckeditor5-paragraph/src/paragraph';
 import Range from '@ckeditor/ckeditor5-engine/src/model/range';
 import View from '@ckeditor/ckeditor5-ui/src/view';
 import { setData } from '@ckeditor/ckeditor5-engine/src/dev-utils/model';
+import env from '@ckeditor/ckeditor5-utils/src/env';
+import testUtils from '@ckeditor/ckeditor5-core/tests/_utils/utils';
 
 describe( 'ImageToolbar', () => {
 	let editor, model, doc, plugin, toolbar, balloon, editorElement;
 
+	testUtils.createSinonSandbox();
+
 	beforeEach( () => {
+		// Most tests assume non-edge environment but we do not set `contenteditable=false` on Edge so stub `env.isEdge`.
+		testUtils.sinon.stub( env, 'isEdge' ).get( () => false );
+
 		editorElement = global.document.createElement( 'div' );
 		global.document.body.appendChild( editorElement );
 

+ 4 - 0
packages/ckeditor5-image/tests/imageupload/imageuploadediting.js

@@ -25,6 +25,7 @@ import Range from '@ckeditor/ckeditor5-engine/src/model/range';
 import Position from '@ckeditor/ckeditor5-engine/src/model/position';
 
 import log from '@ckeditor/ckeditor5-utils/src/log';
+import env from '@ckeditor/ckeditor5-utils/src/env';
 import testUtils from '@ckeditor/ckeditor5-core/tests/_utils/utils';
 import Notification from '@ckeditor/ckeditor5-ui/src/notification/notification';
 
@@ -48,6 +49,9 @@ describe( 'ImageUploadEditing', () => {
 	}
 
 	beforeEach( () => {
+		// Most tests assume non-edge environment but we do not set `contenteditable=false` on Edge so stub `env.isEdge`.
+		testUtils.sinon.stub( env, 'isEdge' ).get( () => false );
+
 		testUtils.sinon.stub( window, 'FileReader' ).callsFake( () => {
 			nativeReaderMock = new NativeFileReaderMock();
 

+ 3 - 0
packages/ckeditor5-image/tests/imageupload/imageuploadprogress.js

@@ -43,6 +43,9 @@ describe( 'ImageUploadProgress', () => {
 	testUtils.createSinonSandbox();
 
 	beforeEach( () => {
+		// Most tests assume non-edge environment but we do not set `contenteditable=false` on Edge so stub `env.isEdge`.
+		testUtils.sinon.stub( env, 'isEdge' ).get( () => false );
+
 		testUtils.sinon.stub( window, 'FileReader' ).callsFake( () => {
 			nativeReaderMock = new NativeFileReaderMock();
 

+ 7 - 0
packages/ckeditor5-image/tests/integration.js

@@ -11,12 +11,19 @@ import global from '@ckeditor/ckeditor5-utils/src/dom/global';
 import Image from '../src/image';
 import ImageToolbar from '../src/imagetoolbar';
 import View from '@ckeditor/ckeditor5-ui/src/view';
+import env from '@ckeditor/ckeditor5-utils/src/env';
+import testUtils from '@ckeditor/ckeditor5-core/tests/_utils/utils';
 
 describe( 'ImageToolbar integration', () => {
 	describe( 'with the BalloonToolbar', () => {
 		let balloon, balloonToolbar, newEditor, editorElement;
 
+		testUtils.createSinonSandbox();
+
 		beforeEach( () => {
+			// Most tests assume non-edge environment but we do not set `contenteditable=false` on Edge so stub `env.isEdge`.
+			testUtils.sinon.stub( env, 'isEdge' ).get( () => false );
+
 			editorElement = global.document.createElement( 'div' );
 			global.document.body.appendChild( editorElement );