From a3cd88f14bc99cbb80e6abe939576cbed093cb47 Mon Sep 17 00:00:00 2001 From: Oleg Date: Tue, 14 Jan 2014 21:48:42 +0400 Subject: [PATCH] Tests: Remove one of the checks for memory leak This check is redundant since jQuery.fragments object does not exist anymore (cherry-picked from 38c122a73a843a0d117d4f962cc94fda6f553086) --- test/data/testrunner.js | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/test/data/testrunner.js b/test/data/testrunner.js index b25c4146f..db68ffc86 100644 --- a/test/data/testrunner.js +++ b/test/data/testrunner.js @@ -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() { -- 2.39.5