refactor: merge dynamic pool events emission code

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

Author: jerome-benoit

src/pools/abstract-pool.ts

@@ -2184,14 +2184,6 @@ export abstract class AbstractPool<
     return workerNodeKey
   }
 
-  private checkAndEmitEmptyEvent(): void {
-    if (this.eventTarget != null && this.empty) {
-      this.eventTarget.dispatchEvent(
-        new CustomEvent<PoolInfo>(PoolEvents.empty, { detail: this.info }),
-      )
-    }
-  }
-
   /**
    * Removes the worker node from the pool worker nodes.
    *
@@ -2204,7 +2196,6 @@ export abstract class AbstractPool<
       this.workerChoiceStrategiesContext?.remove(workerNodeKey)
       workerNode.info.dynamic &&
         this.checkAndEmitDynamicWorkerDestructionEvents()
-      this.checkAndEmitEmptyEvent()
     }
   }
 

src/pools/thread/dynamic.ts

@@ -61,11 +61,18 @@ export class DynamicThreadPool<
 
   /** @inheritDoc */
   protected checkAndEmitDynamicWorkerDestructionEvents(): void {
-    if (this.eventTarget != null && this.fullEventEmitted && !this.full) {
-      this.eventTarget.dispatchEvent(
-        new CustomEvent<PoolInfo>(PoolEvents.fullEnd, { detail: this.info }),
-      )
-      this.fullEventEmitted = false
+    if (this.eventTarget != null) {
+      if (this.fullEventEmitted && !this.full) {
+        this.eventTarget.dispatchEvent(
+          new CustomEvent<PoolInfo>(PoolEvents.fullEnd, { detail: this.info }),
+        )
+        this.fullEventEmitted = false
+      }
+      if (this.empty) {
+        this.eventTarget.dispatchEvent(
+          new CustomEvent<PoolInfo>(PoolEvents.empty, { detail: this.info }),
+        )
+      }
     }
   }