From ae82e85e64b0a06d585934d8f2d2859ce3e755b6 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Micha=C5=82=20Go=C5=82=C4=99biowski-Owczarek?= Date: Mon, 30 Jul 2018 18:41:08 +0200 Subject: [PATCH] Tests: Skip module tests in Edge Edge sometimes doesn't execute module scripts. It needs to be investigated why but for now, we're skipping the test to make our tests more stable. Closes gh-4140 --- test/unit/manipulation.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/test/unit/manipulation.js b/test/unit/manipulation.js index 28a62fe0b..ddb04c7d6 100644 --- a/test/unit/manipulation.js +++ b/test/unit/manipulation.js @@ -1797,7 +1797,13 @@ QUnit.test( "html(Function)", function( assert ) { testHtml( manipulationFunctionReturningObj, assert ); } ); -QUnit[ QUnit.moduleTypeSupported ? "test" : "skip" ]( "html(script type module)", function( assert ) { +QUnit[ + // Support: Edge 16-17 + // Edge sometimes doesn't execute module scripts so skip the test there. + ( QUnit.moduleTypeSupported && !/edge\//i.test( navigator.userAgent ) ) ? + "test" : + "skip" +]( "html(script type module)", function( assert ) { assert.expect( 4 ); var done = assert.async(), $fixture = jQuery( "#qunit-fixture" ); -- 2.39.5