@zuplo/cli 7.0.2 → 7.1.0

package/node_modules/pino-pretty/test/cli-rc.test.js

@@ -2,23 +2,24 @@
 
 process.env.TZ = 'UTC'
 
-const path = require('path')
-const spawn = require('child_process').spawn
-const test = require('tap').test
-const fs = require('fs')
-const rimraf = require('rimraf')
+const path = require('node:path')
+const { spawn } = require('node:child_process')
+const { describe, after, test } = require('node:test')
+const match = require('@jsumners/assert-match')
+const fs = require('node:fs')
+const { rimraf } = require('rimraf')
+const { once } = require('./helper')
 
 const bin = require.resolve('../bin')
 const logLine = '{"level":30,"time":1522431328992,"msg":"hello world","pid":42,"hostname":"foo"}\n'
-const noop = () => {}
 
-test('cli', (t) => {
+describe('cli', () => {
   const tmpDir = path.join(__dirname, '.tmp_' + Date.now())
   fs.mkdirSync(tmpDir)
 
-  t.teardown(() => rimraf(tmpDir, noop))
+  after(() => rimraf(tmpDir))
 
-  t.test('loads and applies default config file: pino-pretty.config.js', (t) => {
+  test('loads and applies default config file: pino-pretty.config.js', async (t) => {
     t.plan(1)
     // Set translateTime: true on run configuration
     const configFile = path.join(tmpDir, 'pino-pretty.config.js')
@@ -26,18 +27,19 @@ test('cli', (t) => {
     const env = { TERM: 'dumb', TZ: 'UTC' }
     const child = spawn(process.argv[0], [bin], { env, cwd: tmpDir })
     // Validate that the time has been translated
-    child.on('error', t.threw)
-    child.stdout.on('data', (data) => {
-      t.equal(data.toString(), '[17:35:28.992] INFO (42): hello world\n')
+    child.on('error', t.assert.fail)
+    const endPromise = once(child.stdout, 'data', (data) => {
+      t.assert.strictEqual(data.toString(), '[17:35:28.992] INFO (42): hello world\n')
     })
     child.stdin.write(logLine)
-    t.teardown(() => {
+    await endPromise
+    t.after(() => {
       fs.unlinkSync(configFile)
       child.kill()
     })
   })
 
-  t.test('loads and applies default config file: pino-pretty.config.cjs', (t) => {
+  test('loads and applies default config file: pino-pretty.config.cjs', async (t) => {
     t.plan(1)
     // Set translateTime: true on run configuration
     const configFile = path.join(tmpDir, 'pino-pretty.config.cjs')
@@ -48,19 +50,20 @@ test('cli', (t) => {
     const env = { TERM: ' dumb', TZ: 'UTC' }
     const child = spawn(process.argv[0], [bin], { env, cwd: tmpDir })
     // Validate that the time has been translated
-    child.on('error', t.threw)
-    child.stdout.on('data', (data) => {
-      t.equal(data.toString(), '[17:35:28.992] INFO (42): hello world\n')
+    child.on('error', t.assert.fail)
+    const endPromise = once(child.stdout, 'data', (data) => {
+      t.assert.strictEqual(data.toString(), '[17:35:28.992] INFO (42): hello world\n')
     })
     child.stdin.write(logLine)
-    t.teardown(() => {
+    await endPromise
+    t.after(() => {
       fs.unlinkSync(configFile)
       fs.unlinkSync(packageJsonFile)
       child.kill()
     })
   })
 
-  t.test('loads and applies default config file: .pino-prettyrc', (t) => {
+  test('loads and applies default config file: .pino-prettyrc', async (t) => {
     t.plan(1)
     // Set translateTime: true on run configuration
     const configFile = path.join(tmpDir, '.pino-prettyrc')
@@ -68,18 +71,19 @@ test('cli', (t) => {
     const env = { TERM: ' dumb', TZ: 'UTC' }
     const child = spawn(process.argv[0], [bin], { env, cwd: tmpDir })
     // Validate that the time has been translated
-    child.on('error', t.threw)
-    child.stdout.on('data', (data) => {
-      t.equal(data.toString(), '[17:35:28.992] INFO (42): hello world\n')
+    child.on('error', t.assert.fail)
+    const endPromise = once(child.stdout, 'data', (data) => {
+      t.assert.strictEqual(data.toString(), '[17:35:28.992] INFO (42): hello world\n')
     })
     child.stdin.write(logLine)
-    t.teardown(() => {
+    await endPromise
+    t.after(() => {
       fs.unlinkSync(configFile)
       child.kill()
     })
   })
 
-  t.test('loads and applies default config file: .pino-prettyrc.json', (t) => {
+  test('loads and applies default config file: .pino-prettyrc.json', async (t) => {
     t.plan(1)
     // Set translateTime: true on run configuration
     const configFile = path.join(tmpDir, '.pino-prettyrc.json')
@@ -87,18 +91,19 @@ test('cli', (t) => {
     const env = { TERM: ' dumb', TZ: 'UTC' }
     const child = spawn(process.argv[0], [bin], { env, cwd: tmpDir })
     // Validate that the time has been translated
-    child.on('error', t.threw)
-    child.stdout.on('data', (data) => {
-      t.equal(data.toString(), '[17:35:28.992] INFO (42): hello world\n')
+    child.on('error', t.assert.fail)
+    const endPromise = once(child.stdout, 'data', (data) => {
+      t.assert.strictEqual(data.toString(), '[17:35:28.992] INFO (42): hello world\n')
     })
     child.stdin.write(logLine)
-    t.teardown(() => {
+    await endPromise
+    t.after(() => {
       fs.unlinkSync(configFile)
       child.kill()
     })
   })
 
-  t.test('loads and applies custom config file: pino-pretty.config.test.json', (t) => {
+  test('loads and applies custom config file: pino-pretty.config.test.json', async (t) => {
     t.plan(1)
     // Set translateTime: true on run configuration
     const configFile = path.join(tmpDir, 'pino-pretty.config.test.json')
@@ -106,15 +111,16 @@ test('cli', (t) => {
     const env = { TERM: ' dumb', TZ: 'UTC' }
     const child = spawn(process.argv[0], [bin, '--config', configFile], { env, cwd: tmpDir })
     // Validate that the time has been translated
-    child.on('error', t.threw)
-    child.stdout.on('data', (data) => {
-      t.equal(data.toString(), '[17:35:28.992] INFO (42): hello world\n')
+    child.on('error', t.assert.fail)
+    const endPromise = once(child.stdout, 'data', (data) => {
+      t.assert.strictEqual(data.toString(), '[17:35:28.992] INFO (42): hello world\n')
     })
     child.stdin.write(logLine)
-    t.teardown(() => child.kill())
+    await endPromise
+    t.after(() => child.kill())
   })
 
-  t.test('loads and applies custom config file: pino-pretty.config.test.js', (t) => {
+  test('loads and applies custom config file: pino-pretty.config.test.js', async (t) => {
     t.plan(1)
     // Set translateTime: true on run configuration
     const configFile = path.join(tmpDir, 'pino-pretty.config.test.js')
@@ -122,16 +128,17 @@ test('cli', (t) => {
     const env = { TERM: ' dumb', TZ: 'UTC' }
     const child = spawn(process.argv[0], [bin, '--config', configFile], { env, cwd: tmpDir })
     // Validate that the time has been translated
-    child.on('error', t.threw)
-    child.stdout.on('data', (data) => {
-      t.equal(data.toString(), '[17:35:28.992] INFO (42): hello world\n')
+    child.on('error', t.assert.fail)
+    const endPromise = once(child.stdout, 'data', (data) => {
+      t.assert.strictEqual(data.toString(), '[17:35:28.992] INFO (42): hello world\n')
     })
     child.stdin.write(logLine)
-    t.teardown(() => child.kill())
+    await endPromise
+    t.after(() => child.kill())
   })
 
-  ;['--messageKey', '-m'].forEach((optionName) => {
-    t.test(`cli options override config options via ${optionName}`, (t) => {
+  for (const optionName of ['--messageKey', '-m']) {
+    test(`cli options override config options via ${optionName}`, async (t) => {
       t.plan(1)
       // Set translateTime: true on run configuration
       const configFile = path.join(tmpDir, 'pino-pretty.config.js')
@@ -145,19 +152,20 @@ test('cli', (t) => {
       const env = { TERM: ' dumb', TZ: 'UTC' }
       const child = spawn(process.argv[0], [bin, optionName, 'new_msg'], { env, cwd: tmpDir })
       // Validate that the time has been translated and correct message key has been used
-      child.on('error', t.threw)
-      child.stdout.on('data', (data) => {
-        t.equal(data.toString(), '[17:35:28.992] INFO (42): hello world\n')
+      child.on('error', t.assert.fail)
+      const endPromise = once(child.stdout, 'data', (data) => {
+        t.assert.strictEqual(data.toString(), '[17:35:28.992] INFO (42): hello world\n')
       })
       child.stdin.write(logLine.replace(/"msg"/, '"new_msg"'))
-      t.teardown(() => {
+      await endPromise
+      t.after(() => {
         fs.unlinkSync(configFile)
         child.kill()
       })
     })
-  })
+  }
 
-  t.test('cli options with defaults can be overridden by config', (t) => {
+  test('cli options with defaults can be overridden by config', async (t) => {
     t.plan(1)
     // Set errorProps: '*' on run configuration
     const configFile = path.join(tmpDir, 'pino-pretty.config.js')
@@ -170,86 +178,98 @@ test('cli', (t) => {
     const env = { TERM: ' dumb', TZ: 'UTC' }
     const child = spawn(process.argv[0], [bin], { env, cwd: tmpDir })
     // Validate that the time has been translated and correct message key has been used
-    child.on('error', t.threw)
-    child.stdout.on('data', (data) => {
-      t.equal(data.toString(), '[21:31:36.006] FATAL: There was an error starting the process.\n    QueryError: Error during sql query: syntax error at or near SELECTT\n        at /home/me/projects/example/sql.js\n        at /home/me/projects/example/index.js\n    querySql: SELECTT * FROM "test" WHERE id = $1;\n    queryArgs: 12\n')
+    child.on('error', t.assert.fail)
+    const endPromise = once(child.stdout, 'data', (data) => {
+      t.assert.strictEqual(data.toString(), '[21:31:36.006] FATAL: There was an error starting the process.\n    QueryError: Error during sql query: syntax error at or near SELECTT\n        at /home/me/projects/example/sql.js\n        at /home/me/projects/example/index.js\n    querySql: SELECTT * FROM "test" WHERE id = $1;\n    queryArgs: 12\n')
     })
     child.stdin.write('{"level":60,"time":1594416696006,"msg":"There was an error starting the process.","type":"Error","stack":"QueryError: Error during sql query: syntax error at or near SELECTT\\n    at /home/me/projects/example/sql.js\\n    at /home/me/projects/example/index.js","querySql":"SELECTT * FROM \\"test\\" WHERE id = $1;","queryArgs":[12]}\n')
-    t.teardown(() => {
+    await endPromise
+    t.after(() => {
       fs.unlinkSync(configFile)
       child.kill()
     })
   })
 
-  t.test('throws on missing config file', (t) => {
+  test('throws on missing config file', async (t) => {
     t.plan(2)
     const args = [bin, '--config', 'pino-pretty.config.missing.json']
     const env = { TERM: ' dumb', TZ: 'UTC' }
     const child = spawn(process.argv[0], args, { env, cwd: tmpDir })
-    child.on('close', (code) => t.equal(code, 1))
+    const endPromise1 = once(child, 'close', (code) => {
+      t.assert.strictEqual(code, 1)
+    })
     child.stdout.pipe(process.stdout)
     child.stderr.setEncoding('utf8')
     let data = ''
     child.stderr.on('data', (chunk) => {
       data += chunk
     })
-    child.on('close', function () {
-      t.match(
-        data.toString(), 'Error: Failed to load runtime configuration file: pino-pretty.config.missing.json')
+    const endPromise2 = once(child, 'close', () => {
+      match(
+        data.toString(), 'Error: Failed to load runtime configuration file: pino-pretty.config.missing.json', t)
     })
-    t.teardown(() => child.kill())
+    await Promise.all([endPromise1, endPromise2])
+    t.after(() => child.kill())
   })
 
-  t.test('throws on invalid default config file', (t) => {
+  test('throws on invalid default config file', async (t) => {
     t.plan(2)
     const configFile = path.join(tmpDir, 'pino-pretty.config.js')
     fs.writeFileSync(configFile, 'module.exports = () => {}')
     const env = { TERM: ' dumb', TZ: 'UTC' }
     const child = spawn(process.argv[0], [bin], { env, cwd: tmpDir })
-    child.on('close', (code) => t.equal(code, 1))
+    const endPromise1 = once(child, 'close', (code) => {
+      t.assert.strictEqual(code, 1)
+    })
     child.stdout.pipe(process.stdout)
     child.stderr.setEncoding('utf8')
     let data = ''
     child.stderr.on('data', (chunk) => {
       data += chunk
     })
-    child.on('close', function () {
-      t.match(data, 'Error: Invalid runtime configuration file: pino-pretty.config.js')
+    const endPromise2 = once(child, 'close', () => {
+      match(data, 'Error: Invalid runtime configuration file: pino-pretty.config.js', t)
     })
-    t.teardown(() => child.kill())
+    await Promise.all([endPromise1, endPromise2])
+    t.after(() => child.kill())
   })
 
-  t.test('throws on invalid custom config file', (t) => {
+  test('throws on invalid custom config file', async (t) => {
     t.plan(2)
     const configFile = path.join(tmpDir, 'pino-pretty.config.invalid.js')
     fs.writeFileSync(configFile, 'module.exports = () => {}')
     const args = [bin, '--config', path.relative(tmpDir, configFile)]
     const env = { TERM: ' dumb', TZ: 'UTC' }
     const child = spawn(process.argv[0], args, { env, cwd: tmpDir })
-    child.on('close', (code) => t.equal(code, 1))
+    const endPromise1 = once(child, 'close', (code) => {
+      t.assert.strictEqual(code, 1)
+    })
     child.stdout.pipe(process.stdout)
     child.stderr.setEncoding('utf8')
     let data = ''
     child.stderr.on('data', (chunk) => {
       data += chunk
     })
-    child.on('close', function () {
-      t.match(data, 'Error: Invalid runtime configuration file: pino-pretty.config.invalid.js')
+    const endPromise2 = once(child, 'close', () => {
+      match(data, 'Error: Invalid runtime configuration file: pino-pretty.config.invalid.js', t)
     })
-    t.teardown(() => child.kill())
+    await Promise.all([endPromise1, endPromise2])
+    t.after(() => child.kill())
   })
 
-  t.test('test help', (t) => {
+  test('test help', async (t) => {
     t.plan(1)
     const env = { TERM: ' dumb', TZ: 'UTC' }
     const child = spawn(process.argv[0], [bin, '--help'], { env })
     const file = fs.readFileSync('help/help.txt').toString()
-    child.on('error', t.threw)
-    child.stdout.on('data', (data) => {
-      t.equal(data.toString(), file)
+    child.on('error', t.assert.fail)
+
+    await new Promise(resolve => {
+      child.stdout.on('data', (data) => {
+        t.assert.strictEqual(data.toString(), file)
+        resolve()
+      })
     })
-    t.teardown(() => child.kill())
+    t.after(() => child.kill())
   })
-
-  t.end()
 })