]> source.dussan.org Git - jquery.git/commitdiff
Tests: Skip module tests in Edge
authorMichał Gołębiowski-Owczarek <m.goleb@gmail.com>
Mon, 30 Jul 2018 16:41:08 +0000 (18:41 +0200)
committerGitHub <noreply@github.com>
Mon, 30 Jul 2018 16:41:08 +0000 (18:41 +0200)
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

index 28a62fe0b69825c4cbad9d6e136eb8dff594a221..ddb04c7d62249de2fb6d825ed3ba9a21b5b0ae93 100644 (file)
@@ -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" );