Fix handling of errors with empty stack trace (#19)

Co-authored-by: Sindre Sorhus <[email protected]>

Author: tommie

index.js

@@ -29,7 +29,7 @@ export default class AggregateError extends Error {
 		let message = errors
 			.map(error => {
 				// The `stack` property is not standardized, so we can't assume it exists
-				return typeof error.stack === 'string' ? cleanInternalStack(cleanStack(error.stack)) : String(error);
+				return typeof error.stack === 'string' && error.stack.length > 0 ? cleanInternalStack(cleanStack(error.stack)) : String(error);
 			})
 			.join('\n');
 		message = '\n' + indentString(message, 4);

test.js

@@ -51,3 +51,28 @@ test('gracefully handle Error instances without a stack', t => {
 		new StacklessError('stackless')
 	]);
 });
+
+test('gracefully handle Error instances with empty stack', t => {
+	class EmptyStackError extends Error {
+		constructor(...args) {
+			super(...args);
+			this.name = this.constructor.name;
+			this.stack = '';
+		}
+	}
+
+	const error = new AggregateError([
+		new Error('foo'),
+		new EmptyStackError('emptystack')
+	]);
+
+	console.log(error);
+
+	t.regex(error.message, /Error: foo\n {8}at /);
+	t.regex(error.message, /EmptyStackError: emptystack/);
+
+	t.deepEqual([...error.errors], [
+		new Error('foo'),
+		new EmptyStackError('emptystack')
+	]);
+});