Fix phantomjs test runner with requirejs

Summary:
The problem is QUnit is loaded before everything else, so it fires the
QUnit.done event before any of the tests have actually ran.

This kind of hacky fix just has the test runner ignore a QUnit.done
event that didn't actually run any tests.

It's kind of ugly but I am not sure if there's a simpler solution. I
also tried moving QUnit into a require in the last callback, but that
caused the test runner to complain that QUnit wasn't present.

Test Plan: * `phantomjs test/runner.js test/index.html`

Reviewers: jack, alpert

Reviewed By: alpert

CC: alpert

Differential Revision: http://phabricator.khanacademy.org/D7217

Author: Justin Jaffray

test/runner.js

@@ -176,6 +176,12 @@
 			});
 
 			QUnit.done(function ( result ) {
+				// Because the tests are loaded asynchronously, QUnit
+				// originally thinks that it's done because none of them have
+				// loaded yet.
+				if ( result.total === 0 ) {
+					return
+				}
 				console.log( "---------------" );
 				console.log( "Took " + result.runtime +  "ms to run " + result.total + " tests. " + ANSI.colorizeText( result.passed + " passed", "green" ) + ", " + ANSI.colorizeText( result.failed + " failed", "red" ) + "." );
 				console.log( "---------------" );
@@ -189,4 +195,4 @@
 			});
 		}, false );
 	}
-})();
+})();