8
0
Просмотр исходного кода

Merge branch t/ckeditor5-engine/1209

Internal: Update API usage after merge t/1209 on engine.
Piotr Jasiun 8 лет назад
Родитель
Сommit
3173da494a

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

@@ -96,9 +96,9 @@ describe( 'Image', () => {
 				'</figure>'
 			);
 
-			model.change( () => {
+			model.change( writer => {
 				const secondImage = document.getRoot().getChild( 1 );
-				document.selection.setRanges( [ ModelRange.createOn( secondImage ) ] );
+				writer.setSelection( ModelRange.createOn( secondImage ) );
 			} );
 
 			expect( getViewData( viewDocument ) ).to.equal(

+ 6 - 6
packages/ckeditor5-image/tests/imagecaption/imagecaptionengine.js

@@ -363,8 +363,8 @@ describe( 'ImageCaptionEngine', () => {
 		it( 'should add hidden class to figcaption when caption is empty and image is no longer selected', () => {
 			setModelData( model, '<paragraph>foo</paragraph>[<image src=""><caption></caption></image>]' );
 
-			model.change( () => {
-				doc.selection.removeAllRanges();
+			model.change( writer => {
+				writer.setSelection( null );
 			} );
 
 			expect( getViewData( viewDocument ) ).to.equal(
@@ -401,7 +401,7 @@ describe( 'ImageCaptionEngine', () => {
 
 			model.change( writer => {
 				writer.remove( doc.selection.getFirstRange() );
-				doc.selection.setRanges( [ ModelRange.createOn( image ) ] );
+				writer.setSelection( ModelRange.createOn( image ) );
 			} );
 
 			expect( getViewData( viewDocument ) ).to.equal(
@@ -417,8 +417,8 @@ describe( 'ImageCaptionEngine', () => {
 			setModelData( model, '<image src=""><caption>[foo bar]</caption></image><image src=""><caption></caption></image>' );
 			const image = doc.getRoot().getChild( 1 );
 
-			model.change( () => {
-				doc.selection.setRanges( [ ModelRange.createOn( image ) ] );
+			model.change( writer => {
+				writer.setSelection( ModelRange.createOn( image ) );
 			} );
 
 			expect( getViewData( viewDocument ) ).to.equal(
@@ -445,7 +445,7 @@ describe( 'ImageCaptionEngine', () => {
 				// Remove text and selection from caption.
 				model.change( writer => {
 					writer.remove( ModelRange.createIn( modelCaption ) );
-					doc.selection.removeAllRanges();
+					writer.setSelection( null );
 				} );
 
 				// Check if there is no figcaption in the view.

+ 6 - 6
packages/ckeditor5-image/tests/imagetoolbar.js

@@ -127,11 +127,11 @@ describe( 'ImageToolbar', () => {
 			editingView.fire( 'render' );
 			expect( balloon.visibleView ).to.be.null;
 
-			model.change( () => {
+			model.change( writer => {
 				// Select the [<image></image>]
-				doc.selection.setRanges( [
+				writer.setSelection(
 					Range.createOn( doc.getRoot().getChild( 1 ) )
-				] );
+				);
 			} );
 
 			expect( balloon.visibleView ).to.equal( toolbar );
@@ -161,11 +161,11 @@ describe( 'ImageToolbar', () => {
 
 			expect( balloon.visibleView ).to.equal( toolbar );
 
-			model.change( () => {
+			model.change( writer => {
 				// Select the <paragraph>[...]</paragraph>
-				doc.selection.setRanges( [
+				writer.setSelection(
 					Range.createIn( doc.getRoot().getChild( 0 ) )
-				] );
+				);
 			} );
 
 			expect( balloon.visibleView ).to.be.null;