Core: Implement a node stdout reporter

Gruntfile.js

@@ -170,56 +170,13 @@ grunt.registerTask( "testswarm", function( commit, configFile, projectName, brow
 		});
 });
 
-// TODO: Extract this task later, if feasible
-// Also spawn a separate process to keep tests atomic
 grunt.registerTask( "test-on-node", function() {
-	var testActive = false,
-		runDone = false,
-		done = this.async(),
-		QUnit = require( "./dist/qunit" );
+	var done = this.async();
+	var reporterDone = require( "./test/reporter-stdout.js" );
 
-	QUnit.testStart(function() {
-		testActive = true;
+	reporterDone(function( details ) {
+		done( details.failed === 0 );
 	});
-	QUnit.log(function( details ) {
-		if ( !testActive || details.result ) {
-			return;
-		}
-		var message = "name: " + details.name + " module: " + details.module +
-			" message: " + details.message;
-		grunt.log.error( message );
-	});
-	QUnit.testDone(function() {
-		testActive = false;
-	});
-	QUnit.done(function( details ) {
-		if ( runDone ) {
-			return;
-		}
-		var succeeded = ( details.failed === 0 ),
-			message = details.total + " assertions in (" + details.runtime + "ms), passed: " +
-				details.passed + ", failed: " + details.failed;
-		if ( succeeded ) {
-			grunt.log.ok( message );
-		} else {
-			grunt.log.error( message );
-		}
-		done( succeeded );
-		runDone = true;
-	});
-	QUnit.config.autorun = false;
-
-	require( "./test/logs" );
-	require( "./test/test" );
-	require( "./test/assert" );
-	require( "./test/async" );
-	require( "./test/promise" );
-	require( "./test/modules" );
-	require( "./test/deepEqual" );
-	require( "./test/globals" );
-	require( "./test/globals-node" );
-
-	QUnit.load();
 });
 
 grunt.registerTask( "build", [ "concat" ] );

package.json

@@ -31,7 +31,9 @@
     "qunit/qunit.css",
     "LICENSE.txt"
   ],
-  "dependencies": {},
+  "dependencies": {
+    "chalk": "1.0.0"
+  },
   "devDependencies": {
     "browserstack-runner": "0.2.2",
     "commitplease": "2.0.0",
@@ -45,6 +47,7 @@
     "grunt-qunit-istanbul": "0.4.5",
     "grunt-search": "0.1.6",
     "load-grunt-tasks": "0.3.0",
+    "qunit-reporter-stdout": "0.1.0",
     "testswarm": "1.1.0"
   },
   "scripts": {

src/export.js

@@ -51,10 +51,21 @@ if ( typeof window !== "undefined" ) {
 
 // For nodejs
 if ( typeof module !== "undefined" && module && module.exports ) {
+	/*jshint node:true*/
+
+	// Load node reporter
+	QUnit.stdout = function( options ) {
+		var stdout = require( "qunit-reporter-stdout" );
+
+		return stdout( QUnit, options );
+	};
+
 	module.exports = QUnit;
 
 	// For consistency with CommonJS environments' exports
 	module.exports.QUnit = QUnit;
+
+	/*jshint node:false*/
 }
 
 // For CommonJS with exports, but without module.exports, like Rhino

test/reporter-stdout.js

@@ -0,0 +1,21 @@
+/*globals QUnit:true*/
+/*jshint node:true*/
+var QUnit = require( "../dist/qunit" );
+
+// Options: { output: "minimal" || "verbose" }
+QUnit.stdout();
+
+// Load QUnit tests
+require( "./logs" );
+require( "./test" );
+require( "./assert" );
+require( "./async" );
+require( "./promise" );
+require( "./modules" );
+require( "./deepEqual" );
+require( "./globals" );
+require( "./globals-node" );
+
+module.exports = QUnit.done;
+
+QUnit.load();