Merge branch 'main' into chore/use-vitest

src/node/SetupServerApi.ts

@@ -14,7 +14,7 @@ import { mergeRight } from '~/core/utils/internal/mergeRight'
 import { handleRequest } from '~/core/utils/handleRequest'
 import { devUtils } from '~/core/utils/internal/devUtils'
 import { SetupServer } from './glossary'
-import { isNodeException } from './utils/isNodeException'
+import { isNodeExceptionLike } from './utils/isNodeExceptionLike'
 
 const DEFAULT_LISTEN_OPTIONS: RequiredDeep<SharedOptions> = {
   onUnhandledRequest: 'warn',
@@ -80,7 +80,10 @@ export class SetupServerApi
            * which won't be printed anyway if `setMaxListeners` fails.
            */
           if (
-            !(isNodeException(error) && error.code === 'ERR_INVALID_ARG_TYPE')
+            !(
+              isNodeExceptionLike(error) &&
+              error.code === 'ERR_INVALID_ARG_TYPE'
+            )
           ) {
             throw error
           }

src/node/utils/isNodeExceptionLike.ts

@@ -0,0 +1,14 @@
+/**
+ * Determines if the given value is shaped like a Node.js exception.
+ * Node.js exceptions have additional information, like
+ * the `code` and `errno` properties.
+ *
+ * In some environments, particularly jsdom/jest these may not
+ * instances of `Error` or its subclasses, despite being similar
+ * to them.
+ */
+export function isNodeExceptionLike(
+  error: unknown,
+): error is NodeJS.ErrnoException {
+  return !!error && typeof error === 'object' && 'code' in error
+}