Improve error handling

ehmicky · Jan 23, 2022 · 09cf169 · 09cf169
1 parent 65b020b
commit 09cf169
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/config/normalize/main.js b/src/config/normalize/main.js
@@ -1,6 +1,8 @@
 // eslint-disable-next-line no-restricted-imports, node/no-restricted-import
 import { AssertionError } from 'assert'
 
+import { wrapError } from '../../error/wrap.js'
+
 import { normalizeConfigProps } from './lib/main.js'
 import { DEFINITIONS } from './prop_defs.js'
 
@@ -32,7 +34,9 @@ const safeNormalizeConfig = async function (config, context, ErrorType) {
 
 // `AssertionErrors` are user-validation errors. Others are system errors.
 const handleConfigError = function (error, ErrorType) {
-  return error instanceof AssertionError ? new ErrorType(error.message) : error
+  return error instanceof AssertionError
+    ? wrapError(error, '', ErrorType)
+    : error
 }
 
 // Perform normalization that is difficult to do with the main configuration

diff --git a/src/error/wrap.js b/src/error/wrap.js
@@ -28,7 +28,7 @@ const changeErrorType = function (error, ErrorType) {
   return newError
 }
 
-// Prepend a prefix to an error message
+// Prepend|append a prefix|suffix to an error message
 const wrapErrorMessage = function (error, message) {
   if (message === '') {
     return error