From eeed3e863ad6d20b7954aae848f2f4e6797da4a7 Mon Sep 17 00:00:00 2001 From: Yusuke Wada Date: Fri, 23 May 2025 07:57:55 +0900 Subject: [PATCH] fix: set `RequestError` name properly --- src/request.ts | 2 +- test/request.test.ts | 13 +++++++++++++ 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/src/request.ts b/src/request.ts index 7508faf..4065204 100644 --- a/src/request.ts +++ b/src/request.ts @@ -7,7 +7,6 @@ import { Readable } from 'node:stream' import type { TLSSocket } from 'node:tls' export class RequestError extends Error { - static name = 'RequestError' constructor( message: string, options?: { @@ -15,6 +14,7 @@ export class RequestError extends Error { } ) { super(message, options) + this.name = 'RequestError' } } diff --git a/test/request.test.ts b/test/request.test.ts index aa5380c..fc5d7cd 100644 --- a/test/request.test.ts +++ b/test/request.test.ts @@ -320,3 +320,16 @@ describe('Request', () => { }) }) }) + +describe('RequestError', () => { + it('should have a static name property (class name)', () => { + expect(RequestError.name).toBe('RequestError') + expect(Object.hasOwn(RequestError, 'name')).toBe(true) + }) + + it('should have an instance name property', () => { + const error = new RequestError('message') + expect(error.name).toBe('RequestError') + expect(Object.hasOwn(error, 'name')).toBe(true) + }) +})