Add a test

Author: ehmicky

CHANGELOG.md

@@ -38,7 +38,7 @@ logProcessErrors({
 ## Pretty-printing
 
 Errors are not pretty-printed anymore. As a consequence, the `colors` option was
-removed. The [`onError` option](README.md#onerror) can be used instead to
+removed too. The [`onError` option](README.md#onerror) can be used instead to
 customize how the errors are printed.
 
 ## Filtering
@@ -49,7 +49,11 @@ be used for filtering.
 Before:
 
 ```js
-logProcessErrors({ warning: 'silent' })
+logProcessErrors({
+  levels: {
+    warning: 'silent',
+  },
+})
 ```
 
 After:
@@ -79,6 +83,7 @@ After:
 
 ```js
 logProcessErrors({
+  // Throw the `error` to make the unit test fail while letting other tests run
   onError(error) {
     throw error
   },
@@ -115,19 +120,19 @@ process. This created conflicts with this library. This has been fixed by making
 the [`exit` option](README.md#exit) default to `false` when process events
 listeners already exist.
 
-## TypeScript
-
-TypeScript types have been simplified.
-
 ## Bug fixes
 
 - Fix support for `--unhandled-rejections=strict`
 - Do not crash when `error.stack` is `undefined` or `null`
 - Support cross-realm errors
 
+## TypeScript
+
+TypeScript types have been simplified.
+
 ## Internal
 
-- Add 100% test coverage
+Added 100% test coverage.
 
 # 9.4.0
 

README.md

@@ -91,7 +91,7 @@ seconds.
 
 ### onError
 
-_Type_: `(Error, string) => Promise<void> | void`\
+_Type_: `(error, event) => Promise<void> | void`\
 _Default_: `console.error(error)`
 
 Function called once per process error. Duplicate process errors are ignored.

test/helpers/error.js

@@ -14,3 +14,12 @@ export const getRandomMessageError = function () {
 export const getObjectError = function (eventName) {
   return eventName === 'warning' ? 'message' : { message: 'message' }
 }
+
+export const getInvalidError = function () {
+  // eslint-disable-next-line fp/no-mutating-assign
+  return Object.assign(getError(), {
+    name: undefined,
+    message: undefined,
+    stack: undefined,
+  })
+}

test/repeat.js

@@ -1,7 +1,11 @@
 import test from 'ava'
 import { each } from 'test-each'
 
-import { getRandomMessageError, getObjectError } from './helpers/error.js'
+import {
+  getRandomMessageError,
+  getObjectError,
+  getInvalidError,
+} from './helpers/error.js'
 import {
   EVENTS,
   emitMany,
@@ -70,6 +74,23 @@ each(EVENTS, ({ title }, eventName) => {
       stopLogging()
     },
   )
+
+  test.serial(
+    `should not repeat values that are invalid errors | ${title}`,
+    async (t) => {
+      if (eventName === 'rejectionHandled') {
+        return t.pass()
+      }
+
+      const { onError, stopLogging } = startLogging()
+
+      t.is(onError.callCount, 0)
+      await emitManyValues(getInvalidError, eventName, 2)
+      t.is(onError.callCount, getCallCount(eventName))
+
+      stopLogging()
+    },
+  )
 })
 
 removeProcessListeners()