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

Tests: Added an explanation in tests.

Szymon Cofalik 8 лет назад
Родитель
Сommit
dc4010be81
1 измененных файлов с 5 добавлено и 0 удалено
  1. 5 0
      packages/ckeditor5-engine/tests/view/range.js

+ 5 - 0
packages/ckeditor5-engine/tests/view/range.js

@@ -172,16 +172,21 @@ describe( 'Range', () => {
 				.to.equal( '<p>foo<b>[<img></img>]</b>bom</p>' );
 		} );
 
+		// Other results may theoretically be correct too. It is not decided whether the trimmed range should
+		// be collapsed in attribute element, at its start or its end. This is one of possible correct results
+		// and we won't know for sure unless we have more cases. See #1058.
 		it( 'case 8', () => {
 			expect( trim( '<p>[<b></b>]</p>' ) )
 				.to.equal( '<p><b></b>[]</p>' );
 		} );
 
+		// As above.
 		it( 'case 9', () => {
 			expect( trim( '<p><b></b>[<b></b>]<b></b></p>' ) )
 				.to.equal( '<p><b></b><b></b>[]<b></b></p>' );
 		} );
 
+		// As above.
 		it( 'case 10', () => {
 			expect( trim( '<p>[<b></b><b></b>]</p>' ) )
 				.to.equal( '<p><b></b><b></b>[]</p>' );