mirror of
https://github.com/jquery/jquery.git
synced 2026-04-20 03:01:22 -04:00
Tests: Remove obsolete jQuery data tests
The tests relied on `jQuery.cache` so they only ever worked in jQuery 1.x. Closes gh-4586
This commit is contained in:
committed by
GitHub
parent
9e66fe9acf
commit
eb35be528f
@@ -1411,10 +1411,6 @@ var testToggleClass = function( valueObj, assert ) {
|
||||
e.toggleClass( false );
|
||||
e.toggleClass();
|
||||
assert.ok( e.is( ".testD.testE" ), "Assert class present (restored from data)" );
|
||||
|
||||
// Cleanup
|
||||
e.removeClass( "testD" );
|
||||
assert.expectJqData( this, e[ 0 ], "__className__" );
|
||||
};
|
||||
|
||||
QUnit.test( "toggleClass(String|boolean|undefined[, boolean])", function( assert ) {
|
||||
|
||||
@@ -132,11 +132,6 @@ QUnit.test( "jQuery.data(div)", function( assert ) {
|
||||
var div = document.createElement( "div" );
|
||||
|
||||
dataTests( div, assert );
|
||||
|
||||
// We stored one key in the private data
|
||||
// assert that nothing else was put in there, and that that
|
||||
// one stayed there.
|
||||
assert.expectJqData( this, div, "foo" );
|
||||
} );
|
||||
|
||||
QUnit.test( "jQuery.data({})", function( assert ) {
|
||||
@@ -159,8 +154,6 @@ QUnit.test( "jQuery.data(document)", function( assert ) {
|
||||
assert.expect( 25 );
|
||||
|
||||
dataTests( document, assert );
|
||||
|
||||
assert.expectJqData( this, document, "foo" );
|
||||
} );
|
||||
|
||||
QUnit.test( "jQuery.data(<embed>)", function( assert ) {
|
||||
|
||||
12
test/unit/effects.js
vendored
12
test/unit/effects.js
vendored
@@ -90,9 +90,6 @@ QUnit.test( "show()", function( assert ) {
|
||||
assert.ok( pass, "Show with " + name + " does not call animate callback" );
|
||||
} );
|
||||
|
||||
// Tolerate data from show()/hide()
|
||||
assert.expectJqData( this, div, "olddisplay" );
|
||||
|
||||
jQuery(
|
||||
"<div id='show-tests'>" +
|
||||
"<div><p><a href='#'></a></p><code></code><pre></pre><span></span></div>" +
|
||||
@@ -217,8 +214,6 @@ supportjQuery.each( hideOptions, function( type, setup ) {
|
||||
} );
|
||||
|
||||
clock.tick( 300 );
|
||||
|
||||
assert.expectJqData( this, $span, "olddisplay" );
|
||||
} );
|
||||
|
||||
// Support: IE 11+, Edge 12 - 18+
|
||||
@@ -251,8 +246,6 @@ supportjQuery.each( hideOptions, function( type, setup ) {
|
||||
} );
|
||||
|
||||
clock.tick( 300 );
|
||||
|
||||
assert.expectJqData( this, $shadowChild, "olddisplay" );
|
||||
} );
|
||||
} );
|
||||
|
||||
@@ -1161,9 +1154,6 @@ QUnit.test( "interrupt toggle", function( assert ) {
|
||||
|
||||
// Save original property value for comparison
|
||||
jQuery.data( this, "startVal", jQuery( this ).css( prop ) );
|
||||
|
||||
// Expect olddisplay data from our .hide() call below
|
||||
assert.expectJqData( env, this, "olddisplay" );
|
||||
} );
|
||||
|
||||
// Interrupt a hiding toggle
|
||||
@@ -1613,8 +1603,6 @@ QUnit.test( "animate should set display for disconnected nodes", function( asser
|
||||
assert.strictEqual( $divInline.show()[ 0 ].style.display, "inline",
|
||||
"show() should not change display if it already set" );
|
||||
|
||||
assert.expectJqData( env, $divNone[ 0 ], "olddisplay" );
|
||||
|
||||
jQuery.each( showMethods, function( name, opt ) {
|
||||
jQuery.fn[ name ].apply( jQuery( "<div/>" ), opt.concat( [ function() {
|
||||
assert.strictEqual( jQuery( this ).css( "display" ), nullParentDisplay,
|
||||
|
||||
Reference in New Issue
Block a user