]> Piment Noir Git Repositories - e-mobility-charging-stations-simulator.git/commitdiff
refactor: use union types consistently in common code and tests
authorJérôme Benoit <jerome.benoit@sap.com>
Mon, 16 Mar 2026 12:36:04 +0000 (13:36 +0100)
committerJérôme Benoit <jerome.benoit@sap.com>
Mon, 16 Mar 2026 12:36:04 +0000 (13:36 +0100)
- OCPPServiceUtils: use SampledValue union in generics, use
  StatusNotificationRequest union in satisfies, remove unused
  OCPP20StatusNotificationRequest import
- BroadcastChannel: use MeterValuesRequest/Response unions
- OCPPServiceUtils-validation.test: OCPP16MessageTrigger → MessageTrigger,
  remove redundant casts

tests/charging-station/ocpp/OCPPServiceUtils-validation.test.ts

index 9561c1e8fd6a9da93016ff904cae85118d5e5ed7..5ecc8d354d01b5ba633e93dc7e8dfc22ba5991e3 100644 (file)
@@ -15,12 +15,7 @@ import { afterEach, describe, it } from 'node:test'
 import type { ChargingStation } from '../../../src/charging-station/ChargingStation.js'
 
 import { OCPPServiceUtils } from '../../../src/charging-station/ocpp/OCPPServiceUtils.js'
-import {
-  IncomingRequestCommand,
-  type MessageTrigger,
-  OCPP16MessageTrigger,
-  RequestCommand,
-} from '../../../src/types/index.js'
+import { IncomingRequestCommand, MessageTrigger, RequestCommand } from '../../../src/types/index.js'
 import { standardCleanup } from '../../helpers/TestLifecycleHelpers.js'
 
 /**
@@ -152,14 +147,11 @@ await describe('OCPPServiceUtils — command/trigger validation', async () => {
     await it('should return true when trigger is not explicitly disabled', () => {
       const station = makeStationMock({
         messageTriggerSupport: {
-          [OCPP16MessageTrigger.Heartbeat]: true,
+          [MessageTrigger.Heartbeat]: true,
         },
       })
 
-      const result = OCPPServiceUtils.isMessageTriggerSupported(
-        station,
-        OCPP16MessageTrigger.Heartbeat as MessageTrigger
-      )
+      const result = OCPPServiceUtils.isMessageTriggerSupported(station, MessageTrigger.Heartbeat)
 
       assert.strictEqual(result, true)
     })
@@ -167,14 +159,11 @@ await describe('OCPPServiceUtils — command/trigger validation', async () => {
     await it('should return false when trigger is explicitly disabled', () => {
       const station = makeStationMock({
         messageTriggerSupport: {
-          [OCPP16MessageTrigger.Heartbeat]: false,
+          [MessageTrigger.Heartbeat]: false,
         },
       })
 
-      const result = OCPPServiceUtils.isMessageTriggerSupported(
-        station,
-        OCPP16MessageTrigger.Heartbeat as MessageTrigger
-      )
+      const result = OCPPServiceUtils.isMessageTriggerSupported(station, MessageTrigger.Heartbeat)
 
       assert.strictEqual(result, false)
     })
@@ -182,10 +171,7 @@ await describe('OCPPServiceUtils — command/trigger validation', async () => {
     await it('should return true when messageTriggerSupport is undefined', () => {
       const station = makeStationMock({})
 
-      const result = OCPPServiceUtils.isMessageTriggerSupported(
-        station,
-        OCPP16MessageTrigger.Heartbeat as MessageTrigger
-      )
+      const result = OCPPServiceUtils.isMessageTriggerSupported(station, MessageTrigger.Heartbeat)
 
       assert.strictEqual(result, true)
     })
@@ -195,10 +181,7 @@ await describe('OCPPServiceUtils — command/trigger validation', async () => {
         messageTriggerSupport: null,
       })
 
-      const result = OCPPServiceUtils.isMessageTriggerSupported(
-        station,
-        OCPP16MessageTrigger.Heartbeat as MessageTrigger
-      )
+      const result = OCPPServiceUtils.isMessageTriggerSupported(station, MessageTrigger.Heartbeat)
 
       assert.strictEqual(result, true)
     })