minireporter now works well in everything except on OSX Terminal.app

Author: jamestalmage

lib/reporters/mini.js

@@ -1,7 +1,9 @@
 'use strict';
-var logUpdate = require('log-update');
-var colors = require('../colors');
+var ansiEscapes = require('ansi-escapes');
+var cliCursor = require('cli-cursor');
+var lastLineStream = require('last-line-stream');
 var plur = require('plur');
+var colors = require('../colors');
 var beautifyStack = require('../beautify-stack');
 
 function MiniReporter() {
@@ -14,8 +16,9 @@ function MiniReporter() {
 	this.skipCount = 0;
 	this.rejectionCount = 0;
 	this.exceptionCount = 0;
-	this.finished = false;
 	this.lastWritten = undefined;
+	this.prevLineCount = 0;
+	this.stream = lastLineStream(process.stderr);
 }
 
 module.exports = MiniReporter;
@@ -62,8 +65,6 @@ MiniReporter.prototype.unhandledError = function (err) {
 };
 
 MiniReporter.prototype.finish = function () {
-	this.finished = true;
-
 	var status = '\n';
 
 	if (this.passCount > 0) {
@@ -135,24 +136,27 @@ MiniReporter.prototype.finish = function () {
 };
 
 MiniReporter.prototype.write = function (str) {
-	logUpdate.stderr(str);
-	this.lastWritten = str;
-
-	if (this.finished) {
-		logUpdate.stderr.done();
-	}
+	cliCursor.hide();
+	this.lastWritten = str + '\n';
+	this.stream.push(this._clear() + this.lastWritten);
+	this.prevLineCount = str.split('\n').length;
 };
 
 MiniReporter.prototype.stdout = MiniReporter.prototype.stderr = function (data) {
-	var rewrite = this.lastWritten !== undefined && !this.finished;
+	this.stream.push(this._clear());
+	this.stream.write(data);
+	this.stream.push(this.lastWritten);
+};
 
-	if (rewrite) {
-		logUpdate.stderr.clear();
+MiniReporter.prototype._clear = function () {
+	var ct = this.prevLineCount;
+	if (ct === 0) {
+		return '';
 	}
-
-	process.stderr.write(data);
-
-	if (rewrite) {
-		logUpdate.stderr(this.lastWritten);
+	var lastLine = this.stream.lastLine;
+	var columns = process.stdout.columns;
+	while (lastLine.length > columns) {
+		lastLine = lastLine.substr(columns);
 	}
+	return ansiEscapes.eraseLines(ct + 1) + lastLine;
 };

package.json

@@ -75,6 +75,7 @@
     "tap"
   ],
   "dependencies": {
+    "ansi-escapes": "^1.1.1",
     "arr-flatten": "^1.0.1",
     "arrify": "^1.0.0",
     "ava-init": "^0.1.0",
@@ -87,6 +88,7 @@
     "bluebird": "^3.0.0",
     "caching-transform": "^1.0.0",
     "chalk": "^1.0.0",
+    "cli-cursor": "^1.0.2",
     "co-with-promise": "^4.6.0",
     "commondir": "^1.0.1",
     "convert-source-map": "^1.1.2",
@@ -101,7 +103,7 @@
     "is-generator-fn": "^1.0.0",
     "is-observable": "^0.1.0",
     "is-promise": "^2.1.0",
-    "log-update": "^1.0.2",
+    "last-line-stream": "^0.1.0",
     "loud-rejection": "^1.2.0",
     "max-timeout": "^1.0.0",
     "md5-hex": "^1.2.0",

test/reporters/mini.js

@@ -3,6 +3,8 @@ var chalk = require('chalk');
 var test = require('tap').test;
 var miniReporter = require('../../lib/reporters/mini');
 
+process.stderr.setMaxListeners(50);
+
 test('start', function (t) {
 	var reporter = miniReporter();
 

test/visual/console-log.js

@@ -3,15 +3,15 @@ import delay from 'delay';
 
 test('testName1', async t => {
 	console.log('foo');
-	await delay(200);
+	await delay(2000);
 	console.log('baz');
 	t.pass();
 });
 
 test('testName2', async t => {
-	await delay(100);
+	await delay(1000);
 	console.log('bar');
-	await delay(200);
+	await delay(2000);
 	console.log('quz');
 	t.pass();
 });

test/visual/lorem-ipsum.js

@@ -0,0 +1,45 @@
+'use strict';
+
+var fs = require('fs');
+var path = require('path');
+var text = fs.readFileSync(path.join(__dirname, 'lorem-ipsum.txt'), 'utf8');
+var test = require('../../');
+var delay = require('delay');
+
+var lines = text.split(/\r?\n/g).map(function (line) {
+	return line.split(' ');
+});
+
+setTimeout(function () {
+	var lineNum = 0;
+	var wordNum = 0;
+
+	var interval = setInterval(function () {
+		if (lineNum >= lines.length) {
+			clearInterval(interval);
+			return;
+		}
+		var line = lines[lineNum];
+		if (wordNum >= line.length) {
+			process.stdout.write('\n');
+			lineNum++;
+			wordNum = 0;
+			return;
+		}
+		var word = line[wordNum];
+		wordNum++;
+		if (wordNum < line.length) {
+			word += ' ';
+		}
+		process.stdout.write(word);
+	}, 50);
+}, 200);
+
+async function testFn(t) {
+	await delay(40);
+	t.pass();
+}
+
+for (var i = 0; i < 250; i++) {
+	test.serial('test number ' + i, testFn);
+}

test/visual/lorem-ipsum.txt

@@ -0,0 +1,9 @@
+Lorem ipsum dolor sit amet, est ut alia vitae, commune appetere fabellas et sea, id diam error iisque eum. Has legendos adipiscing ad. Mea everti audiam te. Ne est legendos qualisque, id fabulas nominati eos, vim an magna praesent. Cum eu eius conclusionemque, paulo iudico populo vix eu, dicunt aperiri abhorreant ad est.
+
+Suas constituam sit ea, populo moderatius eam ei. Ei melius nonumes invidunt vim, duo minim veniam cu. Omnis maiorum nam no, eu his choro dolorem disputando. Vel te lorem mandamus, sed in prompta electram. Nam id omnium mandamus erroribus, ut eos duis etiam perfecto.
+
+Qui viris euripidis eloquentiam ut. Eam vide disputando ei, pro no partiendo iracundia, ne sed vocibus graecis. Mel id saepe iracundia philosophia, oratio rationibus te vel. Ad delicata assentior vel, cu sit dicit deseruisse, ne aliquip propriae concludaturque est.
+
+Ipsum munere neglegentur an est. Mundi aliquid delenit ei duo, duis fabellas reformidans pro te. An soluta copiosae indoctum cum. Ut his menandri suavitate.
+
+Alia laoreet pri id. Per solum lorem percipit te, eu diam modus iusto vel, id pro primis noster quaestio. Quot choro lucilius cum no. Nobis alterum habemus duo no. Scripta regione eloquentiam ad eam, ad sonet sententiae pro.

test/visual/stdout-write.js

@@ -3,15 +3,21 @@ import delay from 'delay';
 
 test('testName1', async t => {
 	process.stdout.write('foo ');
-	await delay(200);
+	await delay(2000);
 	process.stdout.write('baz ');
 	t.pass();
 });
 
 test('testName2', async t => {
-	await delay(100);
+	await delay(1000);
 	process.stdout.write('bar ');
-	await delay(200);
+	await delay(2000);
 	process.stdout.write('quz ');
+	await delay(1000);
+	t.pass();
+});
+
+test('testName3', async t => {
+	await delay(5000);
 	t.pass();
 });