minor style tweaks

Author: sindresorhus

lib/logger.js

@@ -44,10 +44,6 @@ x.test = function (props) {
 		return;
 	}
 
-	// if (runner.stats.testCount === 1) {
-	// 	return;
-	// }
-
 	// display duration only over a threshold
 	var threshold = 100;
 	var dur = props.duration > threshold ? chalk.gray.dim(' (' + prettyMs(props.duration) + ')') : '';
@@ -76,9 +72,11 @@ x.report = function (passed, failed, unhandled, uncaught) {
 	} else {
 		log.writelpad(chalk.green(passed, plur('test', passed), 'passed'));
 	}
+
 	if (unhandled > 0) {
 		log.writelpad(chalk.red(unhandled, 'unhandled', plur('rejection', unhandled)));
 	}
+
 	if (uncaught > 0) {
 		log.writelpad(chalk.red(uncaught, 'uncaught', plur('exception', uncaught)));
 	}
@@ -88,23 +86,28 @@ x.unhandledRejections = function (file, rejections) {
 	if (!(rejections && rejections.length)) {
 		return;
 	}
+
 	rejections.forEach(function (rejection) {
 		log.write(chalk.red('Unhandled Rejection: ', file));
+
 		if (rejection.stack) {
 			log.writelpad(chalk.red(beautifyStack(rejection.stack)));
 		} else {
 			log.writelpad(chalk.red(JSON.stringify(rejection)));
 		}
+
 		log.write();
 	});
 };
 
 x.uncaughtException = function (file, error) {
 	log.write(chalk.red('Uncaught Exception: ', file));
+
 	if (error.stack) {
 		log.writelpad(chalk.red(beautifyStack(error.stack)));
 	} else {
 		log.writelpad(chalk.red(JSON.stringify(error)));
 	}
+
 	log.write();
 };

lib/serialize-value.js

@@ -2,15 +2,16 @@
 var destroyCircular = require('destroy-circular');
 
 // Make a value ready for JSON.stringify() / process.send()
-
 module.exports = function serializeValue(value) {
 	if (typeof value === 'object') {
 		return destroyCircular(value);
 	}
+
 	if (typeof value === 'function') {
 		// JSON.stringify discards functions, leaving no context information once we serialize and send across.
 		// We replace thrown functions with a string to provide as much information to the user as possible.
 		return '[Function: ' + (value.name || 'anonymous') + ']';
 	}
+
 	return value;
 };

lib/symbol.js

@@ -14,5 +14,6 @@ module.exports = function (key) {
 	} else if (typeof Symbol !== 'undefined' && typeof Symbol.for === 'function') {
 		return Symbol.for(key);
 	}
+
 	return '@@' + key;
 };

lib/test.js

@@ -102,7 +102,7 @@ Test.prototype.plan = function (count) {
 Test.prototype.run = function () {
 	this.promise = {};
 
-	// TODO: refactor this to avoid storing the promise
+	// TODO(vdemedes): refactor this to avoid storing the promise
 	return new Promise(function (resolve, reject) {
 		this.promise.resolve = resolve;
 		this.promise.reject = reject;

test/test.js

@@ -1213,7 +1213,7 @@ test('uncaught exception will be reported to console', function (t) {
 	execCli('fixture/uncaught-exception.js', function (err, stdout, stderr) {
 		t.ok(err);
 		t.ok(/Can't catch me!/.test(stderr));
-		// TODO: This should get printed, but we reject the promise (ending all tests) instead of just ending that one test and reporting.
+		// TODO(jamestalmage): This should get printed, but we reject the promise (ending all tests) instead of just ending that one test and reporting.
 		// t.ok(/1 uncaught exception[^s]/.test(stdout));
 		t.end();
 	});