Change serializer errors to use error codes

mochajs · Oct 5, 2020 · e7214f5 · e7214f5
1 parent 8f5d6a9
commit e7214f5
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 16 deletions.
diff --git a/lib/nodejs/serializer.js b/lib/nodejs/serializer.js
@@ -7,7 +7,10 @@
 'use strict';
 
 const {type} = require('../utils');
-const {createInvalidArgumentTypeError} = require('../errors');
+const {
+  createInvalidArgumentTypeError,
+  createInvalidArgumentValueError
+} = require('../errors');
 // this is not named `mocha:parallel:serializer` because it's noisy and it's
 // helpful to be able to write `DEBUG=mocha:parallel*` and get everything else.
 const debug = require('debug')('mocha:serializer');
@@ -131,7 +134,12 @@ class SerializableEvent {
    */
   constructor(eventName, originalValue, originalError) {
     if (!eventName) {
-      throw new Error('expected a non-empty `eventName` string argument');
+      throw createInvalidArgumentValueError(
+        'Empty `eventName` string argument',
+        'eventName',
+        eventName,
+        'is empty'
+      );
     }
     /**
      * The event name.
@@ -140,8 +148,10 @@ class SerializableEvent {
     this.eventName = eventName;
     const originalValueType = type(originalValue);
     if (originalValueType !== 'object' && originalValueType !== 'undefined') {
-      throw new Error(
-        `expected object, received [${originalValueType}]: ${originalValue}`
+      throw createInvalidArgumentTypeError(
+        `Expected object but received ${originalValueType}`,
+        'originalValue',
+        'object'
       );
     }
     /**

diff --git a/test/node-unit/serializer.spec.js b/test/node-unit/serializer.spec.js
@@ -88,22 +88,18 @@ describe('serializer', function() {
   describe('SerializableEvent', function() {
     describe('constructor', function() {
       describe('when called without `eventName`', function() {
-        it('should throw', function() {
-          expect(
-            () => new SerializableEvent(),
-            'to throw',
-            /expected a non-empty `eventName`/
-          );
+        it('should throw "invalid arg value" error', function() {
+          expect(() => new SerializableEvent(), 'to throw', {
+            code: 'ERR_MOCHA_INVALID_ARG_VALUE'
+          });
         });
       });
 
       describe('when called with a non-object `rawObject`', function() {
-        it('should throw', function() {
-          expect(
-            () => new SerializableEvent('blub', 'glug'),
-            'to throw',
-            /expected object, received \[string\]/
-          );
+        it('should throw "invalid arg type" error', function() {
+          expect(() => new SerializableEvent('blub', 'glug'), 'to throw', {
+            code: 'ERR_MOCHA_INVALID_ARG_TYPE'
+          });
         });
       });
     });