feat(log): Behave like native Error when using console.log

Only CustomError class was behaving different from native Error

Add basic unit test to avoid regression

fix #30

src/custom-error.spec.ts

@@ -54,3 +54,6 @@ test('Without message', () =>
 		name: 'CustomError',
 		message: '',
 	}))
+
+test('native log behaviour', () =>
+	expect(`${new CustomError('Hello')}`).toMatch('CustomError: Hello'))

src/custom-error.ts

@@ -22,9 +22,13 @@ export class CustomError extends Error {
 
 	constructor(message?: string) {
 		super(message)
-		// set error name as constructor name
+		// set error name as constructor name, make it not enumerable to keep native Error behavior
 		// see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/new.target#new.target_in_constructors
-		this.name = new.target.name
+		// see https://github.com/adriengibrat/ts-custom-error/issues/30
+		Object.defineProperty(this, 'name', {
+			value: new.target.name,
+			enumerable: false,
+		})
 		// fix the extended error prototype chain
 		// because typescript __extends implementation can't
 		// see https://github.com/Microsoft/TypeScript-wiki/blob/master/Breaking-Changes.md#extending-built-ins-like-error-array-and-map-may-no-longer-work

src/factory.spec.ts

@@ -85,3 +85,8 @@ test('Factory properties', () => {
 		message: 'foo',
 	})
 })
+
+test('native log behaviour', () =>
+	expect(`${customErrorFactory(function TestError(this: Props, message) {
+		this.message = message
+	})('Hello')}`).toMatch('TestError: Hello'))