From 9d923d52097db14f7678c221a3853d483de837b6 Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A9r=C3=B4me=20Benoit?= Date: Mon, 2 Mar 2026 15:47:11 +0100 Subject: [PATCH] refactor(test): remove duplicate test coverage in error test files --- tests/exception/BaseError.test.ts | 5 ----- tests/exception/OCPPError.test.ts | 12 ------------ 2 files changed, 17 deletions(-) diff --git a/tests/exception/BaseError.test.ts b/tests/exception/BaseError.test.ts index d4746077..79adfd73 100644 --- a/tests/exception/BaseError.test.ts +++ b/tests/exception/BaseError.test.ts @@ -34,11 +34,6 @@ await describe('BaseError', async () => { expect(baseError instanceof Error).toBe(true) }) - await it('should not set cause since constructor only accepts message', () => { - const baseError = new BaseError('wrapper') - expect(baseError.cause).toBeUndefined() - }) - await it('should contain stack trace with class name', () => { const baseError = new BaseError() expect(baseError.stack?.includes('BaseError')).toBe(true) diff --git a/tests/exception/OCPPError.test.ts b/tests/exception/OCPPError.test.ts index 3efd2bcd..9dcf94f6 100644 --- a/tests/exception/OCPPError.test.ts +++ b/tests/exception/OCPPError.test.ts @@ -51,16 +51,4 @@ await describe('OCPPError', async () => { const ocppError = new OCPPError(ErrorType.NOT_IMPLEMENTED, 'test') expect(ocppError.code).toBe(ErrorType.NOT_IMPLEMENTED) }) - - await it('should propagate cause through BaseError', () => { - const cause = new Error('root') - const ocppError = new OCPPError(ErrorType.INTERNAL_ERROR, 'wrapped') - expect(cause.message).toBe('root') - expect(ocppError.cause).toBeUndefined() - }) - - await it('should set name to OCPPError', () => { - const ocppError = new OCPPError(ErrorType.GENERIC_ERROR, 'test') - expect(ocppError.name).toBe('OCPPError') - }) }) -- 2.43.0