fixup! build error message with error.name; share reportError message

Author: timmywil

listeners.js

@@ -87,7 +87,11 @@
 		send( "ack" );
 
 		QUnit.on( "error", function( error ) {
-			send( "error", { message: error.message, stack: error.stack } );
+			send( "error", {
+				message: error.message,
+				name: error.name,
+				stack: error.stack
+			} );
 		} );
 
 		QUnit.on( "testEnd", function( data ) {

reporter.js

@@ -114,9 +114,14 @@ export function reportTest( test, { fullBrowser, id } ) {
 }
 
 export function reportError( error ) {
-	console.error( chalk.red( `\n\nError: ${ error.message }` ) );
-	console.error( chalk.gray( error.stack ) );
-	return error;
+	const title = `${ error.name || "Error" }: ${ error.message }`;
+	let message = chalk.red( title );
+
+	// Chromium error stacks include the title in the first line,
+	// but Firefox error stacks do not.
+	message += `\n${ chalk.gray( error.stack.replace( `${ title }\n`, "" ) ) }`;
+	console.error( `\n\n${ message }` );
+	return message;
 }
 
 export function reportEnd( result, { descriptiveUrl, fullBrowser, id } ) {

run.js

@@ -110,10 +110,9 @@ export async function run( {
 					const reportId = message.id;
 					const report = reports[ reportId ];
 					touchBrowser( report.browser );
-					reportError( message.data );
+					const errorMessage = reportError( message.data );
 					pendingErrors[ reportId ] ??= Object.create( null );
-					pendingErrors[ reportId ][ message.data.message ] =
-						`${ message.data.message }\n${ message.data.stack }`;
+					pendingErrors[ reportId ][ message.data.message ] = errorMessage;
 					break;
 				}
 				case "runEnd": {