From: Oleg <markelog@gmail.com>
Date: Tue, 14 Jan 2014 17:48:42 +0000 (+0400)
Subject: Tests: Remove one of the checks for memory leak
X-Git-Tag: 1.11.0-rc1~7
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=a3cd88f14bc99cbb80e6abe939576cbed093cb47;p=jquery.git

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)
---

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() {