test_runner: differentiate todo and failure styles

PR-URL: #61564
Reviewed-By: Jacob Smith <jacob@frende.me>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>

Author: MoLow

lib/internal/test_runner/reporter/utils.js

@@ -24,6 +24,7 @@ const reporterUnicodeSymbolMap = {
 'test:coverage': '\u2139 ',
 'arrow:right': '\u25B6 ',
 'hyphen:minus': '\uFE63 ',
+'warning:alert': '\u26A0 ',
 };
 
 constreporterColorMap={
@@ -76,18 +77,20 @@ function formatTestReport(type, data, showErrorDetails = true, prefix = '', inde
 
 if(skip!==undefined){
 title+=` # ${typeofskip==='string'&&skip.length ? skip : 'SKIP'}`;
+color=colors.gray;
+symbol=reporterUnicodeSymbolMap['hyphen:minus'];
 }elseif(todo!==undefined){
 title+=` # ${typeoftodo==='string'&&todo.length ? todo : 'TODO'}`;
+if(type==='test:fail'){
+color=colors.yellow;
+symbol=reporterUnicodeSymbolMap['warning:alert'];
+}
 }elseif(expectFailure!==undefined){
 title+=` # EXPECTED FAILURE`;
 }
 
 consterr=showErrorDetails&&data.details?.error ? formatError(data.details.error,indent) : '';
 
-if(skip!==undefined){
-color=colors.gray;
-symbol=reporterUnicodeSymbolMap['hyphen:minus'];
-}
 return`${prefix}${indent}${color}${symbol}${title}${colors.white}${err}`;
 }
 

test/fixtures/test-runner/output/dot_reporter.snapshot

@@ -6,7 +6,7 @@ X
 
 Failed tests:
 
-✖ sync fail todo (*ms) # TODO
+⚠ sync fail todo (*ms) # TODO
  Error: thrown from sync fail todo
  *
  *
@@ -15,7 +15,7 @@ Failed tests:
  *
  *
  *
-✖ sync fail todo with message (*ms) # this is a failing todo
+⚠ sync fail todo with message (*ms) # this is a failing todo
  Error: thrown from sync fail todo with message
  *
  *