Browse Source

Improved tests of transfering markers.

Oskar Wróbel 8 years ago
parent
commit
a62a05a88a
1 changed files with 5 additions and 9 deletions
  1. 5 9
      packages/ckeditor5-engine/tests/model/writer.js

+ 5 - 9
packages/ckeditor5-engine/tests/model/writer.js

@@ -64,17 +64,14 @@ describe( 'writer', () => {
 					new Text( 'foo bar' ),
 				] )
 			] );
-			const range = new Range( new Position( docFrag, [ 1, 2 ] ), new Position( docFrag, [ 1, 4 ] ) );
 
-			docFrag.markers.set( 'foo', range );
+			docFrag.markers.set( 'foo', new Range( new Position( docFrag, [ 0, 2 ] ), new Position( docFrag, [ 0, 6 ] ) ) );
 
 			writer.insert( new Position( root, [ 10, 0 ] ), docFrag );
 
-			const marker = doc.markers.get( 'foo' );
-			const expectedRange = new Range( new Position( root, [ 10, 1, 2 ] ), new Position( root, [ 10, 1, 4 ] ) );
-
-			expectData( 'foo<$text bold="true">bar</$text><image src="img.jpg"></image>xyz<div><paragraph>foo bar</paragraph></div>' );
-			expect( marker.getRange().isEqual( expectedRange ) ).to.true;
+			expect( stringify( root, doc.markers.get( 'foo' ).getRange() ) ).to.equal(
+				'foo<$text bold="true">bar</$text><image src="img.jpg"></image>xyz<div><paragraph>fo[o ba]r</paragraph></div>'
+			);
 		} );
 
 		it( 'should transfer markers from given node to the root element of target position when root is a documentFragment', () => {
@@ -85,9 +82,8 @@ describe( 'writer', () => {
 					new Text( 'foo bar' ),
 				] )
 			] );
-			const range = new Range( new Position( docFrag, [ 0, 2 ] ), new Position( docFrag, [ 0, 6 ] ) );
 
-			docFrag.markers.set( 'foo', range );
+			docFrag.markers.set( 'foo', new Range( new Position( docFrag, [ 0, 2 ] ), new Position( docFrag, [ 0, 6 ] ) ) );
 
 			writer.insert( new Position( targetDocFrag, [ 0, 0 ] ), docFrag );