]> 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:01:39 +0000 (22:01 +0400)
This check is redundant since jQuery.fragments object does not exist anymore

test/data/testrunner.js

index be1b62ae72a7fd44d0cb8f29f2995cad4abe7b34..07e2262074b71a5ec2e905c259b4d09a7b41a37f 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 = {},
@@ -117,7 +116,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
@@ -161,22 +159,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() {