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

Disabled unstable test on Gecko.

Kuba Niegowski 5 лет назад
Родитель
Сommit
e6e4a2bd77
1 измененных файлов с 61 добавлено и 55 удалено
  1. 61 55
      packages/ckeditor5-table/tests/tablenavigation.js

+ 61 - 55
packages/ckeditor5-table/tests/tablenavigation.js

@@ -17,6 +17,7 @@ import Image from '@ckeditor/ckeditor5-image/src/image';
 import ImageCaption from '@ckeditor/ckeditor5-image/src/imagecaption';
 import HorizontalLine from '@ckeditor/ckeditor5-horizontal-line/src/horizontalline';
 import global from '@ckeditor/ckeditor5-utils/src/dom/global';
+import env from '@ckeditor/ckeditor5-utils/src/env';
 
 describe( 'TableNavigation', () => {
 	let editor, model, modelRoot, tableSelection;
@@ -1835,61 +1836,66 @@ describe( 'TableNavigation', () => {
 					} );
 				} );
 
-				describe( 'with selection in the wrap area', () => {
-					const text = new Array( 10 ).fill( 0 ).map( () => 'word' ).join( ' ' );
-
-					it( 'should move caret to end if caret is after the last space in the line next to the last one', () => {
-						// This is also first position in the last line.
-						setModelData( model, modelTable( [
-							[ '00', '01', '02' ],
-							[ '10', text + ' []word word word', '12' ],
-							[ '20', '21', '22' ]
-						] ) );
-
-						editor.editing.view.document.fire( 'keydown', downArrowDomEvtDataStub );
-
-						sinon.assert.calledOnce( downArrowDomEvtDataStub.preventDefault );
-						sinon.assert.calledOnce( downArrowDomEvtDataStub.stopPropagation );
-
-						assertEqualMarkup( getModelData( model ), modelTable( [
-							[ '00', '01', '02' ],
-							[ '10', text + ' word word word[]', '12' ],
-							[ '20', '21', '22' ]
-						] ) );
-					} );
-
-					it( 'should move caret to end if caret is at the last space in the line next to last one', () => {
-						setModelData( model, modelTable( [
-							[ '00', '01', '02' ],
-							[ '10', text + '[] word word word', '12' ],
-							[ '20', '21', '22' ]
-						] ) );
-
-						editor.editing.view.document.fire( 'keydown', downArrowDomEvtDataStub );
-
-						sinon.assert.calledOnce( downArrowDomEvtDataStub.preventDefault );
-						sinon.assert.calledOnce( downArrowDomEvtDataStub.stopPropagation );
-
-						assertEqualMarkup( getModelData( model ), modelTable( [
-							[ '00', '01', '02' ],
-							[ '10', text + ' word word word[]', '12' ],
-							[ '20', '21', '22' ]
-						] ) );
-					} );
-
-					it( 'should not move caret if it\'s just before the last space in the line next to last one', () => {
-						setModelData( model, modelTable( [
-							[ '00', '01', '02' ],
-							[ '10', text.substring( 0, text.length - 1 ) + '[]d word word word', '12' ],
-							[ '20', '21', '22' ]
-						] ) );
-
-						editor.editing.view.document.fire( 'keydown', downArrowDomEvtDataStub );
-
-						sinon.assert.notCalled( downArrowDomEvtDataStub.preventDefault );
-						sinon.assert.notCalled( downArrowDomEvtDataStub.stopPropagation );
-					} );
-				} );
+				if ( !env.isGecko ) {
+					// This tests fails on Travis. They work correctly when started on local machine.
+					// Issue is probably related to text rendering and wrapping.
+
+					describe( 'with selection in the wrap area', () => {
+						const text = new Array( 10 ).fill( 0 ).map( () => 'word' ).join( ' ' );
+
+						it( 'should move caret to end if caret is after the last space in the line next to the last one', () => {
+							// This is also first position in the last line.
+							setModelData( model, modelTable( [
+								[ '00', '01', '02' ],
+								[ '10', text + ' []word word word', '12' ],
+								[ '20', '21', '22' ]
+							] ) );
+
+							editor.editing.view.document.fire( 'keydown', downArrowDomEvtDataStub );
+
+							sinon.assert.calledOnce( downArrowDomEvtDataStub.preventDefault );
+							sinon.assert.calledOnce( downArrowDomEvtDataStub.stopPropagation );
+
+							assertEqualMarkup( getModelData( model ), modelTable( [
+								[ '00', '01', '02' ],
+								[ '10', text + ' word word word[]', '12' ],
+								[ '20', '21', '22' ]
+							] ) );
+						} );
+
+						it( 'should move caret to end if caret is at the last space in the line next to last one', () => {
+							setModelData( model, modelTable( [
+								[ '00', '01', '02' ],
+								[ '10', text + '[] word word word', '12' ],
+								[ '20', '21', '22' ]
+							] ) );
+
+							editor.editing.view.document.fire( 'keydown', downArrowDomEvtDataStub );
+
+							sinon.assert.calledOnce( downArrowDomEvtDataStub.preventDefault );
+							sinon.assert.calledOnce( downArrowDomEvtDataStub.stopPropagation );
+
+							assertEqualMarkup( getModelData( model ), modelTable( [
+								[ '00', '01', '02' ],
+								[ '10', text + ' word word word[]', '12' ],
+								[ '20', '21', '22' ]
+							] ) );
+						} );
+
+						it( 'should not move caret if it\'s just before the last space in the line next to last one', () => {
+							setModelData( model, modelTable( [
+								[ '00', '01', '02' ],
+								[ '10', text.substring( 0, text.length - 1 ) + '[]d word word word', '12' ],
+								[ '20', '21', '22' ]
+							] ) );
+
+							editor.editing.view.document.fire( 'keydown', downArrowDomEvtDataStub );
+
+							sinon.assert.notCalled( downArrowDomEvtDataStub.preventDefault );
+							sinon.assert.notCalled( downArrowDomEvtDataStub.stopPropagation );
+						} );
+					} );
+				}
 
 				describe( 'with multiple paragraphs of text', () => {
 					const text = new Array( 100 ).fill( 0 ).map( () => 'word' ).join( ' ' );