Skip to content
Permalink
Browse files

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
  • Loading branch information...
mgol committed Jul 30, 2018
1 parent 063c1f2 commit ae82e85e64b0a06d585934d8f2d2859ce3e755b6
Showing with 7 additions and 1 deletion.
  1. +7 −1 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" );

0 comments on commit ae82e85

Please sign in to comment.
You can’t perform that action at this time.