fix: attach stdout/stderr to error object

microlinkhq · Sep 18, 2023 · a1f41ec · a1f41ec
1 parent 9b29329
commit a1f41ec
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/index.js b/src/index.js
@@ -12,7 +12,7 @@ const isJSON = (str = '') => str.startsWith('{')
 
 const parse = ({ stdout, stderr, ...details }) => {
   if (stdout !== '' && stdout !== 'null') return isJSON(stdout) ? JSON.parse(stdout) : stdout
-  throw Object.assign(new Error(stderr), details)
+  throw Object.assign(new Error(stderr), { stderr, stdout }, details)
 }
 
 const create = binaryPath => {

diff --git a/test/error.js b/test/error.js
@@ -9,7 +9,7 @@ test('show help', async t => {
 })
 
 test('unsupported URLs', async t => {
-  t.plan(4)
+  t.plan(6)
   const url = 'https://www.apple.com/homepod'
   try {
     await youtubedl(url, { dumpSingleJson: true, noWarnings: true })
@@ -20,6 +20,8 @@ test('unsupported URLs', async t => {
     )
     t.true(error instanceof Error)
     t.truthy(error.command)
+    t.truthy(error.stderr)
+    t.truthy(error.stdout)
     t.truthy(error.exitCode)
   }
 })