From d62875fb0181c44f8b34a8e842ae323801637b25 Mon Sep 17 00:00:00 2001 From: Ariel Flesler Date: Fri, 9 Jan 2009 23:49:18 +0000 Subject: [PATCH] testrunner: Removing every global variable leaked within (and outside) tests. Some were deleted and some saved within the jQuery namespace. --- test/data/test.html | 2 +- test/data/test.js | 2 +- test/data/test.php | 2 +- test/data/test2.html | 2 +- test/unit/ajax.js | 40 +++++------ test/unit/core.js | 23 +++--- test/unit/fx.js | 162 ++++++++++++++++++++----------------------- 7 files changed, 108 insertions(+), 125 deletions(-) diff --git a/test/data/test.html b/test/data/test.html index eec028e90f..b7c74c8883 100644 --- a/test/data/test.html +++ b/test/data/test.html @@ -1,6 +1,6 @@ html text
diff --git a/test/data/test.js b/test/data/test.js index a41cb232b5..98e76de36a 100644 --- a/test/data/test.js +++ b/test/data/test.js @@ -1,3 +1,3 @@ -var foobar = "bar"; +jQuery.foobar = "bar"; jQuery('#ap').html('bar'); ok( true, "test.js executed"); diff --git a/test/data/test.php b/test/data/test.php index 3d08f3253a..bb392f151b 100644 --- a/test/data/test.php +++ b/test/data/test.php @@ -1,6 +1,6 @@ html text
diff --git a/test/data/test2.html b/test/data/test2.html index 1df6151a0c..363c6de039 100644 --- a/test/data/test2.html +++ b/test/data/test2.html @@ -1,5 +1,5 @@ diff --git a/test/unit/ajax.js b/test/unit/ajax.js index 09074f3098..357c5366f1 100644 --- a/test/unit/ajax.js +++ b/test/unit/ajax.js @@ -157,18 +157,16 @@ test("jQuery.ajax - beforeSend, cancel request (#2688)", function() { ok( request === false, "canceled request must return false instead of XMLHttpRequest instance" ); }); -var foobar; - test("jQuery.ajax - dataType html", function() { expect(5); stop(); - foobar = null; - testFoo = undefined; + jQuery.foobar = null; + jQuery.testFoo = undefined; var verifyEvaluation = function() { - equals( testFoo, "foo", 'Check if script was evaluated for datatype html' ); - equals( foobar, "bar", 'Check if script src was evaluated for datatype html' ); + equals( jQuery.testFoo, "foo", 'Check if script was evaluated for datatype html' ); + equals( jQuery.foobar, "bar", 'Check if script src was evaluated for datatype html' ); start(); }; @@ -359,17 +357,17 @@ test("load(String, Function) - simple: inject text into DOM", function() { test("load(String, Function) - check scripts", function() { expect(7); stop(); - window.testFoo = undefined; - window.foobar = null; + jQuery.testFoo = undefined; + jQuery.foobar = null; var verifyEvaluation = function() { - equals( foobar, "bar", 'Check if script src was evaluated after load' ); + equals( jQuery.foobar, "bar", 'Check if script src was evaluated after load' ); equals( jQuery('#ap').html(), 'bar', 'Check if script evaluation has modified DOM'); - start(); + start(); }; jQuery('#first').load(url('data/test.html'), function() { ok( jQuery('#first').html().match(/^html text/), 'Check content after loading html' ); equals( jQuery('#foo').html(), 'foo', 'Check if script evaluation has modified DOM'); - equals( testFoo, "foo", 'Check if script was evaluated after load' ); + equals( jQuery.testFoo, "foo", 'Check if script was evaluated after load' ); setTimeout(verifyEvaluation, 600); }); }); @@ -377,10 +375,10 @@ test("load(String, Function) - check scripts", function() { test("load(String, Function) - check file with only a script tag", function() { expect(3); stop(); - testFoo = undefined; + jQuery.testFoo = undefined; jQuery('#first').load(url('data/test2.html'), function() { equals( jQuery('#foo').html(), 'foo', 'Check if script evaluation has modified DOM'); - equals( testFoo, "foo", 'Check if script was evaluated after load' ); + equals( jQuery.testFoo, "foo", 'Check if script was evaluated after load' ); start(); }); }); @@ -426,9 +424,9 @@ test("jQuery.get(String, Hash, Function) - parse xml and use text() on nodes", f test("jQuery.getScript(String, Function) - with callback", function() { expect(2); stop(); - window.foobar = null; + jQuery.foobar = null; jQuery.getScript(url("data/test.js"), function() { - equals( foobar, "bar", 'Check if script was evaluated' ); + equals( jQuery.foobar, "bar", 'Check if script was evaluated' ); setTimeout(start, 100); }); }); @@ -618,12 +616,12 @@ test("jQuery.ajax() - script, Remote", function() { stop(); - window.foobar = null; + jQuery.foobar = null; jQuery.ajax({ url: base + "data/test.js", dataType: "script", success: function(data){ - ok( foobar, "Script results returned (GET, no callback)" ); + ok( jQuery.foobar, "Script results returned (GET, no callback)" ); start(); } }); @@ -636,13 +634,13 @@ test("jQuery.ajax() - script, Remote with POST", function() { stop(); - window.foobar = null; + jQuery.foobar = null; jQuery.ajax({ url: base + "data/test.js", type: "POST", dataType: "script", success: function(data, status){ - ok( foobar, "Script results returned (GET, no callback)" ); + ok( jQuery.foobar, "Script results returned (GET, no callback)" ); equals( status, "success", "Script results returned (GET, no callback)" ); start(); } @@ -657,12 +655,12 @@ test("jQuery.ajax() - script, Remote with scheme-less URL", function() { stop(); - window.foobar = null; + jQuery.foobar = null; jQuery.ajax({ url: base + "data/test.js", dataType: "script", success: function(data){ - ok( foobar, "Script results returned (GET, no callback)" ); + ok( jQuery.foobar, "Script results returned (GET, no callback)" ); start(); } }); diff --git a/test/unit/core.js b/test/unit/core.js index 98a161fa8e..ef36c4f784 100644 --- a/test/unit/core.js +++ b/test/unit/core.js @@ -146,7 +146,7 @@ test("browser", function() { }; for (var i in browsers) { var v = i.toLowerCase().match( /.+(?:rv|it|ra|ie)[\/: ]([\d.]+)/ ); // RegEx from Core jQuery.browser.version check - version = v ? v[1] : null; + var version = v ? v[1] : null; equals( version, browsers[i], "Checking UA string" ); } }); @@ -256,18 +256,16 @@ test("isFunction", function() { }); }); -var foo = false; - test("jQuery('html')", function() { expect(8); reset(); - foo = false; - var s = jQuery("")[0]; + jQuery.foo = false; + var s = jQuery("")[0]; ok( s, "Creating a script" ); - ok( !foo, "Make sure the script wasn't executed prematurely" ); - jQuery("body").append(""); - ok( foo, "Executing a scripts contents in the right context" ); + ok( !jQuery.foo, "Make sure the script wasn't executed prematurely" ); + jQuery("body").append(""); + ok( jQuery.foo, "Executing a scripts contents in the right context" ); reset(); ok( jQuery("")[0], "Creating a link" ); @@ -460,7 +458,7 @@ test("attr(Hash)", function() { test("attr(String, Object)", function() { expect(19); - var div = jQuery("div").attr("foo", "bar"); + var div = jQuery("div").attr("foo", "bar"), fail = false; for ( var i = 0; i < div.size(); i++ ) { if ( div.get(i).getAttribute('foo') != "bar" ){ @@ -1350,10 +1348,11 @@ test("val(String/Number)", function() { j.removeAttr("value"); }); -var scriptorder = 0; - test("html(String)", function() { expect(13); + + jQuery.scriptorder = 0; + var div = jQuery("#main > div"); div.html("test"); var pass = true; @@ -1386,7 +1385,7 @@ test("html(String)", function() { jQuery("#main").html('foo
'); // it was decided that waiting to execute ALL scripts makes sense since nested ones have to wait anyway so this test case is changed, see #1959 - jQuery("#main").html("