fix: throw wrapped error with legacy middleware

unjs · Apr 1, 2022 · 27e9477 · 27e9477
1 parent 5e86d0c
commit 27e9477
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/handler.ts b/src/handler.ts
@@ -1,4 +1,5 @@
 import { withoutTrailingSlash, withoutBase } from 'ufo'
+import { createError } from './error'
 import type { Handler, PromisifiedHandler, Middleware, IncomingMessage, ServerResponse, LazyHandler } from './types'
 
 export const defineHandler = <T>(handler: Handler<T>) => handler
@@ -25,7 +26,7 @@ export function callHandler (handler: Middleware, req: IncomingMessage, res: Ser
         res.off('close', next)
         res.off('error', next)
       }
-      return err ? reject(err) : resolve(undefined)
+      return err ? reject(createError(err)) : resolve(undefined)
     }
     try {
       const returned = handler(req, res, next)

diff --git a/test/error.test.ts b/test/error.test.ts
@@ -38,9 +38,6 @@ describe('error', () => {
     const result = await request.get('/api/test')
 
     expect(result.status).toBe(500)
-    // eslint-disable-next-line
-    expect(console.error).toBeCalled()
-
     expect(JSON.parse(result.text)).toMatchObject({
       statusCode: 500,
       statusMessage: 'H3Error'