" ).appendTo( "#qunit-fixture" ),
$elem = jQuery( "
" ).addClass( "test__customProperties" )
.appendTo( "#qunit-fixture" ),
- webkit = /\bsafari\b/i.test( navigator.userAgent ) &&
- !/\firefox\b/i.test( navigator.userAgent ) &&
- !/\edge\b/i.test( navigator.userAgent ),
- oldSafari = webkit && ( /\b9\.\d(\.\d+)* safari/i.test( navigator.userAgent ) ||
- /\b10\.0(\.\d+)* safari/i.test( navigator.userAgent ) ||
- /iphone os (?:9|10)_/i.test( navigator.userAgent ) ),
+ webkitOrBlink = /\bsafari\b/i.test( navigator.userAgent ) &&
+ !/\bfirefox\b/i.test( navigator.userAgent ) &&
+ !/\bedge\b/i.test( navigator.userAgent ),
+ oldSafari = /iphone os 10_/i.test( navigator.userAgent ),
expected = 10;
- if ( webkit ) {
+ if ( webkitOrBlink ) {
expected -= 2;
}
if ( oldSafari ) {
@@ -1852,17 +1823,17 @@ QUnit.test( "Do not throw on frame elements from css method (#15098)", function(
assert.equal( $elem.css( "--prop1" ), "val1", "Basic CSS custom property" );
- // Support: Safari 9.1-10.0 only
- // Safari collapses whitespaces & quotes. Ignore it.
+ // Support: iOS <=10 only
+ // Old Safari collapses whitespaces & quotes. Ignore it.
if ( !oldSafari ) {
assert.equal( $elem.css( "--prop2" ), " val2", "Preceding whitespace maintained" );
assert.equal( $elem.css( "--prop3" ), "val3 ", "Following whitespace maintained" );
}
- // Support: Chrome 49-55, Safari 9.1-10.0
+ // Support: Chrome <=49 - 73+, Safari <=9.1 - 12.1+
// Chrome treats single quotes as double ones.
// Safari treats double quotes as single ones.
- if ( !webkit ) {
+ if ( !webkitOrBlink ) {
assert.equal( $elem.css( "--prop4" ), "\"val4\"", "Works with double quotes" );
assert.equal( $elem.css( "--prop5" ), "'val5'", "Works with single quotes" );
}
diff --git a/test/unit/deferred.js b/test/unit/deferred.js
index c704f905f9e..936a57524e2 100644
--- a/test/unit/deferred.js
+++ b/test/unit/deferred.js
@@ -172,7 +172,7 @@ QUnit.test( "jQuery.Deferred.catch", function( assert ) {
var value1, value2, value3,
defer = jQuery.Deferred(),
- piped = defer[ "catch" ]( function( a, b ) {
+ piped = defer.catch( function( a, b ) {
return a * b;
} ),
done = jQuery.map( new Array( 3 ), function() { return assert.async(); } );
@@ -186,18 +186,18 @@ QUnit.test( "jQuery.Deferred.catch", function( assert ) {
value2 = b;
} );
- defer.reject( 2, 3 )[ "catch" ]( function() {
+ defer.reject( 2, 3 ).catch( function() {
assert.strictEqual( value1, 2, "first reject value ok" );
assert.strictEqual( value2, 3, "second reject value ok" );
assert.strictEqual( value3, 6, "result of filter ok" );
done.pop().call();
} );
- jQuery.Deferred().resolve()[ "catch" ]( function() {
+ jQuery.Deferred().resolve().catch( function() {
assert.ok( false, "then should not be called on resolve" );
} ).then( done.pop() );
- jQuery.Deferred().reject()[ "catch" ]( jQuery.noop ).done( function( value ) {
+ jQuery.Deferred().reject().catch( jQuery.noop ).done( function( value ) {
assert.strictEqual( value, undefined, "then fail callback can return undefined/null" );
done.pop().call();
} );
@@ -563,8 +563,7 @@ QUnit[ typeof Symbol === "function" && Symbol.toStringTag ? "test" : "skip" ](
}
);
-// Test fails in IE9 but is skipped there because console is not active
-QUnit[ window.console ? "test" : "skip" ]( "jQuery.Deferred.exceptionHook", function( assert ) {
+QUnit.test( "jQuery.Deferred.exceptionHook", function( assert ) {
assert.expect( 2 );
@@ -573,24 +572,10 @@ QUnit[ window.console ? "test" : "skip" ]( "jQuery.Deferred.exceptionHook", func
oldWarn = window.console.warn;
window.console.warn = function() {
-
- // Support: Chrome <=41 only
- // Some Chrome versions newer than 30 but older than 42 display the "undefined is
- // not a function" error, not mentioning the function name. This has been fixed
- // in Chrome 42. Relax this test there.
- // This affects our Android 5.0 & Yandex.Browser testing.
- var msg = Array.prototype.join.call( arguments, " " ),
- oldChromium = false;
- if ( /chrome/i.test( navigator.userAgent ) ) {
- oldChromium = parseInt(
- navigator.userAgent.match( /chrome\/(\d+)/i )[ 1 ], 10 ) < 42;
- }
- if ( oldChromium ) {
- assert.ok( /(?:barf|undefined)/.test( msg ), "Message (weak assertion): " + msg );
- } else {
- assert.ok( /barf/.test( msg ), "Message: " + msg );
- }
+ var msg = Array.prototype.join.call( arguments, " " );
+ assert.ok( /barf/.test( msg ), "Message: " + msg );
};
+
jQuery.when(
defer.then( function() {
@@ -613,8 +598,7 @@ QUnit[ window.console ? "test" : "skip" ]( "jQuery.Deferred.exceptionHook", func
defer.resolve();
} );
-// Test fails in IE9 but is skipped there because console is not active
-QUnit[ window.console ? "test" : "skip" ]( "jQuery.Deferred.exceptionHook with stack hooks", function( assert ) {
+QUnit.test( "jQuery.Deferred.exceptionHook with stack hooks", function( assert ) {
assert.expect( 2 );
@@ -632,26 +616,11 @@ QUnit[ window.console ? "test" : "skip" ]( "jQuery.Deferred.exceptionHook with s
};
window.console.warn = function() {
-
- // Support: Chrome <=41 only
- // Some Chrome versions newer than 30 but older than 42 display the "undefined is
- // not a function" error, not mentioning the function name. This has been fixed
- // in Chrome 42. Relax this test there.
- // This affects our Android 5.0 & Yandex.Browser testing.
- var msg = Array.prototype.join.call( arguments, " " ),
- oldChromium = false;
- if ( /chrome/i.test( navigator.userAgent ) ) {
- oldChromium = parseInt(
- navigator.userAgent.match( /chrome\/(\d+)/i )[ 1 ], 10 ) < 42;
- }
- if ( oldChromium ) {
- assert.ok( /(?:cough_up_hairball|undefined)/.test( msg ),
- "Function mentioned (weak assertion): " + msg );
- } else {
- assert.ok( /cough_up_hairball/.test( msg ), "Function mentioned: " + msg );
- }
+ var msg = Array.prototype.join.call( arguments, " " );
+ assert.ok( /cough_up_hairball/.test( msg ), "Function mentioned: " + msg );
assert.ok( /NO STACK FOR YOU/.test( msg ), "Stack trace included: " + msg );
};
+
defer.then( function() {
jQuery.cough_up_hairball();
} ).then( null, function( ) {
@@ -825,11 +794,7 @@ QUnit.test( "jQuery.when(nonThenable) - like Promise.resolve", function( assert
assert.expect( 44 );
- var
-
- // Support: Android 4.0 only
- // Strict mode functions invoked without .call/.apply get global-object context
- defaultContext = ( function getDefaultContext() { return this; } ).call(),
+ var defaultContext = ( function getDefaultContext() { return this; } )(),
done = assert.async( 20 );
@@ -884,9 +849,7 @@ QUnit.test( "jQuery.when(thenable) - like Promise.resolve", function( assert ) {
var customToStringThen = {
then: function( onFulfilled ) {
- // Support: Android 4.0 only
- // Strict mode functions invoked without .call/.apply get global-object context
- onFulfilled.call();
+ onFulfilled();
}
};
if ( typeof Symbol === "function" ) {
@@ -947,9 +910,7 @@ QUnit.test( "jQuery.when(thenable) - like Promise.resolve", function( assert ) {
},
numCases = Object.keys( willSucceed ).length + Object.keys( willError ).length,
- // Support: Android 4.0 only
- // Strict mode functions invoked without .call/.apply get global-object context
- defaultContext = ( function getDefaultContext() { return this; } ).call(),
+ defaultContext = ( function getDefaultContext() { return this; } )(),
done = assert.async( numCases * 2 );
@@ -1027,9 +988,7 @@ QUnit.test( "jQuery.when(a, b) - like Promise.all", function( assert ) {
rejectedStandardPromise: true
},
- // Support: Android 4.0 only
- // Strict mode functions invoked without .call/.apply get global-object context
- defaultContext = ( function getDefaultContext() { return this; } ).call(),
+ defaultContext = ( function getDefaultContext() { return this; } )(),
done = assert.async( 98 );
diff --git a/test/unit/deprecated.js b/test/unit/deprecated.js
index d8b5a518148..aeadd405691 100644
--- a/test/unit/deprecated.js
+++ b/test/unit/deprecated.js
@@ -159,9 +159,9 @@ QUnit.test( "jQuery.parseJSON", function( assert ) {
assert.throws( function() {
var result = jQuery.parseJSON( "0101" );
- // Support: IE <=9 only
+ // Support: IE <=9 - 11+
// Ensure base-10 interpretation on browsers that erroneously accept leading-zero numbers
- if ( result === 101 ) {
+ if ( document.documentMode && result === 101 ) {
throw new Error( "close enough" );
}
}, null, "Leading-zero number raises an error or is parsed as decimal" );
diff --git a/test/unit/dimensions.js b/test/unit/dimensions.js
index 23a58523cfe..4a3ad5ecb8e 100644
--- a/test/unit/dimensions.js
+++ b/test/unit/dimensions.js
@@ -687,19 +687,7 @@ QUnit.test( "interaction with scrollbars (gh-3589)", function( assert ) {
.css( { position: "relative" } ),
$boxes = jQuery(
[ plainBox[ 0 ], contentBox[ 0 ], borderBox[ 0 ], relativeBorderBox[ 0 ] ]
- ).appendTo( parent ),
-
- // Support: IE 9 only
- // Computed width seems to report content width even with "box-sizing: border-box", and
- // "overflow: scroll" actually _shrinks_ the element (gh-3699).
- borderBoxLoss =
- borderBox.clone().css( { overflow: "auto" } ).appendTo( parent )[ 0 ].offsetWidth -
- borderBox[ 0 ].offsetWidth;
-
- if ( borderBoxLoss > 0 ) {
- borderBox[ 0 ].style.width = ( size + borderBoxLoss ) + "px";
- borderBox[ 0 ].style.height = ( size + borderBoxLoss ) + "px";
- }
+ ).appendTo( parent );
for ( i = 0; i < 3; i++ ) {
if ( i === 1 ) {
@@ -758,10 +746,9 @@ QUnit.test( "outerWidth/Height for table cells and textarea with border-box in I
$secondTh = jQuery( "
| " ),
$thirdTh = jQuery( " | " ),
- // Support: Firefox 63, Edge 16-17, Android 8, iOS 7-11
- // These browsers completely ignore the border-box and height settings
- // The computed height is instead just line-height + border
- // Either way, what we're doing in css.js is correct
+ // Most browsers completely ignore the border-box and height settings.
+ // The computed height is instead just line-height + border.
+ // Either way, what we're doing in css.js is correct.
$td = jQuery( "text | " ),
$tbody = jQuery( "" ).appendTo( $table ),
@@ -775,18 +762,7 @@ QUnit.test( "outerWidth/Height for table cells and textarea with border-box in I
assert.strictEqual( $firstTh.outerWidth(), 200, "First th has outerWidth 200." );
assert.strictEqual( $secondTh.outerWidth(), 200, "Second th has outerWidth 200." );
assert.strictEqual( $thirdTh.outerWidth(), 200, "Third th has outerWidth 200." );
-
- // Support: Android 4.0-4.3 only
- // Android Browser disregards td's box-sizing, treating it like it was content-box.
- // Unlike in IE, offsetHeight shares the same issue so there's no easy way to workaround
- // the issue without incurring high size penalty. Let's at least check we get the size
- // as the browser sees it.
- if ( /android 4\.[0-3]/i.test( navigator.userAgent ) ) {
- assert.ok( [ 30, 32 ].indexOf( $td.outerHeight() ) > -1,
- "outerHeight of td with border-box should include padding." );
- } else {
- assert.strictEqual( $td.outerHeight(), 30, "outerHeight of td with border-box should include padding." );
- }
+ assert.strictEqual( $td.outerHeight(), 30, "outerHeight of td with border-box should include padding." );
assert.strictEqual( $textarea.outerHeight(), 6, "outerHeight of textarea with border-box should include padding and border." );
} );
diff --git a/test/unit/effects.js b/test/unit/effects.js
index 05fd1283ef0..6b0efa17b7e 100644
--- a/test/unit/effects.js
+++ b/test/unit/effects.js
@@ -690,44 +690,35 @@ QUnit.test( "stop()", function( assert ) {
this.clock.tick( 100 );
} );
-// In IE9 inside testswarm this test doesn't work properly
-( function() {
- var type = "test";
-
- if ( QUnit.isSwarm && /msie 9\.0/i.test( window.navigator.userAgent ) ) {
- type = "skip";
- }
-
- QUnit[ type ]( "stop() - several in queue", function( assert ) {
- assert.expect( 5 );
+QUnit.test( "stop() - several in queue", function( assert ) {
+ assert.expect( 5 );
- var nw, $foo = jQuery( "#foo" );
+ var nw, $foo = jQuery( "#foo" );
- // default duration is 400ms, so 800px ensures we aren't 0 or 1 after 1ms
- $foo.hide().css( "width", 800 );
+ // default duration is 400ms, so 800px ensures we aren't 0 or 1 after 1ms
+ $foo.hide().css( "width", 800 );
- $foo.animate( { "width": "show" }, 400, "linear" );
- $foo.animate( { "width": "hide" } );
- $foo.animate( { "width": "show" } );
+ $foo.animate( { "width": "show" }, 400, "linear" );
+ $foo.animate( { "width": "hide" } );
+ $foo.animate( { "width": "show" } );
- this.clock.tick( 1 );
+ this.clock.tick( 1 );
- jQuery.fx.tick();
- assert.equal( $foo.queue().length, 3, "3 in the queue" );
+ jQuery.fx.tick();
+ assert.equal( $foo.queue().length, 3, "3 in the queue" );
- nw = $foo.css( "width" );
- assert.notEqual( parseFloat( nw ), 1, "An animation occurred " + nw );
- $foo.stop();
+ nw = $foo.css( "width" );
+ assert.notEqual( parseFloat( nw ), 1, "An animation occurred " + nw );
+ $foo.stop();
- assert.equal( $foo.queue().length, 2, "2 in the queue" );
- nw = $foo.css( "width" );
- assert.notEqual( parseFloat( nw ), 1, "Stop didn't reset the animation " + nw );
+ assert.equal( $foo.queue().length, 2, "2 in the queue" );
+ nw = $foo.css( "width" );
+ assert.notEqual( parseFloat( nw ), 1, "Stop didn't reset the animation " + nw );
- $foo.stop( true );
+ $foo.stop( true );
- assert.equal( $foo.queue().length, 0, "0 in the queue" );
- } );
-} )();
+ assert.equal( $foo.queue().length, 0, "0 in the queue" );
+} );
QUnit.test( "stop(clearQueue)", function( assert ) {
assert.expect( 4 );
diff --git a/test/unit/event.js b/test/unit/event.js
index c7497b9b0f4..5c8b2f420a5 100644
--- a/test/unit/event.js
+++ b/test/unit/event.js
@@ -1414,7 +1414,7 @@ QUnit.test( "Submit event can be stopped (#11049)", function( assert ) {
form.remove();
} );
-// Support: iOS 7 - 9
+// Support: iOS <=7 - 12+
// iOS has the window.onbeforeunload field but doesn't support the beforeunload
// handler making it impossible to feature-detect the support.
QUnit[ /(ipad|iphone|ipod)/i.test( navigator.userAgent ) ? "skip" : "test" ](
@@ -2434,7 +2434,7 @@ QUnit.test( "event object properties on natively-triggered event", function( ass
$link = jQuery( link ),
evt = document.createEvent( "MouseEvents" );
- // Support: IE <=9 - 11 only
+ // Support: IE <=9 - 11+
// IE requires element to be in the body before it will dispatch
$link.appendTo( "body" ).on( "click", function( e ) {
@@ -2931,21 +2931,6 @@ QUnit.test( "trigger('click') on radio passes extra params", function( assert )
$radio.trigger( "click", [ true ] );
} );
-// Support: IE <=9 only
-// https://msdn.microsoft.com/en-us/library/hh801223(v=vs.85).aspx
-QUnit.test( "VML with special event handlers (trac-7071)", function( assert ) {
- assert.expect( 1 );
-
- var ns = jQuery( "" ).appendTo( "head" );
-
- jQuery( " " ).appendTo( "#form" );
- jQuery( "#form" ).on( "keydown", function() {
- assert.ok( true, "no error was thrown" );
- } );
- jQuery( "#oval" ).trigger( "click" ).trigger( "keydown" );
- ns.remove();
-} );
-
QUnit.test( "Check order of focusin/focusout events", function( assert ) {
assert.expect( 2 );
diff --git a/test/unit/manipulation.js b/test/unit/manipulation.js
index b0d3e3a884a..e203b2bb1a6 100644
--- a/test/unit/manipulation.js
+++ b/test/unit/manipulation.js
@@ -511,18 +511,6 @@ QUnit.test( "Tag name processing respects the HTML Standard (gh-2005)", function
}
function assertSpecialCharsSupport( method, characters ) {
- // Support: Android 4.4 only
- // Chromium < 35 incorrectly upper-cases µ; Android 4.4 uses such a version by default
- // (and its WebView, being un-updatable, will use it for eternity) so we need to blacklist
- // that one for the tests to pass.
- if ( characters === "µ" && /chrome/i.test( navigator.userAgent ) &&
- navigator.userAgent.match( /chrome\/(\d+)/i )[ 1 ] < 35 ) {
- assert.ok( true, "This Chromium version upper-cases µ incorrectly; skip test" );
- assert.ok( true, "This Chromium version upper-cases µ incorrectly; skip test" );
- assert.ok( true, "This Chromium version upper-cases µ incorrectly; skip test" );
- return;
- }
-
var child,
codepoint = characters.charCodeAt( 0 ).toString( 16 ).toUpperCase(),
description = characters.length === 1 ?
@@ -592,28 +580,7 @@ QUnit.test( "append(xml)", function( assert ) {
var xmlDoc, xml1, xml2;
function createXMLDoc() {
-
- // Initialize DOM based upon latest installed MSXML or Netscape
- var elem, n, len,
- aActiveX =
- [ "MSXML6.DomDocument",
- "MSXML3.DomDocument",
- "MSXML2.DomDocument",
- "MSXML.DomDocument",
- "Microsoft.XmlDom" ];
-
- if ( document.implementation && "createDocument" in document.implementation ) {
- return document.implementation.createDocument( "", "", null );
- } else {
-
- // IE
- for ( n = 0, len = aActiveX.length; n < len; n++ ) {
- try {
- elem = new window.ActiveXObject( aActiveX[ n ] );
- return elem;
- } catch ( _ ) {}
- }
- }
+ return document.implementation.createDocument( "", "", null );
}
xmlDoc = createXMLDoc();
@@ -1798,7 +1765,7 @@ QUnit.test( "html(Function)", function( assert ) {
} );
QUnit[
- // Support: Edge 16-18+
+ // Support: Edge 16 - 18 only
// Edge sometimes doesn't execute module scripts so skip the test there.
( QUnit.moduleTypeSupported && !/edge\//i.test( navigator.userAgent ) ) ?
"test" :
@@ -1826,7 +1793,7 @@ QUnit[
} );
QUnit[
- // Support: IE 9-11 only, Android 4.0-4.4 only, iOS 7-10 only
+ // Support: IE 9 - 11+, iOS <=7 - 10 only
// `nomodule` scripts should be executed by legacy browsers only.
// iOS 10 supports `