Tests: Remove remaining obsolete jQuery.cache references

PR gh-4586 removed some of those but not all.

Closes gh-4715
Ref gh-4586

(cherry picked from commit d96111e18b42ae1bc7def72a8a0d156ea39e4d0e)

diff --git a/test/data/testrunner.js b/test/data/testrunner.js
index 2eb60e499f1399b79db50db58f74a6fba4bea593..efe88c3c6458d0d6513fb372f50babd3d755ad9d 100644 (file)
--- a/test/data/testrunner.js
+++ b/test/data/testrunner.js
@@ -1,11 +1,11 @@
 ( function() {
 
-// Store the old counts so that we only assert on tests that have actually leaked,
+"use strict";
+
+// Store the old count so that we only assert on tests that have actually leaked,
 // instead of asserting every time a test has leaked sometime in the past
-var oldCacheLength = 0,
-       oldActive = 0,
+var oldActive = 0,
 
-       expectedDataKeys = {},
        splice = [].splice,
        ajaxSettings = jQuery.ajaxSettings;
 
@@ -24,11 +24,6 @@ QUnit.config.requireExpects = true;
  * teardown function on all modules' lifecycle object.
  */
 window.moduleTeardown = function( assert ) {
-       var i, expectedKeys, actualKeys,
-               cacheLength = 0;
-
-       // Reset data register
-       expectedDataKeys = {};
 
        // Check for (and clean up, if possible) incomplete animations/requests/etc.
        if ( jQuery.timers && jQuery.timers.length !== 0 ) {
@@ -45,19 +40,6 @@ window.moduleTeardown = function( assert ) {
        }
 
        Globals.cleanup();
-
-       for ( i in jQuery.cache ) {
-               ++cacheLength;
-       }
-
-       // Because QUnit doesn't have a mechanism for retrieving
-       // the number of expected assertions for a test,
-       // if we unconditionally assert any of these,
-       // the test will fail with too many assertions :|
-       if ( cacheLength !== oldCacheLength ) {
-               assert.equal( cacheLength, oldCacheLength, "No unit tests leak memory in jQuery.cache" );
-               oldCacheLength = cacheLength;
-       }
 };
 
 QUnit.done( function() {

diff --git a/test/unit/wrap.js b/test/unit/wrap.js
index 21335c36f6bb1aeb36e80d55d63932c40331d776..3c0990eac25b70a433153cf2a213e7775275eb6a 100644 (file)
--- a/test/unit/wrap.js
+++ b/test/unit/wrap.js
@@ -21,7 +21,7 @@ function manipulationFunctionReturningObj( value ) {
 
 function testWrap( val, assert ) {
 
-       assert.expect( 19 );
+       assert.expect( 18 );
 
        var defaultText, result, j, i, cacheLength;
 
@@ -68,12 +68,6 @@ function testWrap( val, assert ) {
                "Check node,textnode,comment wraps doesn't hurt text"
        );
 
-       // Try wrapping a disconnected node
-       cacheLength = 0;
-       for ( i in jQuery.cache ) {
-               cacheLength++;
-       }
-
        j = jQuery( "<label></label>" ).wrap( val( "<li></li>" ) );
        assert.equal(
                j[ 0 ] .nodeName.toUpperCase(), "LABEL", "Element is a label"
@@ -82,13 +76,6 @@ function testWrap( val, assert ) {
                j[ 0 ].parentNode.nodeName.toUpperCase(), "LI", "Element has been wrapped"
        );
 
-       for ( i in jQuery.cache ) {
-               cacheLength--;
-       }
-       assert.equal(
-               cacheLength, 0, "No memory leak in jQuery.cache (bug #7165)"
-       );
-
        // Wrap an element containing a text node
        j = jQuery( "<span></span>" ).wrap( "<div>test</div>" );
        assert.equal(