updated test-success, convert buffer to string, added fix script

Author: BboyAkers

.github/workflows/release-please.yml

@@ -21,7 +21,6 @@ jobs:
       - uses: actions/setup-node@v3
         with:
           node-version: 16
-          registry-url: 'https://external-dot-oss-automation.appspot.com'
           cache: npm
         if: ${{ steps.release.outputs.release_created }}
       - run: npm publish

package-lock.json

@@ -8,6 +8,7 @@
     "pretest": "npm run lint && npm run clean && npm run instrument",
     "test": "tap",
     "snap": "npm test -- --snapshot",
+    "fix": "standard --fix",
     "posttest": "npm run report",
     "clean": "node ./npm-run-clean.js",
     "instrument": "node ./build-self-coverage.js",

package.json

@@ -888,7 +888,7 @@ exports[`test/nyc-integration.js > TAP > passes configuration via environment va
 `
 
 exports[`test/nyc-integration.js > TAP > recursive run does not throw > stdout 1`] = `
-
+Buffer <2268 656c 6c6f 220d 0a  "hello"..>
 `
 
 exports[`test/nyc-integration.js > TAP > report and check should show coverage check along with report > stderr 1`] = `

tap-snapshots/test/nyc-integration.js.test.cjs

@@ -13,7 +13,7 @@ if ('APPDATA' in process.env) {
   envPath.APPDATA = process.env.APPDATA
 }
 
-function sanitizeString (str, cwd, leavePathSep) {
+function sanitizeString(str, cwd, leavePathSep) {
   /*
    * File paths are different on different systems:
    *   - make everything relative to cwd
@@ -31,7 +31,7 @@ function sanitizeString (str, cwd, leavePathSep) {
   return str
 }
 
-async function runNYC ({ args, tempDir, leavePathSep, cwd = fixturesCLI, env = {} }) {
+async function runNYC({ args, tempDir, leavePathSep, cwd = fixturesCLI, env = {} }) {
   const runArgs = [nycBin].concat(tempDir ? ['--temp-dir', tempDir] : [], args)
   const { status, stderr, stdout } = await spawn(process.execPath, runArgs, {
     cwd: cwd,
@@ -47,8 +47,8 @@ async function runNYC ({ args, tempDir, leavePathSep, cwd = fixturesCLI, env = {
       stderr,
       stdout
     },
-    stderr: sanitizeString(stderr, cwd, leavePathSep),
-    stdout: sanitizeString(stdout, cwd, leavePathSep)
+    stderr: sanitizeString(stderr.toString('utf8'), cwd, leavePathSep),
+    stdout: sanitizeString(stdout.toString('utf8'), cwd, leavePathSep)
   }
 }
 

test/helpers/run-nyc.js

@@ -2,15 +2,14 @@
 
 const runNYC = require('./run-nyc')
 
-async function testSuccess (t, opts) {
+async function testSuccess(t, opts) {
   const { status, stderr, stdout } = await runNYC({
     tempDir: t.tempDir,
     ...opts
   })
 
   t.equal(status, 0)
   t.equal(stderr, '')
-  console.info(stdout)
   t.matchSnapshot(stdout, 'stdout')
 }
 

test/helpers/test-success.js

@@ -591,7 +591,8 @@ t.test('recursive run does not throw', t => testSuccess(t, {
     nycBin,
     process.execPath,
     nycBin,
-    'true'
+    'echo',
+    'hello'
   ],
   cwd: path.resolve(__dirname, 'fixtures/recursive-run')
 }))