Adopt js reporters standard

Gruntfile.js

@@ -108,7 +108,10 @@ grunt.initConfig( {
 			"test/overload.html",
 			"test/regex-filter.html",
 			"test/regex-exclude-filter.html",
-			"test/string-filter.html"
+			"test/string-filter.html",
+			"test/events/global-test.html",
+			"test/events/nested-suites.html",
+			"test/events/combined.html"
 		]
 	},
 	coveralls: {
@@ -123,6 +126,9 @@ grunt.initConfig( {
 	},
 	"test-on-node": {
 		files: [
+			"test/events/global-test",
+			"test/events/nested-suites",
+			"test/events/combined",
 			"test/logs",
 			"test/main/test",
 			"test/main/assert",

build/browserstack-current-1.json

@@ -5,7 +5,8 @@
 	"test_path": [
 		"test/index.html",
 		"test/logs.html",
-		"test/seed.html"
+		"test/seed.html",
+		"test/events/combined.html"
 	],
 	"browsers": [
 		"chrome_current",

build/browserstack-legacy-1.json

@@ -5,7 +5,8 @@
 	"test_path": [
 		"test/index.html",
 		"test/logs.html",
-		"test/seed.html"
+		"test/seed.html",
+		"test/events/combined.html"
 	],
 	"browsers": [
 		"chrome_previous",

package.json

@@ -26,7 +26,9 @@
     "qunit/qunit.css",
     "LICENSE.txt"
   ],
-  "dependencies": {},
+  "dependencies": {
+    "js-reporters": "1.1.0"
+  },
   "devDependencies": {
     "async": "1.5.2",
     "babel-plugin-external-helpers": "6.8.0",
@@ -47,7 +49,8 @@
     "grunt-search": "0.1.8",
     "load-grunt-tasks": "3.4.1",
     "requirejs": "2.2.0",
-    "rollup-plugin-babel": "2.6.1"
+    "rollup-plugin-babel": "2.6.1",
+    "rollup-plugin-node-resolve": "2.0.0"
   },
   "scripts": {
     "browserstack": "sh build/run-browserstack.sh",

rollup.config.js

@@ -1,11 +1,13 @@
 /* jshint multistr:true, node:true */
 
 var babel = require( "rollup-plugin-babel" );
+var nodeResolve = require( "rollup-plugin-node-resolve" );
 
 module.exports = {
 	format: "iife",
 	exports: "none",
 	plugins: [
+		nodeResolve(),
 		babel( {
 			presets: [
 

src/core.js

@@ -8,9 +8,12 @@ import exportQUnit from "./export";
 
 import config from "./core/config";
 import { defined, extend, objectType, is, now } from "./core/utilities";
-import { registerLoggingCallbacks, runLoggingCallbacks } from "./core/logging";
+import { on, emit, registerLoggingCallbacks,
+	runLoggingCallbacks } from "./core/logging";
 import { sourceFromStacktrace } from "./core/stacktrace";
 
+import {Suite} from "js-reporters/lib/Data";
+
 const QUnit = {};
 
 var globalStartCalled = false;
@@ -41,6 +44,7 @@ extend( QUnit, {
 		}
 
 		module = createModule();
+		createSuite( module );
 
 		if ( testEnvironment && ( testEnvironment.setup || testEnvironment.teardown ) ) {
 			console.warn(
@@ -94,6 +98,30 @@ extend( QUnit, {
 			return module;
 		}
 
+		function createSuite( module ) {
+			var parentSuite;
+			var fullName;
+			var suite;
+
+			if ( module.parentModule !== null ) {
+				parentSuite = config.moduleToSuite[ module.parentModule.moduleId ];
+			} else {
+				parentSuite = config.globalSuite;
+			}
+
+			fullName = parentSuite.fullName.slice();
+			fullName.push( module.name );
+
+			suite = new Suite( module.name, fullName, [], [] );
+
+			parentSuite.childSuites.push( suite );
+			suite.parent = parentSuite;
+			suite.finishedTests = 0;
+			config.moduleToSuite[ module.moduleId ] = suite;
+
+			return suite;
+		}
+
 		function setCurrentModule( module ) {
 			config.currentModule = module;
 		}
@@ -106,6 +134,8 @@ extend( QUnit, {
 
 	only: only,
 
+	on: on,
+
 	start: function( count ) {
 		var globalStartAlreadyCalled = globalStartCalled;
 
@@ -224,6 +254,8 @@ export function begin() {
 			totalTests: Test.count,
 			modules: modulesLog
 		} );
+
+		emit( "runStart", config.globalSuite );
 	}
 
 	config.blocking = false;
@@ -272,6 +304,13 @@ function done() {
 			total: config.moduleStats.all,
 			runtime: now() - config.moduleStats.started
 		} );
+
+		// Do not emit the "suiteEnd" event for the globalSuite.
+		if ( config.previousModule.moduleId ) {
+			var suite = config.moduleToSuite[ config.previousModule.moduleId ];
+
+			emit( "suiteEnd", suite );
+		}
 	}
 	delete config.previousModule;
 
@@ -284,6 +323,8 @@ function done() {
 		total: config.stats.all,
 		runtime: runtime
 	} );
+
+	emit( "runEnd", config.globalSuite );
 }
 
 function setHook( module, hookName ) {

src/core/config.js

@@ -1,3 +1,5 @@
+import {Suite} from "js-reporters/lib/Data";
+
 /**
  * Config object: Maintain internal state
  * Later exposed as QUnit.config
@@ -46,7 +48,11 @@ const config = {
 		tests: []
 	},
 
-	callbacks: {}
+	callbacks: {},
+
+	globalSuite: new Suite( undefined, [], [], [] ),
+
+	moduleToSuite: {}
 };
 
 // Push a loose unnamed module to the modules collection

src/core/logging.js

@@ -1,5 +1,7 @@
 import config from "./config";
-import { objectType } from "./utilities";
+import { objectType, inArray } from "./utilities";
+
+var listeners = {};
 
 // Register logging callbacks
 export function registerLoggingCallbacks( obj ) {
@@ -41,3 +43,40 @@ export function runLoggingCallbacks( key, args ) {
 		callbacks[ i ]( args );
 	}
 }
+
+export function emit( type, data ) {
+	var i, callbacks;
+
+	// Validate
+	if ( objectType( type ) !== "string" ) {
+		throw new Error( "Emitting QUnit events requires an event type" );
+	}
+
+	// Ensure a consistent event run
+	callbacks = [].slice.call( listeners[ type ] || [] );
+	for ( i = 0; i < callbacks.length; i++ ) {
+		callbacks[ i ]( data );
+	}
+}
+
+export function on( type, listener ) {
+
+	// Validate
+	if ( objectType( type ) !== "string" ) {
+		throw new Error( "Adding QUnit events requires an event type" );
+	}
+
+	if ( objectType( listener ) !== "function" ) {
+		throw new Error( "Adding QUnit events requires a listener function" );
+	}
+
+	// Initialize collection of this logging callback
+	if ( !listeners[ type ] ) {
+		listeners[ type ] = [];
+	}
+
+	// Filter out duplicate listeners
+	if ( inArray( listener, listeners[ type ] ) < 0 ) {
+		listeners[ type ].push( listener );
+	}
+}