library-mirrors/jquery
1
0
Fork 0
mirror of https://github.com/jquery/jquery.git synced 2024-12-09 08:04:24 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Tests: Minor updates for QUnit 1.16 compatibility

Browse Source 
More to come later.

(cherry picked from commit f6f8848fbe)
This commit is contained in:
Richard Gibson 2014-12-05 12:28:40 -05:00
parent 6748ba3496
commit 26276a307c
5 changed files with 24 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
test/data/testrunner.js
View File

@ -24,14 +24,13 @@ QUnit.config.requireExpects = true;
* @param {jQuery|HTMLElement|Object|Array} elems Target (or array of targets) for jQuery.data. * @param {jQuery|HTMLElement|Object|Array} elems Target (or array of targets) for jQuery.data.
* @param {string} key * @param {string} key
*/ */
QUnit.expectJqData = function( elems, key ) { QUnit.expectJqData = function( env, elems, key ) {
var i, elem, expando, var i, elem, expando;
currentEnv = "current_testEnvironment";
// As of jQuery 2.0, there will be no "cache"-data is // As of jQuery 2.0, there will be no "cache"-data is
// stored and managed completely below the API surface // stored and managed completely below the API surface
if ( jQuery.cache ) { if ( jQuery.cache ) {
QUnit[ currentEnv ].checkJqData = true; env.checkJqData = true;
if ( elems.jquery && elems.toArray ) { if ( elems.jquery && elems.toArray ) {
elems = elems.toArray(); elems = elems.toArray();

2
test/unit/attributes.js
View File

@ -1266,7 +1266,7 @@ var testToggleClass = function(valueObj) {
// Cleanup // Cleanup
e.removeClass("testD"); e.removeClass("testD");
QUnit.expectJqData( e[ 0 ], "__className__" ); QUnit.expectJqData( this, e[ 0 ], "__className__" );
}; };
test( "toggleClass(String|boolean|undefined[, boolean])", function() { test( "toggleClass(String|boolean|undefined[, boolean])", function() {

4
test/unit/data.js
View File

@ -134,7 +134,7 @@ test("jQuery.data(div)", 25, function() {
// We stored one key in the private data // We stored one key in the private data
// assert that nothing else was put in there, and that that // assert that nothing else was put in there, and that that
// one stayed there. // one stayed there.
QUnit.expectJqData( div, "foo" ); QUnit.expectJqData( this, div, "foo" );
}); });
test("jQuery.data({})", 25, function() { test("jQuery.data({})", 25, function() {
@ -152,7 +152,7 @@ test("jQuery.data(window)", 25, function() {
test("jQuery.data(document)", 25, function() { test("jQuery.data(document)", 25, function() {
dataTests( document ); dataTests( document );
QUnit.expectJqData( document, "foo" ); QUnit.expectJqData( this, document, "foo" );
}); });
test("jQuery.data(<embed>)", 25, function() { test("jQuery.data(<embed>)", 25, function() {

12
test/unit/dimensions.js
View File

@ -57,7 +57,7 @@ function testWidth( val ) {
equal( jQuery(window).width(), document.documentElement.clientWidth, "Window width is equal to width reported by window/document." ); equal( jQuery(window).width(), document.documentElement.clientWidth, "Window width is equal to width reported by window/document." );
QUnit.expectJqData( $div[0], "olddisplay" ); QUnit.expectJqData( this, $div[0], "olddisplay" );
} }
test("width()", function() { test("width()", function() {
@ -110,7 +110,7 @@ function testHeight( val ) {
equal( jQuery(window).height(), document.documentElement.clientHeight, "Window width is equal to width reported by window/document." ); equal( jQuery(window).height(), document.documentElement.clientHeight, "Window width is equal to width reported by window/document." );
QUnit.expectJqData( $div[0], "olddisplay" ); QUnit.expectJqData( this, $div[0], "olddisplay" );
} }
test("height()", function() { test("height()", function() {
@ -166,7 +166,7 @@ test("innerWidth()", function() {
equal( div.innerWidth(), 0, "Make sure that disconnected nodes are handled." ); equal( div.innerWidth(), 0, "Make sure that disconnected nodes are handled." );
div.remove(); div.remove();
QUnit.expectJqData( $div[0], "olddisplay" ); QUnit.expectJqData( this, $div[0], "olddisplay" );
}); });
test("innerHeight()", function() { test("innerHeight()", function() {
@ -202,7 +202,7 @@ test("innerHeight()", function() {
equal( div.innerHeight(), 0, "Make sure that disconnected nodes are handled." ); equal( div.innerHeight(), 0, "Make sure that disconnected nodes are handled." );
div.remove(); div.remove();
QUnit.expectJqData( $div[0], "olddisplay" ); QUnit.expectJqData( this, $div[0], "olddisplay" );
}); });
test("outerWidth()", function() { test("outerWidth()", function() {
@ -241,7 +241,7 @@ test("outerWidth()", function() {
equal( div.outerWidth(), 0, "Make sure that disconnected nodes are handled." ); equal( div.outerWidth(), 0, "Make sure that disconnected nodes are handled." );
div.remove(); div.remove();
QUnit.expectJqData( $div[0], "olddisplay" ); QUnit.expectJqData( this, $div[0], "olddisplay" );
}); });
test("child of a hidden elem (or unconnected node) has accurate inner/outer/Width()/Height() see #9441 #9300", function() { test("child of a hidden elem (or unconnected node) has accurate inner/outer/Width()/Height() see #9441 #9300", function() {
@ -388,7 +388,7 @@ test("outerHeight()", function() {
equal( div.outerHeight(), 0, "Make sure that disconnected nodes are handled." ); equal( div.outerHeight(), 0, "Make sure that disconnected nodes are handled." );
div.remove(); div.remove();
QUnit.expectJqData( $div[0], "olddisplay" ); QUnit.expectJqData( this, $div[0], "olddisplay" );
}); });
test("passing undefined is a setter #5571", function() { test("passing undefined is a setter #5571", function() {

22
test/unit/effects.js vendored
View File

@ -45,7 +45,7 @@ test("show() basic", 2, function() {
// Clean up the detached node // Clean up the detached node
div.remove(); div.remove();
QUnit.expectJqData(hiddendiv, "olddisplay"); QUnit.expectJqData( this, hiddendiv, "olddisplay" );
}); });
test("show()", 27, function () { test("show()", 27, function () {
@ -92,7 +92,7 @@ test("show()", 27, function () {
}); });
// Tolerate data from show()/hide() // Tolerate data from show()/hide()
QUnit.expectJqData(div, "olddisplay"); QUnit.expectJqData( this, div, "olddisplay" );
// #show-tests * is set display: none in CSS // #show-tests * is set display: none in CSS
jQuery("#qunit-fixture").append("<div id='show-tests'><div><p><a href='#'></a></p><code></code><pre></pre><span></span></div><table><thead><tr><th></th></tr></thead><tbody><tr><td></td></tr></tbody></table><ul><li></li></ul></div><table id='test-table'></table>"); jQuery("#qunit-fixture").append("<div id='show-tests'><div><p><a href='#'></a></p><code></code><pre></pre><span></span></div><table><thead><tr><th></th></tr></thead><tbody><tr><td></td></tr></tbody></table><ul><li></li></ul></div><table id='test-table'></table>");
@ -200,7 +200,7 @@ test("Persist correct display value", function() {
clock.tick( 300 ); clock.tick( 300 );
QUnit.expectJqData($span, "olddisplay"); QUnit.expectJqData( this, $span, "olddisplay" );
}); });
test("animate(Hash, Object, Function)", function() { test("animate(Hash, Object, Function)", function() {
@ -1077,7 +1077,8 @@ test("jQuery.show('fast') doesn't clear radio buttons (bug #1095)", function ()
test( "interrupt toggle", function() { test( "interrupt toggle", function() {
expect( 24 ); expect( 24 );
var longDuration = 2000, var env = this,
longDuration = 2000,
shortDuration = 500, shortDuration = 500,
remaining = 0, remaining = 0,
$elems = jQuery(".chain-test"), $elems = jQuery(".chain-test"),
@ -1094,7 +1095,7 @@ test( "interrupt toggle", function() {
jQuery.data( this, "startVal", jQuery( this ).css( prop ) ); jQuery.data( this, "startVal", jQuery( this ).css( prop ) );
// Expect olddisplay data from our .hide() call below // Expect olddisplay data from our .hide() call below
QUnit.expectJqData( this, "olddisplay" ); QUnit.expectJqData( env, this, "olddisplay" );
}); });
// Interrupt a hiding toggle // Interrupt a hiding toggle
@ -1499,7 +1500,8 @@ test( "User supplied callback called after show when fx off (#8892)", 2, functio
test( "animate should set display for disconnected nodes", function() { test( "animate should set display for disconnected nodes", function() {
expect( 18 ); expect( 18 );
var methods = { var env = this,
methods = {
toggle: [ 1 ], toggle: [ 1 ],
slideToggle: [], slideToggle: [],
fadeIn: [], fadeIn: [],
@ -1520,9 +1522,9 @@ test( "animate should set display for disconnected nodes", function() {
strictEqual( $divNone.show()[ 0 ].style.display, "block", "show() should change display if it already set to none" ); strictEqual( $divNone.show()[ 0 ].style.display, "block", "show() should change display if it already set to none" );
strictEqual( $divInline.show()[ 0 ].style.display, "inline", "show() should not change display if it already set" ); strictEqual( $divInline.show()[ 0 ].style.display, "inline", "show() should not change display if it already set" );
QUnit.expectJqData( $divTest[ 0 ], "olddisplay" ); QUnit.expectJqData( env, $divTest[ 0 ], "olddisplay" );
QUnit.expectJqData( $divEmpty[ 0 ], "olddisplay" ); QUnit.expectJqData( env, $divEmpty[ 0 ], "olddisplay" );
QUnit.expectJqData( $divNone[ 0 ], "olddisplay" ); QUnit.expectJqData( env, $divNone[ 0 ], "olddisplay" );
jQuery.each( methods, function( name, opt ) { jQuery.each( methods, function( name, opt ) {
jQuery.each([ jQuery.each([
@ -1537,7 +1539,7 @@ test( "animate should set display for disconnected nodes", function() {
var callback = [function () { var callback = [function () {
strictEqual( this.style.display, "block", "set display to block with " + name ); strictEqual( this.style.display, "block", "set display to block with " + name );
QUnit.expectJqData( this, "olddisplay" ); QUnit.expectJqData( env, this, "olddisplay" );
}]; }];
jQuery.fn[ name ].apply( this, opt.concat( callback ) ); jQuery.fn[ name ].apply( this, opt.concat( callback ) );