Tests: Correct logic of a broken assertion

diff --git a/test/unit/attributes.js b/test/unit/attributes.js
index 2ab583caa64deee613c113970f65a3b1245061a1..48d845f7a15c514102e81ac405b6611bcb828b02 100644 (file)
--- a/test/unit/attributes.js
+++ b/test/unit/attributes.js
@@ -378,8 +378,6 @@ test( "attr(String, Object)", function() {
                strictEqual( $elem.attr("nonexisting"), undefined, "attr(name, value) works correctly on comment and text nodes (bug #7500)." );
        });
 
-       // Register the property name to avoid generating a new global when testing window
-       Globals.register("nonexisting");
        jQuery.each( [ window, document, obj, "#firstp" ], function( i, elem ) {
                var oldVal = elem.nonexisting,
                        $elem = jQuery( elem );
@@ -388,6 +386,9 @@ test( "attr(String, Object)", function() {
                elem.nonexisting = oldVal;
        });
 
+       // Register the property on the window for the previous assertion so it will be clean up
+       Globals.register( "nonexisting" );
+
        table = jQuery("#table").append("<tr><td>cell</td></tr><tr><td>cell</td><td>cell</td></tr><tr><td>cell</td><td>cell</td></tr>");
        td = table.find("td").eq(0);
        td.attr( "rowspan", "2" );