feat: ensure charging station add op return its station info

[e-mobility-charging-stations-simulator.git] / src / worker / WorkerDynamicPool.ts

diff --git a/src/worker/WorkerDynamicPool.ts b/src/worker/WorkerDynamicPool.ts
index eebc8fbbd34a280d10cdd9462e96abcb11914719..5217d7fe16a6772a81b946969702fe1189f0d5dc 100644 (file)
--- a/src/worker/WorkerDynamicPool.ts
+++ b/src/worker/WorkerDynamicPool.ts
@@ -1,11 +1,16 @@
-import { DynamicThreadPool, type PoolEmitter, type PoolInfo } from 'poolifier';
+import type { EventEmitterAsyncResource } from 'node:events'
 
-import { WorkerAbstract } from './WorkerAbstract';
-import type { WorkerData, WorkerOptions } from './WorkerTypes';
-import { sleep } from './WorkerUtils';
+import { DynamicThreadPool, type PoolInfo } from 'poolifier'
 
-export class WorkerDynamicPool extends WorkerAbstract<WorkerData> {
-  private readonly pool: DynamicThreadPool<WorkerData>;
+import { WorkerAbstract } from './WorkerAbstract.js'
+import type { WorkerData, WorkerOptions } from './WorkerTypes.js'
+import { randomizeDelay, sleep } from './WorkerUtils.js'
+
+export class WorkerDynamicPool<D extends WorkerData, R extends WorkerData> extends WorkerAbstract<
+D,
+R
+> {
+  private readonly pool: DynamicThreadPool<D, R>
 
   /**
    * Creates a new `WorkerDynamicPool`.
@@ -13,46 +18,50 @@ export class WorkerDynamicPool extends WorkerAbstract<WorkerData> {
    * @param workerScript -
    * @param workerOptions -
    */
-  constructor(workerScript: string, workerOptions?: WorkerOptions) {
-    super(workerScript, workerOptions);
-    this.pool = new DynamicThreadPool<WorkerData>(
+  constructor (workerScript: string, workerOptions: WorkerOptions) {
+    super(workerScript, workerOptions)
+    this.pool = new DynamicThreadPool<D, R>(
       this.workerOptions.poolMinSize,
       this.workerOptions.poolMaxSize,
       this.workerScript,
-      this.workerOptions.poolOptions,
-    );
+      this.workerOptions.poolOptions
+    )
   }
 
-  get info(): PoolInfo {
-    return this.pool.info;
+  get info (): PoolInfo {
+    return this.pool.info
   }
 
-  get size(): number {
-    return this.pool.info.workerNodes;
+  get size (): number {
+    return this.pool.info.workerNodes
   }
 
-  get maxElementsPerWorker(): number | undefined {
-    return undefined;
+  get maxElementsPerWorker (): number | undefined {
+    return undefined
   }
 
-  get emitter(): PoolEmitter | undefined {
-    return this.pool?.emitter;
+  get emitter (): EventEmitterAsyncResource | undefined {
+    return this.pool.emitter
   }
 
   /** @inheritDoc */
-  public async start(): Promise<void> {
-    // This is intentional
+  public start (): void {
+    this.pool.start()
   }
 
   /** @inheritDoc */
-  public async stop(): Promise<void> {
-    return this.pool.destroy();
+  public async stop (): Promise<void> {
+    await this.pool.destroy()
   }
 
   /** @inheritDoc */
-  public async addElement(elementData: WorkerData): Promise<void> {
-    await this.pool.execute(elementData);
+  public async addElement (elementData: D): Promise<R> {
+    const response = await this.pool.execute(elementData)
     // Start element sequentially to optimize memory at startup
-    this.workerOptions.elementStartDelay > 0 && (await sleep(this.workerOptions.elementStartDelay));
+    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+    this.workerOptions.elementAddDelay! > 0 &&
+      // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+      (await sleep(randomizeDelay(this.workerOptions.elementAddDelay!)))
+    return response
   }
 }