]> source.dussan.org Git - jquery.git/commitdiff
Tests: Remove one of the checks for memory leak
authorOleg <markelog@gmail.com>
Tue, 14 Jan 2014 17:48:42 +0000 (21:48 +0400)
committerOleg <markelog@gmail.com>
Tue, 14 Jan 2014 18:02:21 +0000 (22:02 +0400)
This check is redundant since jQuery.fragments object does not exist anymore
(cherry-picked from 38c122a73a843a0d117d4f962cc94fda6f553086)

test/data/testrunner.js

index b25c4146f950b6f78c34e4d2726f8a69e09baff2..db68ffc86fbabf14a8c62a36d8edafc640bc7be0 100644 (file)
@@ -7,7 +7,6 @@ var oldStart = window.start,
        // Store the old counts so that we only assert on tests that have actually leaked,
        // instead of asserting every time a test has leaked sometime in the past
        oldCacheLength = 0,
-       oldFragmentsLength = 0,
        oldActive = 0,
 
        expectedDataKeys = {},
@@ -118,7 +117,6 @@ QUnit.config.urlConfig.push({
 window.moduleTeardown = function() {
        var i,
                expectedKeys, actualKeys,
-               fragmentsLength = 0,
                cacheLength = 0;
 
        // Only look for jQuery data problems if this test actually
@@ -163,22 +161,12 @@ window.moduleTeardown = function() {
                ++cacheLength;
        }
 
-       jQuery.fragments = {};
-
-       for ( i in jQuery.fragments ) {
-               ++fragmentsLength;
-       }
-
        // 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 ) {
                equal( cacheLength, oldCacheLength, "No unit tests leak memory in jQuery.cache" );
                oldCacheLength = cacheLength;
        }
-       if ( fragmentsLength !== oldFragmentsLength ) {
-               equal( fragmentsLength, oldFragmentsLength, "No unit tests leak memory in jQuery.fragments" );
-               oldFragmentsLength = fragmentsLength;
-       }
 };
 
 QUnit.done(function() {