Modify "Running..." to display test name. Fixes qunitjs#220

rwaldron · Mar 27, 2012 · d6908bb · d6908bb
1 parent 6ec4cc2
commit d6908bb
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 0 deletions.
diff --git a/addons/themes/nv.css b/addons/themes/nv.css
@@ -193,6 +193,9 @@
 	padding: 0.5em 0.5em 0.5em 2.5em;
 	color: #333;
 }
+#qunit-testresult .module-name {
+	font-weight: bold;
+}
 
 /** Fixture */
 

diff --git a/qunit/qunit.css b/qunit/qunit.css
@@ -220,6 +220,9 @@
 
 	border-bottom: 1px solid white;
 }
+#qunit-testresult .module-name {
+	font-weight: bold;
+}
 
 /** Fixture */
 

diff --git a/qunit/qunit.js b/qunit/qunit.js
@@ -100,6 +100,13 @@ Test.prototype = {
 	},
 	run: function() {
 		config.current = this;
+
+		var running = id("qunit-testresult");
+
+		if ( running ) {
+			running.innerHTML = "Running: <br/>" + this.name;
+		}
+
 		if ( this.async ) {
 			QUnit.stop();
 		}

diff --git a/test/test.js b/test/test.js
@@ -329,6 +329,15 @@ test("basics", function() {
 	equal( document.getElementById("qunit-fixture").innerHTML, "test markup", "automatically reset" );
 });
 
+test("running test name displayed", function() {
+	expect(2);
+
+	var displaying = document.getElementById("qunit-testresult");
+
+	ok( /running test name displayed/.test(displaying.innerHTML), "Expect test name to be found in displayed text" );
+	ok( /fixture/.test(displaying.innerHTML), "Expect module name to be found in displayed text" );
+});
+
 }
 
 module("custom assertions");