Skip to content

Commit ce8349e

Browse files
authored
fix: set RequestError name properly (#243)
1 parent a47d192 commit ce8349e

File tree

2 files changed

+14
-1
lines changed

2 files changed

+14
-1
lines changed

src/request.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,14 +7,14 @@ import { Readable } from 'node:stream'
77
import type { TLSSocket } from 'node:tls'
88

99
export class RequestError extends Error {
10-
static name = 'RequestError'
1110
constructor(
1211
message: string,
1312
options?: {
1413
cause?: unknown
1514
}
1615
) {
1716
super(message, options)
17+
this.name = 'RequestError'
1818
}
1919
}
2020

test/request.test.ts

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -320,3 +320,16 @@ describe('Request', () => {
320320
})
321321
})
322322
})
323+
324+
describe('RequestError', () => {
325+
it('should have a static name property (class name)', () => {
326+
expect(RequestError.name).toBe('RequestError')
327+
expect(Object.hasOwn(RequestError, 'name')).toBe(true)
328+
})
329+
330+
it('should have an instance name property', () => {
331+
const error = new RequestError('message')
332+
expect(error.name).toBe('RequestError')
333+
expect(Object.hasOwn(error, 'name')).toBe(true)
334+
})
335+
})

0 commit comments

Comments
 (0)