Преглед изворни кода

Make the code autoformatting tests more atomic, and their descriptions more informative,

as suggested at https://github.com/ckeditor/ckeditor5/pull/7346/files#r434487890.
Tomek Wytrębowicz пре 5 година
родитељ
комит
8314741fc0
1 измењених фајлова са 90 додато и 84 уклоњено
  1. 90 84
      packages/ckeditor5-autoformat/tests/autoformat.js

+ 90 - 84
packages/ckeditor5-autoformat/tests/autoformat.js

@@ -423,130 +423,136 @@ describe( 'Autoformat', () => {
 		} );
 
 		describe( 'with code element', () => {
-			it( 'should not format inside', () => {
-				// Test *.
-				setData( model, '<paragraph><$text code="true">fo*obar[]</$text></paragraph>' );
+			describe( 'should not format', () => {
+				it( '* inside', () => {
+					setData( model, '<paragraph><$text code="true">fo*obar[]</$text></paragraph>' );
 
-				model.change( writer => {
-					writer.insertText( '*', { code: true }, doc.selection.getFirstPosition() );
+					model.change( writer => {
+						writer.insertText( '*', { code: true }, doc.selection.getFirstPosition() );
+					} );
+
+					expect( getData( model ) ).to
+						.equal( '<paragraph><$text code="true">fo*obar*[]</$text></paragraph>' );
 				} );
 
-				expect( getData( model ) ).to
-					.equal( '<paragraph><$text code="true">fo*obar*[]</$text></paragraph>' );
+				it( '__ inside', () => {
+					setData( model, '<paragraph><$text code="true">fo__obar_[]</$text></paragraph>' );
 
-				// Test __.
-				setData( model, '<paragraph><$text code="true">fo__obar_[]</$text></paragraph>' );
+					model.change( writer => {
+						writer.insertText( '_', { code: true }, doc.selection.getFirstPosition() );
+					} );
 
-				model.change( writer => {
-					writer.insertText( '_', { code: true }, doc.selection.getFirstPosition() );
+					expect( getData( model ) ).to
+						.equal( '<paragraph><$text code="true">fo__obar__[]</$text></paragraph>' );
 				} );
 
-				expect( getData( model ) ).to
-					.equal( '<paragraph><$text code="true">fo__obar__[]</$text></paragraph>' );
+				it( '~~ inside', () => {
+					setData( model, '<paragraph><$text code="true">fo~~obar~[]</$text></paragraph>' );
 
-				// Test ~~.
-				setData( model, '<paragraph><$text code="true">fo~~obar~[]</$text></paragraph>' );
+					model.change( writer => {
+						writer.insertText( '~', { code: true }, doc.selection.getFirstPosition() );
+					} );
 
-				model.change( writer => {
-					writer.insertText( '~', { code: true }, doc.selection.getFirstPosition() );
+					expect( getData( model ) ).to
+						.equal( '<paragraph><$text code="true">fo~~obar~~[]</$text></paragraph>' );
 				} );
 
-				expect( getData( model ) ).to
-					.equal( '<paragraph><$text code="true">fo~~obar~~[]</$text></paragraph>' );
+				it( '` inside', () => {
+					setData( model, '<paragraph><$text code="true">fo`obar[]</$text></paragraph>' );
 
-				// Test `.
-				setData( model, '<paragraph><$text code="true">fo`obar[]</$text></paragraph>' );
+					model.change( writer => {
+						writer.insertText( '`', { code: true }, doc.selection.getFirstPosition() );
+					} );
 
-				model.change( writer => {
-					writer.insertText( '`', { code: true }, doc.selection.getFirstPosition() );
+					expect( getData( model ) ).to
+						.equal( '<paragraph><$text code="true">fo`obar`[]</$text></paragraph>' );
 				} );
-
-				expect( getData( model ) ).to
-					.equal( '<paragraph><$text code="true">fo`obar`[]</$text></paragraph>' );
 			} );
 
-			it( 'should not format across', () => {
-				// Test *.
-				setData( model, '<paragraph><$text code="true">fo*o</$text>bar[]</paragraph>' );
+			describe( 'should not format', () => {
+				it( '* across', () => {
+					setData( model, '<paragraph><$text code="true">fo*o</$text>bar[]</paragraph>' );
 
-				model.change( writer => {
-					writer.insertText( '*', doc.selection.getFirstPosition() );
-				} );
+					model.change( writer => {
+						writer.insertText( '*', doc.selection.getFirstPosition() );
+					} );
 
-				expect( getData( model ) ).to
-					.equal( '<paragraph><$text code="true">fo*o</$text>bar*[]</paragraph>' );
+					expect( getData( model ) ).to
+						.equal( '<paragraph><$text code="true">fo*o</$text>bar*[]</paragraph>' );
+				} );
+				it( '__ across', () => {
+					setData( model, '<paragraph><$text code="true">fo__o</$text>bar_[]</paragraph>' );
 
-				// Test __.
-				setData( model, '<paragraph><$text code="true">fo__o</$text>bar_[]</paragraph>' );
+					model.change( writer => {
+						writer.insertText( '_', doc.selection.getFirstPosition() );
+					} );
 
-				model.change( writer => {
-					writer.insertText( '_', doc.selection.getFirstPosition() );
+					expect( getData( model ) ).to
+						.equal( '<paragraph><$text code="true">fo__o</$text>bar__[]</paragraph>' );
 				} );
+				it( '~~ across', () => {
+					setData( model, '<paragraph><$text code="true">fo~~o</$text>bar~[]</paragraph>' );
 
-				expect( getData( model ) ).to
-					.equal( '<paragraph><$text code="true">fo__o</$text>bar__[]</paragraph>' );
-
-				// Test ~~.
-				setData( model, '<paragraph><$text code="true">fo~~o</$text>bar~[]</paragraph>' );
+					model.change( writer => {
+						writer.insertText( '~', doc.selection.getFirstPosition() );
+					} );
 
-				model.change( writer => {
-					writer.insertText( '~', doc.selection.getFirstPosition() );
+					expect( getData( model ) ).to
+						.equal( '<paragraph><$text code="true">fo~~o</$text>bar~~[]</paragraph>' );
 				} );
+				it( '` across', () => {
+					setData( model, '<paragraph><$text code="true">fo`o</$text>bar[]</paragraph>' );
 
-				expect( getData( model ) ).to
-					.equal( '<paragraph><$text code="true">fo~~o</$text>bar~~[]</paragraph>' );
-
-				// Test `.
-				setData( model, '<paragraph><$text code="true">fo`o</$text>bar[]</paragraph>' );
+					model.change( writer => {
+						writer.insertText( '`', doc.selection.getFirstPosition() );
+					} );
 
-				model.change( writer => {
-					writer.insertText( '`', doc.selection.getFirstPosition() );
+					expect( getData( model ) ).to
+						.equal( '<paragraph><$text code="true">fo`o</$text>bar`[]</paragraph>' );
 				} );
-
-				expect( getData( model ) ).to
-					.equal( '<paragraph><$text code="true">fo`o</$text>bar`[]</paragraph>' );
 			} );
 
-			it( 'should format after', () => {
-				// Test *.
-				setData( model, '<paragraph><$text code="true">fo*o</$text>b*ar[]</paragraph>' );
+			describe( 'should format', () => {
+				it( '* after', () => {
+					setData( model, '<paragraph><$text code="true">fo*o</$text>b*ar[]</paragraph>' );
 
-				model.change( writer => {
-					writer.insertText( '*', doc.selection.getFirstPosition() );
-				} );
+					model.change( writer => {
+						writer.insertText( '*', doc.selection.getFirstPosition() );
+					} );
 
-				expect( getData( model ) ).to
-					.equal( '<paragraph><$text code="true">fo*o</$text>b<$text italic="true">ar</$text>[]</paragraph>' );
+					expect( getData( model ) ).to
+						.equal( '<paragraph><$text code="true">fo*o</$text>b<$text italic="true">ar</$text>[]</paragraph>' );
+				} );
+				it( '__ after', () => {
+					setData( model, '<paragraph><$text code="true">fo__o</$text>b__ar_[]</paragraph>' );
 
-				// Test __.
-				setData( model, '<paragraph><$text code="true">fo__o</$text>b__ar_[]</paragraph>' );
+					model.change( writer => {
+						writer.insertText( '_', doc.selection.getFirstPosition() );
+					} );
 
-				model.change( writer => {
-					writer.insertText( '_', doc.selection.getFirstPosition() );
+					expect( getData( model ) ).to
+						.equal( '<paragraph><$text code="true">fo__o</$text>b<$text bold="true">ar</$text>[]</paragraph>' );
 				} );
+				it( '~~ after', () => {
+					setData( model, '<paragraph><$text code="true">fo~~o</$text>b~~ar~[]</paragraph>' );
 
-				expect( getData( model ) ).to
-					.equal( '<paragraph><$text code="true">fo__o</$text>b<$text bold="true">ar</$text>[]</paragraph>' );
-
-				// Test ~~.
-				setData( model, '<paragraph><$text code="true">fo~~o</$text>b~~ar~[]</paragraph>' );
+					model.change( writer => {
+						writer.insertText( '~', doc.selection.getFirstPosition() );
+					} );
 
-				model.change( writer => {
-					writer.insertText( '~', doc.selection.getFirstPosition() );
+					expect( getData( model ) ).to
+						.equal( '<paragraph><$text code="true">fo~~o</$text>b<$text strikethrough="true">ar</$text>[]</paragraph>' );
 				} );
+				it( '` after', () => {
+					setData( model, '<paragraph><$text code="true">fo`o</$text>b`ar[]</paragraph>' );
 
-				expect( getData( model ) ).to
-					.equal( '<paragraph><$text code="true">fo~~o</$text>b<$text strikethrough="true">ar</$text>[]</paragraph>' );
-
-				// Test `.
-				setData( model, '<paragraph><$text code="true">fo`o</$text>b`ar[]</paragraph>' );
+					model.change( writer => {
+						writer.insertText( '`', doc.selection.getFirstPosition() );
+					} );
 
-				model.change( writer => {
-					writer.insertText( '`', doc.selection.getFirstPosition() );
+					expect( getData( model ) ).to
+						.equal( '<paragraph><$text code="true">fo`o</$text>b<$text code="true">ar</$text>[]</paragraph>' );
 				} );
-
-				expect( getData( model ) ).to
-					.equal( '<paragraph><$text code="true">fo`o</$text>b<$text code="true">ar</$text>[]</paragraph>' );
 			} );
 		} );