Revert to QUnit v1.10.0 to avoid asyncTest issues.

Former-commit-id: 6127f8d2492eaef6f097d1ec5b25dadbf25e3af9

vendor/qunit/qunit/qunit.css

@@ -1,5 +1,5 @@
 /**
- * QUnit v1.11.0 - A JavaScript Unit Testing Framework
+ * QUnit v1.10.0 - A JavaScript Unit Testing Framework
  *
  * http://qunitjs.com
  *
@@ -20,7 +20,7 @@
 
 /** Resets */
 
-#qunit-tests, #qunit-header, #qunit-banner, #qunit-userAgent, #qunit-testresult, #qunit-modulefilter {
+#qunit-tests, #qunit-tests ol, #qunit-header, #qunit-banner, #qunit-userAgent, #qunit-testresult, #qunit-modulefilter {
 	margin: 0;
 	padding: 0;
 }
@@ -111,12 +111,7 @@
 	color: #000;
 }
 
-#qunit-tests li .runtime {
+#qunit-tests ol {
-	float: right;
-	font-size: smaller;
-}
-
-.qunit-assert-list {
 	margin-top: 0.5em;
 	padding: 0.5em;
 
@@ -127,10 +122,6 @@
 	-webkit-border-radius: 5px;
 }
 
-.qunit-collapsed {
-	display: none;
-}
-
 #qunit-tests table {
 	border-collapse: collapse;
 	margin-top: .2em;

vendor/qunit/qunit/qunit.js

@@ -1,5 +1,5 @@
 /**
- * QUnit v1.11.0 - A JavaScript Unit Testing Framework
+ * QUnit v1.10.0 - A JavaScript Unit Testing Framework
  *
  * http://qunitjs.com
  *
@@ -11,7 +11,6 @@
 (function( window ) {
 
 var QUnit,
-	assert,
 	config,
 	onErrorFnPrev,
 	testId = 0,
@@ -21,67 +20,18 @@ var QUnit,
 	// Keep a local reference to Date (GH-283)
 	Date = window.Date,
 	defined = {
-		setTimeout: typeof window.setTimeout !== "undefined",
+	setTimeout: typeof window.setTimeout !== "undefined",
-		sessionStorage: (function() {
+	sessionStorage: (function() {
-			var x = "qunit-test-string";
+		var x = "qunit-test-string";
-			try {
+		try {
-				sessionStorage.setItem( x, x );
+			sessionStorage.setItem( x, x );
-				sessionStorage.removeItem( x );
+			sessionStorage.removeItem( x );
-				return true;
+			return true;
-			} catch( e ) {
+		} catch( e ) {
-				return false;
+			return false;
-			}
-		}())
-	},
-	/**
-	 * Provides a normalized error string, correcting an issue
-	 * with IE 7 (and prior) where Error.prototype.toString is
-	 * not properly implemented
-	 *
-	 * Based on http://es5.github.com/#x15.11.4.4
-	 *
-	 * @param {String|Error} error
-	 * @return {String} error message
-	 */
-	errorString = function( error ) {
-		var name, message,
-			errorString = error.toString();
-		if ( errorString.substring( 0, 7 ) === "[object" ) {
-			name = error.name ? error.name.toString() : "Error";
-			message = error.message ? error.message.toString() : "";
-			if ( name && message ) {
-				return name + ": " + message;
-			} else if ( name ) {
-				return name;
-			} else if ( message ) {
-				return message;
-			} else {
-				return "Error";
-			}
-		} else {
-			return errorString;
 		}
-	},
+	}())
-	/**
+};
-	 * Makes a clone of an object using only Array or Object as base,
-	 * and copies over the own enumerable properties.
-	 *
-	 * @param {Object} obj
-	 * @return {Object} New object with only the own properties (recursively).
-	 */
-	objectValues = function( obj ) {
-		// Grunt 0.3.x uses an older version of jshint that still has jshint/jshint#392.
-		/*jshint newcap: false */
-		var key, val,
-			vals = QUnit.is( "array", obj ) ? [] : {};
-		for ( key in obj ) {
-			if ( hasOwn.call( obj, key ) ) {
-				val = obj[key];
-				vals[key] = val === Object(val) ? objectValues(val) : val;
-			}
-		}
-		return vals;
-	};
 
 function Test( settings ) {
 	extend( this, settings );
@@ -94,11 +44,11 @@ Test.count = 0;
 Test.prototype = {
 	init: function() {
 		var a, b, li,
-			tests = id( "qunit-tests" );
+        tests = id( "qunit-tests" );
 
 		if ( tests ) {
 			b = document.createElement( "strong" );
-			b.innerHTML = this.nameHtml;
+			b.innerHTML = this.name;
 
 			// `a` initialized at top of scope
 			a = document.createElement( "a" );
@@ -142,7 +92,6 @@ Test.prototype = {
 			teardown: function() {}
 		}, this.moduleTestEnvironment );
 
-		this.started = +new Date();
 		runLoggingCallbacks( "testStart", QUnit, {
 			name: this.testName,
 			module: this.module
@@ -162,7 +111,7 @@ Test.prototype = {
 		try {
 			this.testEnvironment.setup.call( this.testEnvironment );
 		} catch( e ) {
-			QUnit.pushFailure( "Setup failed on " + this.testName + ": " + ( e.message || e ), extractStacktrace( e, 1 ) );
+			QUnit.pushFailure( "Setup failed on " + this.testName + ": " + e.message, extractStacktrace( e, 1 ) );
 		}
 	},
 	run: function() {
@@ -171,28 +120,22 @@ Test.prototype = {
 		var running = id( "qunit-testresult" );
 
 		if ( running ) {
-			running.innerHTML = "Running: <br/>" + this.nameHtml;
+			running.innerHTML = "Running: <br/>" + this.name;
 		}
 
 		if ( this.async ) {
 			QUnit.stop();
 		}
 
-		this.callbackStarted = +new Date();
-
 		if ( config.notrycatch ) {
 			this.callback.call( this.testEnvironment, QUnit.assert );
-			this.callbackRuntime = +new Date() - this.callbackStarted;
 			return;
 		}
 
 		try {
 			this.callback.call( this.testEnvironment, QUnit.assert );
-			this.callbackRuntime = +new Date() - this.callbackStarted;
 		} catch( e ) {
-			this.callbackRuntime = +new Date() - this.callbackStarted;
+			QUnit.pushFailure( "Died on test #" + (this.assertions.length + 1) + " " + this.stack + ": " + e.message, extractStacktrace( e, 0 ) );
-
-			QUnit.pushFailure( "Died on test #" + (this.assertions.length + 1) + " " + this.stack + ": " + ( e.message || e ), extractStacktrace( e, 0 ) );
 			// else next test will carry the responsibility
 			saveGlobal();
 
@@ -205,43 +148,38 @@ Test.prototype = {
 	teardown: function() {
 		config.current = this;
 		if ( config.notrycatch ) {
-			if ( typeof this.callbackRuntime === "undefined" ) {
-				this.callbackRuntime = +new Date() - this.callbackStarted;
-			}
 			this.testEnvironment.teardown.call( this.testEnvironment );
 			return;
 		} else {
 			try {
 				this.testEnvironment.teardown.call( this.testEnvironment );
 			} catch( e ) {
-				QUnit.pushFailure( "Teardown failed on " + this.testName + ": " + ( e.message || e ), extractStacktrace( e, 1 ) );
+				QUnit.pushFailure( "Teardown failed on " + this.testName + ": " + e.message, extractStacktrace( e, 1 ) );
 			}
 		}
 		checkPollution();
 	},
 	finish: function() {
 		config.current = this;
-		if ( config.requireExpects && this.expected === null ) {
+		if ( config.requireExpects && this.expected == null ) {
 			QUnit.pushFailure( "Expected number of assertions to be defined, but expect() was not called.", this.stack );
-		} else if ( this.expected !== null && this.expected !== this.assertions.length ) {
+		} else if ( this.expected != null && this.expected != this.assertions.length ) {
 			QUnit.pushFailure( "Expected " + this.expected + " assertions, but " + this.assertions.length + " were run", this.stack );
-		} else if ( this.expected === null && !this.assertions.length ) {
+		} else if ( this.expected == null && !this.assertions.length ) {
 			QUnit.pushFailure( "Expected at least one assertion, but none were run - call expect(0) to accept zero assertions.", this.stack );
 		}
 
-		var i, assertion, a, b, time, li, ol,
+		var assertion, a, b, i, li, ol,
 			test = this,
 			good = 0,
 			bad = 0,
 			tests = id( "qunit-tests" );
 
-		this.runtime = +new Date() - this.started;
 		config.stats.all += this.assertions.length;
 		config.moduleStats.all += this.assertions.length;
 
 		if ( tests ) {
 			ol = document.createElement( "ol" );
-			ol.className = "qunit-assert-list";
 
 			for ( i = 0; i < this.assertions.length; i++ ) {
 				assertion = this.assertions[i];
@@ -270,22 +208,22 @@ Test.prototype = {
 			}
 
 			if ( bad === 0 ) {
-				addClass( ol, "qunit-collapsed" );
+				ol.style.display = "none";
 			}
 
 			// `b` initialized at top of scope
 			b = document.createElement( "strong" );
-			b.innerHTML = this.nameHtml + " <b class='counts'>(<b class='failed'>" + bad + "</b>, <b class='passed'>" + good + "</b>, " + this.assertions.length + ")</b>";
+			b.innerHTML = this.name + " <b class='counts'>(<b class='failed'>" + bad + "</b>, <b class='passed'>" + good + "</b>, " + this.assertions.length + ")</b>";
 
 			addEvent(b, "click", function() {
-				var next = b.parentNode.lastChild,
+				var next = b.nextSibling.nextSibling,
-					collapsed = hasClass( next, "qunit-collapsed" );
+					display = next.style.display;
-				( collapsed ? removeClass : addClass )( next, "qunit-collapsed" );
+				next.style.display = display === "none" ? "block" : "none";
 			});
 
 			addEvent(b, "dblclick", function( e ) {
 				var target = e && e.target ? e.target : window.event.srcElement;
-				if ( target.nodeName.toLowerCase() === "span" || target.nodeName.toLowerCase() === "b" ) {
+				if ( target.nodeName.toLowerCase() == "span" || target.nodeName.toLowerCase() == "b" ) {
 					target = target.parentNode;
 				}
 				if ( window.location && target.nodeName.toLowerCase() === "strong" ) {
@@ -293,19 +231,13 @@ Test.prototype = {
 				}
 			});
 
-			// `time` initialized at top of scope
-			time = document.createElement( "span" );
-			time.className = "runtime";
-			time.innerHTML = this.runtime + " ms";
-
 			// `li` initialized at top of scope
 			li = id( this.id );
 			li.className = bad ? "fail" : "pass";
 			li.removeChild( li.firstChild );
 			a = li.firstChild;
 			li.appendChild( b );
-			li.appendChild( a );
+			li.appendChild ( a );
-			li.appendChild( time );
 			li.appendChild( ol );
 
 		} else {
@@ -323,8 +255,7 @@ Test.prototype = {
 			module: this.module,
 			failed: bad,
 			passed: this.assertions.length - bad,
-			total: this.assertions.length,
+			total: this.assertions.length
-			duration: this.runtime
 		});
 
 		QUnit.reset();
@@ -390,7 +321,7 @@ QUnit = {
 
 	test: function( testName, expected, callback, async ) {
 		var test,
-			nameHtml = "<span class='test-name'>" + escapeText( testName ) + "</span>";
+			name = "<span class='test-name'>" + escapeInnerText( testName ) + "</span>";
 
 		if ( arguments.length === 2 ) {
 			callback = expected;
@@ -398,11 +329,11 @@ QUnit = {
 		}
 
 		if ( config.currentModule ) {
-			nameHtml = "<span class='module-name'>" + escapeText( config.currentModule ) + "</span>: " + nameHtml;
+			name = "<span class='module-name'>" + config.currentModule + "</span>: " + name;
 		}
 
 		test = new Test({
-			nameHtml: nameHtml,
+			name: name,
 			testName: testName,
 			expected: expected,
 			async: async,
@@ -429,18 +360,6 @@ QUnit = {
 	},
 
 	start: function( count ) {
-		// QUnit hasn't been initialized yet.
-		// Note: RequireJS (et al) may delay onLoad
-		if ( config.semaphore === undefined ) {
-			QUnit.begin(function() {
-				// This is triggered at the top of QUnit.load, push start() to the event loop, to allow QUnit.load to finish first
-				setTimeout(function() {
-					QUnit.start( count );
-				});
-			});
-			return;
-		}
-
 		config.semaphore -= count || 1;
 		// don't start until equal number of stop-calls
 		if ( config.semaphore > 0 ) {
@@ -449,8 +368,6 @@ QUnit = {
 		// ignore if start is called more often then stop
 		if ( config.semaphore < 0 ) {
 			config.semaphore = 0;
-			QUnit.pushFailure( "Called start() while already started (QUnit.config.semaphore was 0 already)", null, sourceFromStacktrace(2) );
-			return;
 		}
 		// A slight delay, to avoid any current callbacks
 		if ( defined.setTimeout ) {
@@ -486,14 +403,11 @@ QUnit = {
 	}
 };
 
-// `assert` initialized at top of scope
 // Asssert helpers
-// All of these must either call QUnit.push() or manually do:
+// All of these must call either QUnit.push() or manually do:
 // - runLoggingCallbacks( "log", .. );
 // - config.current.assertions.push({ .. });
-// We attach it to the QUnit object *after* we expose the public API,
+QUnit.assert = {
-// otherwise `assert` will become a global variable in browsers (#341).
-assert = {
 	/**
 	 * Asserts rough true-ish result.
 	 * @name ok
@@ -514,14 +428,14 @@ assert = {
 				message: msg
 			};
 
-		msg = escapeText( msg || (result ? "okay" : "failed" ) );
+		msg = escapeInnerText( msg || (result ? "okay" : "failed" ) );
 		msg = "<span class='test-message'>" + msg + "</span>";
 
 		if ( !result ) {
 			source = sourceFromStacktrace( 2 );
 			if ( source ) {
 				details.source = source;
-				msg += "<table><tr class='test-source'><th>Source: </th><td><pre>" + escapeText( source ) + "</pre></td></tr></table>";
+				msg += "<table><tr class='test-source'><th>Source: </th><td><pre>" + escapeInnerText( source ) + "</pre></td></tr></table>";
 			}
 		}
 		runLoggingCallbacks( "log", QUnit, details );
@@ -539,7 +453,6 @@ assert = {
 	 * @example equal( format( "Received {0} bytes.", 2), "Received 2 bytes.", "format() replaces {0} with next argument" );
 	 */
 	equal: function( actual, expected, message ) {
-		/*jshint eqeqeq:false */
 		QUnit.push( expected == actual, actual, expected, message );
 	},
 
@@ -548,30 +461,9 @@ assert = {
 	 * @function
 	 */
 	notEqual: function( actual, expected, message ) {
-		/*jshint eqeqeq:false */
 		QUnit.push( expected != actual, actual, expected, message );
 	},
 
-	/**
-	 * @name propEqual
-	 * @function
-	 */
-	propEqual: function( actual, expected, message ) {
-		actual = objectValues(actual);
-		expected = objectValues(expected);
-		QUnit.push( QUnit.equiv(actual, expected), actual, expected, message );
-	},
-
-	/**
-	 * @name notPropEqual
-	 * @function
-	 */
-	notPropEqual: function( actual, expected, message ) {
-		actual = objectValues(actual);
-		expected = objectValues(expected);
-		QUnit.push( !QUnit.equiv(actual, expected), actual, expected, message );
-	},
-
 	/**
 	 * @name deepEqual
 	 * @function
@@ -604,9 +496,8 @@ assert = {
 		QUnit.push( expected !== actual, actual, expected, message );
 	},
 
-	"throws": function( block, expected, message ) {
+	throws: function( block, expected, message ) {
 		var actual,
-			expectedOutput = expected,
 			ok = false;
 
 		// 'expected' is optional
@@ -627,20 +518,18 @@ assert = {
 			// we don't want to validate thrown error
 			if ( !expected ) {
 				ok = true;
-				expectedOutput = null;
 			// expected is a regexp
 			} else if ( QUnit.objectType( expected ) === "regexp" ) {
-				ok = expected.test( errorString( actual ) );
+				ok = expected.test( actual );
 			// expected is a constructor
 			} else if ( actual instanceof expected ) {
 				ok = true;
 			// expected is a validation function which returns true is validation passed
 			} else if ( expected.call( {}, actual ) === true ) {
-				expectedOutput = null;
 				ok = true;
 			}
 
-			QUnit.push( ok, actual, expectedOutput, message );
+			QUnit.push( ok, actual, null, message );
 		} else {
 			QUnit.pushFailure( message, null, 'No exception was thrown.' );
 		}
@@ -649,16 +538,15 @@ assert = {
 
 /**
  * @deprecate since 1.8.0
- * Kept assertion helpers in root for backwards compatibility.
+ * Kept assertion helpers in root for backwards compatibility
  */
-extend( QUnit, assert );
+extend( QUnit, QUnit.assert );
 
 /**
  * @deprecated since 1.9.0
- * Kept root "raises()" for backwards compatibility.
+ * Kept global "raises()" for backwards compatibility
- * (Note that we don't introduce assert.raises).
  */
-QUnit.raises = assert[ "throws" ];
+QUnit.raises = QUnit.assert.throws;
 
 /**
  * @deprecated since 1.0.0, replaced with error pushes since 1.3.0
@@ -734,15 +622,6 @@ config = {
 	moduleDone: []
 };
 
-// Export global variables, unless an 'exports' object exists,
-// in that case we assume we're in CommonJS (dealt with on the bottom of the script)
-if ( typeof exports === "undefined" ) {
-	extend( window, QUnit );
-
-	// Expose QUnit object
-	window.QUnit = QUnit;
-}
-
 // Initialize more QUnit.config and QUnit.urlParams
 (function() {
 	var i,
@@ -776,11 +655,18 @@ if ( typeof exports === "undefined" ) {
 	QUnit.isLocal = location.protocol === "file:";
 }());
 
+// Export global variables, unless an 'exports' object exists,
+// in that case we assume we're in CommonJS (dealt with on the bottom of the script)
+if ( typeof exports === "undefined" ) {
+	extend( window, QUnit );
+
+	// Expose QUnit object
+	window.QUnit = QUnit;
+}
+
 // Extend QUnit object,
 // these after set here because they should not be exposed as global functions
 extend( QUnit, {
-	assert: assert,
-
 	config: config,
 
 	// Initialize the configuration options
@@ -795,7 +681,7 @@ extend( QUnit, {
 			autorun: false,
 			filter: "",
 			queue: [],
-			semaphore: 1
+			semaphore: 0
 		});
 
 		var tests, banner, result,
@@ -803,7 +689,7 @@ extend( QUnit, {
 
 		if ( qunit ) {
 			qunit.innerHTML =
-				"<h1 id='qunit-header'>" + escapeText( document.title ) + "</h1>" +
+				"<h1 id='qunit-header'>" + escapeInnerText( document.title ) + "</h1>" +
 				"<h2 id='qunit-banner'></h2>" +
 				"<div id='qunit-testrunner-toolbar'></div>" +
 				"<h2 id='qunit-userAgent'></h2>" +
@@ -859,7 +745,7 @@ extend( QUnit, {
 
 	// Safe object type checking
 	is: function( type, obj ) {
-		return QUnit.objectType( obj ) === type;
+		return QUnit.objectType( obj ) == type;
 	},
 
 	objectType: function( obj ) {
@@ -871,8 +757,7 @@ extend( QUnit, {
 				return "null";
 		}
 
-		var match = toString.call( obj ).match(/^\[object\s(.*)\]$/),
+		var type = toString.call( obj ).match(/^\[object\s(.*)\]$/)[1] || "";
-			type = match && match[1] || "";
 
 		switch ( type ) {
 			case "Number":
@@ -909,16 +794,16 @@ extend( QUnit, {
 				expected: expected
 			};
 
-		message = escapeText( message ) || ( result ? "okay" : "failed" );
+		message = escapeInnerText( message ) || ( result ? "okay" : "failed" );
 		message = "<span class='test-message'>" + message + "</span>";
 		output = message;
 
 		if ( !result ) {
-			expected = escapeText( QUnit.jsDump.parse(expected) );
+			expected = escapeInnerText( QUnit.jsDump.parse(expected) );
-			actual = escapeText( QUnit.jsDump.parse(actual) );
+			actual = escapeInnerText( QUnit.jsDump.parse(actual) );
 			output += "<table><tr class='test-expected'><th>Expected: </th><td><pre>" + expected + "</pre></td></tr>";
 
-			if ( actual !== expected ) {
+			if ( actual != expected ) {
 				output += "<tr class='test-actual'><th>Result: </th><td><pre>" + actual + "</pre></td></tr>";
 				output += "<tr class='test-diff'><th>Diff: </th><td><pre>" + QUnit.diff( expected, actual ) + "</pre></td></tr>";
 			}
@@ -927,7 +812,7 @@ extend( QUnit, {
 
 			if ( source ) {
 				details.source = source;
-				output += "<tr class='test-source'><th>Source: </th><td><pre>" + escapeText( source ) + "</pre></td></tr>";
+				output += "<tr class='test-source'><th>Source: </th><td><pre>" + escapeInnerText( source ) + "</pre></td></tr>";
 			}
 
 			output += "</table>";
@@ -954,19 +839,19 @@ extend( QUnit, {
 				message: message
 			};
 
-		message = escapeText( message ) || "error";
+		message = escapeInnerText( message ) || "error";
 		message = "<span class='test-message'>" + message + "</span>";
 		output = message;
 
 		output += "<table>";
 
 		if ( actual ) {
-			output += "<tr class='test-actual'><th>Result: </th><td><pre>" + escapeText( actual ) + "</pre></td></tr>";
+			output += "<tr class='test-actual'><th>Result: </th><td><pre>" + escapeInnerText( actual ) + "</pre></td></tr>";
 		}
 
 		if ( source ) {
 			details.source = source;
-			output += "<tr class='test-source'><th>Source: </th><td><pre>" + escapeText( source ) + "</pre></td></tr>";
+			output += "<tr class='test-source'><th>Source: </th><td><pre>" + escapeInnerText( source ) + "</pre></td></tr>";
 		}
 
 		output += "</table>";
@@ -991,8 +876,7 @@ extend( QUnit, {
 			querystring += encodeURIComponent( key ) + "=" +
 				encodeURIComponent( params[ key ] ) + "&";
 		}
-		return window.location.protocol + "//" + window.location.host +
+		return window.location.pathname + querystring.slice( 0, -1 );
-			window.location.pathname + querystring.slice( 0, -1 );
 	},
 
 	extend: extend,
@@ -1023,7 +907,7 @@ extend( QUnit.constructor.prototype, {
 	// testStart: { name }
 	testStart: registerLoggingCallback( "testStart" ),
 
-	// testDone: { name, failed, passed, total, duration }
+	// testDone: { name, failed, passed, total }
 	testDone: registerLoggingCallback( "testDone" ),
 
 	// moduleStart: { name }
@@ -1041,10 +925,9 @@ QUnit.load = function() {
 	runLoggingCallbacks( "begin", QUnit, {} );
 
 	// Initialize the config, saving the execution queue
-	var banner, filter, i, label, len, main, ol, toolbar, userAgent, val,
+	var banner, filter, i, label, len, main, ol, toolbar, userAgent, val, urlConfigCheckboxes, moduleFilter,
-		urlConfigCheckboxesContainer, urlConfigCheckboxes, moduleFilter,
+	    numModules = 0,
-		numModules = 0,
+	    moduleFilterHtml = "",
-		moduleFilterHtml = "",
 		urlConfigHtml = "",
 		oldconfig = extend( {}, config );
 
@@ -1065,24 +948,14 @@ QUnit.load = function() {
 			};
 		}
 		config[ val.id ] = QUnit.urlParams[ val.id ];
-		urlConfigHtml += "<input id='qunit-urlconfig-" + escapeText( val.id ) +
+		urlConfigHtml += "<input id='qunit-urlconfig-" + val.id + "' name='" + val.id + "' type='checkbox'" + ( config[ val.id ] ? " checked='checked'" : "" ) + " title='" + val.tooltip + "'><label for='qunit-urlconfig-" + val.id + "' title='" + val.tooltip + "'>" + val.label + "</label>";
-			"' name='" + escapeText( val.id ) +
-			"' type='checkbox'" + ( config[ val.id ] ? " checked='checked'" : "" ) +
-			" title='" + escapeText( val.tooltip ) +
-			"'><label for='qunit-urlconfig-" + escapeText( val.id ) +
-			"' title='" + escapeText( val.tooltip ) + "'>" + val.label + "</label>";
 	}
 
-	moduleFilterHtml += "<label for='qunit-modulefilter'>Module: </label><select id='qunit-modulefilter' name='modulefilter'><option value='' " +
+	moduleFilterHtml += "<label for='qunit-modulefilter'>Module: </label><select id='qunit-modulefilter' name='modulefilter'><option value='' " + ( config.module === undefined  ? "selected" : "" ) + ">< All Modules ></option>";
-		( config.module === undefined  ? "selected='selected'" : "" ) +
-		">< All Modules ></option>";
-
 	for ( i in config.modules ) {
 		if ( config.modules.hasOwnProperty( i ) ) {
 			numModules += 1;
-			moduleFilterHtml += "<option value='" + escapeText( encodeURIComponent(i) ) + "' " +
+			moduleFilterHtml += "<option value='" + encodeURIComponent(i) + "' " + ( config.module === i ? "selected" : "" ) + ">" + i + "</option>";
-				( config.module === i ? "selected='selected'" : "" ) +
-				">" + escapeText(i) + "</option>";
 		}
 	}
 	moduleFilterHtml += "</select>";
@@ -1141,28 +1014,22 @@ QUnit.load = function() {
 		label.innerHTML = "Hide passed tests";
 		toolbar.appendChild( label );
 
-		urlConfigCheckboxesContainer = document.createElement("span");
+		urlConfigCheckboxes = document.createElement( 'span' );
-		urlConfigCheckboxesContainer.innerHTML = urlConfigHtml;
+		urlConfigCheckboxes.innerHTML = urlConfigHtml;
-		urlConfigCheckboxes = urlConfigCheckboxesContainer.getElementsByTagName("input");
+		addEvent( urlConfigCheckboxes, "change", function( event ) {
-		// For oldIE support:
+			var params = {};
-		// * Add handlers to the individual elements instead of the container
+			params[ event.target.name ] = event.target.checked ? true : undefined;
-		// * Use "click" instead of "change"
-		// * Fallback from event.target to event.srcElement
-		addEvents( urlConfigCheckboxes, "click", function( event ) {
-			var params = {},
-				target = event.target || event.srcElement;
-			params[ target.name ] = target.checked ? true : undefined;
 			window.location = QUnit.url( params );
 		});
-		toolbar.appendChild( urlConfigCheckboxesContainer );
+		toolbar.appendChild( urlConfigCheckboxes );
 
 		if (numModules > 1) {
 			moduleFilter = document.createElement( 'span' );
 			moduleFilter.setAttribute( 'id', 'qunit-modulefilter-container' );
 			moduleFilter.innerHTML = moduleFilterHtml;
-			addEvent( moduleFilter.lastChild, "change", function() {
+			addEvent( moduleFilter, "change", function() {
 				var selectBox = moduleFilter.getElementsByTagName("select")[0],
-					selectedModule = decodeURIComponent(selectBox.options[selectBox.selectedIndex].value);
+				    selectedModule = decodeURIComponent(selectBox.options[selectBox.selectedIndex].value);
 
 				window.location = QUnit.url( { module: ( selectedModule === "" ) ? undefined : selectedModule } );
 			});
@@ -1239,7 +1106,7 @@ function done() {
 			" milliseconds.<br/>",
 			"<span class='passed'>",
 			passed,
-			"</span> assertions of <span class='total'>",
+			"</span> tests of <span class='total'>",
 			config.stats.all,
 			"</span> passed, <span class='failed'>",
 			config.stats.bad,
@@ -1332,7 +1199,7 @@ function validTest( test ) {
 function extractStacktrace( e, offset ) {
 	offset = offset === undefined ? 3 : offset;
 
-	var stack, include, i;
+	var stack, include, i, regex;
 
 	if ( e.stacktrace ) {
 		// Opera
@@ -1346,7 +1213,7 @@ function extractStacktrace( e, offset ) {
 		if ( fileName ) {
 			include = [];
 			for ( i = offset; i < stack.length; i++ ) {
-				if ( stack[ i ].indexOf( fileName ) !== -1 ) {
+				if ( stack[ i ].indexOf( fileName ) != -1 ) {
 					break;
 				}
 				include.push( stack[ i ] );
@@ -1375,27 +1242,17 @@ function sourceFromStacktrace( offset ) {
 	}
 }
 
-/**
+function escapeInnerText( s ) {
- * Escape text for attribute or text content.
- */
-function escapeText( s ) {
 	if ( !s ) {
 		return "";
 	}
 	s = s + "";
-	// Both single quotes and double quotes (for attributes)
+	return s.replace( /[\&<>]/g, function( s ) {
-	return s.replace( /['"<>&]/g, function( s ) {
 		switch( s ) {
-			case '\'':
+			case "&": return "&amp;";
-				return '&#039;';
+			case "<": return "&lt;";
-			case '"':
+			case ">": return "&gt;";
-				return '&quot;';
+			default: return s;
-			case '<':
-				return '&lt;';
-			case '>':
-				return '&gt;';
-			case '&':
-				return '&amp;';
 		}
 	});
 }
@@ -1443,7 +1300,7 @@ function saveGlobal() {
 	}
 }
 
-function checkPollution() {
+function checkPollution( name ) {
 	var newGlobals,
 		deletedGlobals,
 		old = config.pollution;
@@ -1492,53 +1349,16 @@ function extend( a, b ) {
 	return a;
 }
 
-/**
- * @param {HTMLElement} elem
- * @param {string} type
- * @param {Function} fn
- */
 function addEvent( elem, type, fn ) {
-	// Standards-based browsers
 	if ( elem.addEventListener ) {
 		elem.addEventListener( type, fn, false );
-	// IE
+	} else if ( elem.attachEvent ) {
-	} else {
 		elem.attachEvent( "on" + type, fn );
+	} else {
+		fn();
 	}
 }
 
-/**
- * @param {Array|NodeList} elems
- * @param {string} type
- * @param {Function} fn
- */
-function addEvents( elems, type, fn ) {
-	var i = elems.length;
-	while ( i-- ) {
-		addEvent( elems[i], type, fn );
-	}
-}
-
-function hasClass( elem, name ) {
-	return (" " + elem.className + " ").indexOf(" " + name + " ") > -1;
-}
-
-function addClass( elem, name ) {
-	if ( !hasClass( elem, name ) ) {
-		elem.className += (elem.className ? " " : "") + name;
-	}
-}
-
-function removeClass( elem, name ) {
-	var set = " " + elem.className + " ";
-	// Class name may appear multiple times
-	while ( set.indexOf(" " + name + " ") > -1 ) {
-		set = set.replace(" " + name + " " , " ");
-	}
-	// If possible, trim it for prettiness, but not neccecarily
-	elem.className = window.jQuery ? jQuery.trim( set ) : ( set.trim ? set.trim() : set );
-}
-
 function id( name ) {
 	return !!( typeof document !== "undefined" && document && document.getElementById ) &&
 		document.getElementById( name );
@@ -1552,6 +1372,7 @@ function registerLoggingCallback( key ) {
 
 // Supports deprecated method of completely overwriting logging callbacks
 function runLoggingCallbacks( key, scope, args ) {
+	//debugger;
 	var i, callbacks;
 	if ( QUnit.hasOwnProperty( key ) ) {
 		QUnit[ key ].call(scope, args );
@@ -1593,7 +1414,6 @@ QUnit.equiv = (function() {
 
 			// for string, boolean, number and null
 			function useStrictEquality( b, a ) {
-				/*jshint eqeqeq:false */
 				if ( b instanceof a.constructor || a instanceof b.constructor ) {
 					// to catch short annotaion VS 'new' annotation of a
 					// declaration
@@ -1790,8 +1610,7 @@ QUnit.jsDump = (function() {
 
 	var reName = /^function (\w+)/,
 		jsDump = {
-			// type is used mostly internally, you can fix a (custom)type in advance
+			parse: function( obj, type, stack ) { //type is used mostly internally, you can fix a (custom)type in advance
-			parse: function( obj, type, stack ) {
 				stack = stack || [ ];
 				var inStack, res,
 					parser = this.parsers[ type || this.typeOf(obj) ];
@@ -1799,16 +1618,18 @@ QUnit.jsDump = (function() {
 				type = typeof parser;
 				inStack = inArray( obj, stack );
 
-				if ( inStack !== -1 ) {
+				if ( inStack != -1 ) {
 					return "recursion(" + (inStack - stack.length) + ")";
 				}
-				if ( type === "function" )  {
+				//else
+				if ( type == "function" )  {
 					stack.push( obj );
 					res = parser.call( this, obj, stack );
 					stack.pop();
 					return res;
 				}
-				return ( type === "string" ) ? parser : this.parsers.error;
+				// else
+				return ( type == "string" ) ? parser : this.parsers.error;
 			},
 			typeOf: function( obj ) {
 				var type;
@@ -1835,8 +1656,6 @@ QUnit.jsDump = (function() {
 					( typeof obj.length === "number" && typeof obj.item !== "undefined" && ( obj.length ? obj.item(0) === obj[0] : ( obj.item( 0 ) === null && typeof obj[0] === "undefined" ) ) )
 				) {
 					type = "array";
-				} else if ( obj.constructor === Error.prototype.constructor ) {
-					type = "error";
 				} else {
 					type = typeof obj;
 				}
@@ -1845,8 +1664,7 @@ QUnit.jsDump = (function() {
 			separator: function() {
 				return this.multiline ?	this.HTML ? "<br />" : "\n" : this.HTML ? "&nbsp;" : " ";
 			},
-			// extra can be a number, shortcut for increasing-calling-decreasing
+			indent: function( extra ) {// extra can be a number, shortcut for increasing-calling-decreasing
-			indent: function( extra ) {
 				if ( !this.multiline ) {
 					return "";
 				}
@@ -1875,16 +1693,13 @@ QUnit.jsDump = (function() {
 			parsers: {
 				window: "[Window]",
 				document: "[Document]",
-				error: function(error) {
+				error: "[ERROR]", //when no parser is found, shouldn"t happen
-					return "Error(\"" + error.message + "\")";
-				},
 				unknown: "[Unknown]",
 				"null": "null",
 				"undefined": "undefined",
 				"function": function( fn ) {
 					var ret = "function",
-						// functions never have name in IE
+						name = "name" in fn ? fn.name : (reName.exec(fn) || [])[1];//functions never have name in IE
-						name = "name" in fn ? fn.name : (reName.exec(fn) || [])[1];
 
 					if ( name ) {
 						ret += " " + name;
@@ -1900,9 +1715,13 @@ QUnit.jsDump = (function() {
 				object: function( map, stack ) {
 					var ret = [ ], keys, key, val, i;
 					QUnit.jsDump.up();
-					keys = [];
+					if ( Object.keys ) {
-					for ( key in map ) {
+						keys = Object.keys( map );
-						keys.push( key );
+					} else {
+						keys = [];
+						for ( key in map ) {
+							keys.push( key );
+						}
 					}
 					keys.sort();
 					for ( i = 0; i < keys.length; i++ ) {
@@ -1914,34 +1733,21 @@ QUnit.jsDump = (function() {
 					return join( "{", ret, "}" );
 				},
 				node: function( node ) {
-					var len, i, val,
+					var a, val,
 						open = QUnit.jsDump.HTML ? "&lt;" : "<",
 						close = QUnit.jsDump.HTML ? "&gt;" : ">",
 						tag = node.nodeName.toLowerCase(),
-						ret = open + tag,
+						ret = open + tag;
-						attrs = node.attributes;
 
-					if ( attrs ) {
+					for ( a in QUnit.jsDump.DOMAttrs ) {
-						for ( i = 0, len = attrs.length; i < len; i++ ) {
+						val = node[ QUnit.jsDump.DOMAttrs[a] ];
-							val = attrs[i].nodeValue;
+						if ( val ) {
-							// IE6 includes all attributes in .attributes, even ones not explicitly set.
+							ret += " " + a + "=" + QUnit.jsDump.parse( val, "attribute" );
-							// Those have values like undefined, null, 0, false, "" or "inherit".
-							if ( val && val !== "inherit" ) {
-								ret += " " + attrs[i].nodeName + "=" + QUnit.jsDump.parse( val, "attribute" );
-							}
 						}
 					}
-					ret += close;
+					return ret + close + open + "/" + tag + close;
-
-					// Show content of TextNode or CDATASection
-					if ( node.nodeType === 3 || node.nodeType === 4 ) {
-						ret += node.nodeValue;
-					}
-
-					return ret + open + "/" + tag + close;
 				},
-				// function calls it internally, it's the arguments part of the function
+				functionArgs: function( fn ) {//function calls it internally, it's the arguments part of the function
-				functionArgs: function( fn ) {
 					var args,
 						l = fn.length;
 
@@ -1951,34 +1757,54 @@ QUnit.jsDump = (function() {
 
 					args = new Array(l);
 					while ( l-- ) {
-						// 97 is 'a'
+						args[l] = String.fromCharCode(97+l);//97 is 'a'
-						args[l] = String.fromCharCode(97+l);
 					}
 					return " " + args.join( ", " ) + " ";
 				},
-				// object calls it internally, the key part of an item in a map
+				key: quote, //object calls it internally, the key part of an item in a map
-				key: quote,
+				functionCode: "[code]", //function calls it internally, it's the content of the function
-				// function calls it internally, it's the content of the function
+				attribute: quote, //node calls it internally, it's an html attribute value
-				functionCode: "[code]",
-				// node calls it internally, it's an html attribute value
-				attribute: quote,
 				string: quote,
 				date: quote,
-				regexp: literal,
+				regexp: literal, //regex
 				number: literal,
 				"boolean": literal
 			},
-			// if true, entities are escaped ( <, >, \t, space and \n )
+			DOMAttrs: {
-			HTML: false,
+				//attributes to dump from nodes, name=>realName
-			// indentation unit
+				id: "id",
-			indentChar: "  ",
+				name: "name",
-			// if true, items in a collection, are separated by a \n, else just a space.
+				"class": "className"
-			multiline: true
+			},
+			HTML: false,//if true, entities are escaped ( <, >, \t, space and \n )
+			indentChar: "  ",//indentation unit
+			multiline: true //if true, items in a collection, are separated by a \n, else just a space.
 		};
 
 	return jsDump;
 }());
 
+// from Sizzle.js
+function getText( elems ) {
+	var i, elem,
+		ret = "";
+
+	for ( i = 0; elems[i]; i++ ) {
+		elem = elems[i];
+
+		// Get the text from text nodes and CDATA nodes
+		if ( elem.nodeType === 3 || elem.nodeType === 4 ) {
+			ret += elem.nodeValue;
+
+		// Traverse everything else, except comment nodes
+		} else if ( elem.nodeType !== 8 ) {
+			ret += getText( elem.childNodes );
+		}
+	}
+
+	return ret;
+}
+
 // from jquery.js
 function inArray( elem, array ) {
 	if ( array.indexOf ) {
@@ -2009,14 +1835,13 @@ function inArray( elem, array ) {
  * QUnit.diff( "the quick brown fox jumped over", "the quick fox jumps over" ) == "the  quick <del>brown </del> fox <del>jumped </del><ins>jumps </ins> over"
  */
 QUnit.diff = (function() {
-	/*jshint eqeqeq:false, eqnull:true */
 	function diff( o, n ) {
 		var i,
 			ns = {},
 			os = {};
 
 		for ( i = 0; i < n.length; i++ ) {
-			if ( !hasOwn.call( ns, n[i] ) ) {
+			if ( ns[ n[i] ] == null ) {
 				ns[ n[i] ] = {
 					rows: [],
 					o: null
@@ -2026,7 +1851,7 @@ QUnit.diff = (function() {
 		}
 
 		for ( i = 0; i < o.length; i++ ) {
-			if ( !hasOwn.call( os, o[i] ) ) {
+			if ( os[ o[i] ] == null ) {
 				os[ o[i] ] = {
 					rows: [],
 					n: null
@@ -2039,7 +1864,7 @@ QUnit.diff = (function() {
 			if ( !hasOwn.call( ns, i ) ) {
 				continue;
 			}
-			if ( ns[i].rows.length === 1 && hasOwn.call( os, i ) && os[i].rows.length === 1 ) {
+			if ( ns[i].rows.length == 1 && typeof os[i] != "undefined" && os[i].rows.length == 1 ) {
 				n[ ns[i].rows[0] ] = {
 					text: n[ ns[i].rows[0] ],
 					row: os[i].rows[0]
@@ -2145,7 +1970,7 @@ QUnit.diff = (function() {
 
 // for CommonJS enviroments, export everything
 if ( typeof exports !== "undefined" ) {
-	extend( exports, QUnit );
+	extend(exports, QUnit);
 }
 
 // get at whatever the global object is, like window in browsers