瀏覽代碼

Tests: Ignore rendering performance tests on Edge and increase timeout to make tests more stable.

Krzysztof Krztoń 6 年之前
父節點
當前提交
1ba28446d8
共有 1 個文件被更改,包括 19 次插入12 次删除
  1. 19 12
      packages/ckeditor5-engine/tests/view/renderer.js

+ 19 - 12
packages/ckeditor5-engine/tests/view/renderer.js

@@ -3205,34 +3205,41 @@ describe( 'Renderer', () => {
 		// However, for larger data sets the difference between using `diff()` and `fastDiff()` (see above issue for context)
 		// is more than 10x in execution time so it is clearly visible in these tests when something goes wrong.
 		describe( 'rendering performance', () => {
-			it( 'should not take more than 250ms to render around 300 element nodes (same html)', () => {
+			before( function() {
+				// Ignore on Edge browser where performance is quite poor.
+				if ( env.isEdge ) {
+					this.skip();
+				}
+			} );
+
+			it( 'should not take more than 350ms to render around 300 element nodes (same html)', () => {
 				const renderingTime = measureRenderingTime( viewRoot, generateViewData1( 65 ), generateViewData1( 55 ) );
-				expect( renderingTime ).to.be.within( 0, 250 );
+				expect( renderingTime ).to.be.within( 0, 350 );
 			} );
 
-			it( 'should not take more than 250ms to render around 300 element nodes (different html)', () => {
+			it( 'should not take more than 350ms to render around 300 element nodes (different html)', () => {
 				const renderingTime = measureRenderingTime( viewRoot, generateViewData1( 55 ), generateViewData2( 65 ) );
-				expect( renderingTime ).to.be.within( 0, 250 );
+				expect( renderingTime ).to.be.within( 0, 350 );
 			} );
 
-			it( 'should not take more than 250ms to render around 500 element nodes (same html)', () => {
+			it( 'should not take more than 350ms to render around 500 element nodes (same html)', () => {
 				const renderingTime = measureRenderingTime( viewRoot, generateViewData1( 105 ), generateViewData1( 95 ) );
-				expect( renderingTime ).to.be.within( 0, 250 );
+				expect( renderingTime ).to.be.within( 0, 350 );
 			} );
 
-			it( 'should not take more than 250ms to render around 500 element nodes (different html)', () => {
+			it( 'should not take more than 350ms to render around 500 element nodes (different html)', () => {
 				const renderingTime = measureRenderingTime( viewRoot, generateViewData1( 95 ), generateViewData2( 105 ) );
-				expect( renderingTime ).to.be.within( 0, 250 );
+				expect( renderingTime ).to.be.within( 0, 350 );
 			} );
 
-			it( 'should not take more than 250ms to render around 1000 element nodes (same html)', () => {
+			it( 'should not take more than 350ms to render around 1000 element nodes (same html)', () => {
 				const renderingTime = measureRenderingTime( viewRoot, generateViewData1( 195 ), generateViewData1( 205 ) );
-				expect( renderingTime ).to.be.within( 0, 250 );
+				expect( renderingTime ).to.be.within( 0, 350 );
 			} );
 
-			it( 'should not take more than 250ms to render around 1000 element nodes (different html)', () => {
+			it( 'should not take more than 350ms to render around 1000 element nodes (different html)', () => {
 				const renderingTime = measureRenderingTime( viewRoot, generateViewData1( 205 ), generateViewData2( 195 ) );
-				expect( renderingTime ).to.be.within( 0, 250 );
+				expect( renderingTime ).to.be.within( 0, 350 );
 			} );
 
 			function measureRenderingTime( viewRoot, initialData, newData ) {