fix(index): expose error codes for pdffonts and pdfinfo

Fdawgs · Sep 23, 2023 · c2f277b · c2f277b
1 parent dc524a7
commit c2f277b
Showing 1 changed file with 28 additions and 4 deletions.
diff --git a/src/index.js b/src/index.js
@@ -254,11 +254,23 @@ class Poppler {
 					stdErr += data;
 				});
 
-				child.on("close", () => {
+				child.on("close", (code) => {
+					/* istanbul ignore else */
 					if (stdOut !== "") {
 						resolve(stdOut.trim());
+					} else if (code === 0) {
+						resolve(errorMessages[code]);
+					} else if (stdErr !== "") {
+						reject(new Error(stdErr.trim()));
 					} else {
-						reject(new Error(stdErr ? stdErr.trim() : undefined));
+						reject(
+							new Error(
+								errorMessages[code] ||
+									`pdffonts ${args.join(
+										" "
+									)} exited with code ${code}`
+							)
+						);
 					}
 				});
 			});
@@ -479,7 +491,8 @@ class Poppler {
 					stdErr += data;
 				});
 
-				child.on("close", () => {
+				child.on("close", (code) => {
+					/* istanbul ignore else */
 					if (stdOut !== "") {
 						if (fileSize) {
 							stdOut = stdOut.replace(
@@ -505,8 +518,19 @@ class Poppler {
 						} else {
 							resolve(stdOut.trim());
 						}
+					} else if (code === 0) {
+						resolve(errorMessages[code]);
+					} else if (stdErr !== "") {
+						reject(new Error(stdErr.trim()));
 					} else {
-						reject(new Error(stdErr ? stdErr.trim() : undefined));
+						reject(
+							new Error(
+								errorMessages[code] ||
+									`pdfinfo ${args.join(
+										" "
+									)} exited with code ${code}`
+							)
+						);
 					}
 				});
 			});