Parcourir la source

Minor code style improvements in tests.

Oskar Wróbel il y a 6 ans
Parent
commit
6f90edc2db
1 fichiers modifiés avec 24 ajouts et 26 suppressions
  1. 24 26
      packages/ckeditor5-list/tests/todolistediting.js

+ 24 - 26
packages/ckeditor5-list/tests/todolistediting.js

@@ -412,6 +412,26 @@ describe( 'TodoListEditing', () => {
 			expect( getViewData( view ) ).to.equal( '<test class="checked">{}Foo</test>' );
 			expect( getViewData( view ) ).to.equal( '<test class="checked">{}Foo</test>' );
 		} );
 		} );
 
 
+		it( 'should move selection after checkmark element to the first text node', () => {
+			setModelData( model, '<listItem listType="todo" listIndent="0">Foo</listItem>' );
+
+			expect( getViewData( view ) ).to.equal(
+				'<ul class="todo-list">' +
+					'<li><label class="todo-list__checkmark" contenteditable="false"></label>{}Foo</li>' +
+				'</ul>'
+			);
+		} );
+
+		it( 'should move selection after checkmark element when list item does not contain any text node', () => {
+			setModelData( model, '<listItem listType="todo" listIndent="0">[]</listItem>' );
+
+			expect( getViewData( view ) ).to.equal(
+				'<ul class="todo-list">' +
+					'<li><label class="todo-list__checkmark" contenteditable="false"></label>[]</li>' +
+				'</ul>'
+			);
+		} );
+
 		it( 'should properly handle typing inside text node with attribute', () => {
 		it( 'should properly handle typing inside text node with attribute', () => {
 			setModelData( model, '<listItem listType="todo" listIndent="0"><$text bold="true">[]foo</$text></listItem>' );
 			setModelData( model, '<listItem listType="todo" listIndent="0"><$text bold="true">[]foo</$text></listItem>' );
 
 
@@ -424,8 +444,8 @@ describe( 'TodoListEditing', () => {
 			expect( getViewData( view ) ).to.equal(
 			expect( getViewData( view ) ).to.equal(
 				'<ul class="todo-list">' +
 				'<ul class="todo-list">' +
 					'<li>' +
 					'<li>' +
-				'<label class="todo-list__checkmark" contenteditable="false"></label>' +
-				'<strong>b{}foo</strong>' +
+						'<label class="todo-list__checkmark" contenteditable="false"></label>' +
+						'<strong>b{}foo</strong>' +
 					'</li>' +
 					'</li>' +
 				'</ul>'
 				'</ul>'
 			);
 			);
@@ -445,8 +465,8 @@ describe( 'TodoListEditing', () => {
 			expect( getViewData( view ) ).to.equal(
 			expect( getViewData( view ) ).to.equal(
 				'<ul class="todo-list">' +
 				'<ul class="todo-list">' +
 					'<li>' +
 					'<li>' +
-				'<label class="todo-list__checkmark" contenteditable="false"></label>' +
-				'<a href="foo"><strong>b{}foo</strong></a>' +
+						'<label class="todo-list__checkmark" contenteditable="false"></label>' +
+						'<a href="foo"><strong>b{}foo</strong></a>' +
 					'</li>' +
 					'</li>' +
 				'</ul>'
 				'</ul>'
 			);
 			);
@@ -775,28 +795,6 @@ describe( 'TodoListEditing', () => {
 		} );
 		} );
 	} );
 	} );
 
 
-	describe( 'selection view post-fixer', () => {
-		it( 'should move selection after checkmark element to the first text node', () => {
-			setModelData( model, '<listItem listType="todo" listIndent="0">Foo</listItem>' );
-
-			expect( getViewData( view ) ).to.equal(
-				'<ul class="todo-list">' +
-					'<li><label class="todo-list__checkmark" contenteditable="false"></label>{}Foo</li>' +
-				'</ul>'
-			);
-		} );
-
-		it( 'should move selection after checkmark element when list item does not contain any text node', () => {
-			setModelData( model, '<listItem listType="todo" listIndent="0">[]</listItem>' );
-
-			expect( getViewData( view ) ).to.equal(
-				'<ul class="todo-list">' +
-					'<li><label class="todo-list__checkmark" contenteditable="false"></label>[]</li>' +
-				'</ul>'
-			);
-		} );
-	} );
-
 	describe( 'uiElements view post-fixer', () => {
 	describe( 'uiElements view post-fixer', () => {
 		it( 'should move all UIElements from before a checkmark after the checkmark element', () => {
 		it( 'should move all UIElements from before a checkmark after the checkmark element', () => {
 			setModelData( model,
 			setModelData( model,