CLI: Fix TAP compliance for actual/expected indent and skip/todo colors

Cherry-picked from adc4493 (3.0.0-dev).

Author: Krinkle

.github/workflows/coverage.yaml

@@ -9,6 +9,7 @@ jobs:
     runs-on: ubuntu-20.04
     env:
       PUPPETEER_DOWNLOAD_PATH: "${{ github.workspace }}/.puppeteer_download"
+      FORCE_COLOR: "1"
     steps:
       - uses: actions/checkout@v4
 

package-lock.json

@@ -80,7 +80,8 @@
     "requirejs": "^2.3.6",
     "rimraf": "^3.0.2",
     "rollup": "^2.79.1",
-    "semver": "^7.6.2"
+    "semver": "^7.6.2",
+    "tap-parser": "11.0.2"
   },
   "scripts": {
     "build": "rollup -c && grunt copy:src-css",

package.json

@@ -35,7 +35,7 @@ import { annotateStacktrace } from '../core/stacktrace';
  * Objects with cyclical references will be stringifed as
  * "[Circular]" as they cannot otherwise be represented.
  */
-function prettyYamlValue (value, indent = 4) {
+function prettyYamlValue (value, indent = 2) {
   if (value === undefined) {
     // Not supported in JSON/YAML, turn into string
     // and let the below output it as bare string.
@@ -94,7 +94,7 @@ function prettyYamlValue (value, indent = 4) {
 
       // See also <https://yaml-multiline.info/>
       // Support IE 9-11: Avoid ES6 String#repeat
-      const prefix = (new Array(indent + 1)).join(' ');
+      const prefix = (new Array((indent * 2) + 1)).join(' ');
 
       const trailingLinebreakMatch = value.match(/\n+$/);
       const trailingLinebreaks = trailingLinebreakMatch
@@ -126,8 +126,13 @@ function prettyYamlValue (value, indent = 4) {
     }
   }
 
+  const prefix = (new Array(indent + 1)).join(' ');
+
   // Handle null, boolean, array, and object
-  return JSON.stringify(decycledShallowClone(value), null, 2);
+  return JSON.stringify(decycledShallowClone(value), null, 2)
+    .split('\n')
+    .map((line, i) => i === 0 ? line : prefix + line)
+    .join('\n');
 }
 
 /**
@@ -223,11 +228,11 @@ export default class TapReporter {
       this.log(`ok ${this.testCount} ${test.fullName.join(' > ')}`);
     } else if (test.status === 'skipped') {
       this.log(
-        `ok ${this.testCount} ${kleur.yellow(`# SKIP ${test.fullName.join(' > ')}`)}`
+        `ok ${this.testCount} ${kleur.yellow(test.fullName.join(' > '))} # SKIP`
       );
     } else if (test.status === 'todo') {
       this.log(
-        `not ok ${this.testCount} ${kleur.cyan(`# TODO ${test.fullName.join(' > ')}`)}`
+        `not ok ${this.testCount} ${kleur.cyan(test.fullName.join(' > '))} # TODO`
       );
       test.errors.forEach((error) => this.logAssertion(error, 'todo'));
     } else {