fix(EventEmitter): fixed a bug when emit returns false if emitter has once listener

Author: IPRIT

src/lib/utils/event-emitter/event-emitter.spec.ts

@@ -24,6 +24,15 @@ test('emits', t => {
 
   eventEmitter.emit(events.CREATED, originatedValue, 2);
   eventEmitter.emit(events.DESTROYED, originatedValue, 2);
+
+  eventEmitter.removeAllListeners();
+
+  eventEmitter.once(events.CREATED, value => {
+    t.is(value, originatedValue);
+  });
+
+  t.is(eventEmitter.emit(events.CREATED, originatedValue), true);
+  t.is(eventEmitter.emit(events.CREATED, originatedValue), false);
 });
 
 test('once', t => {
@@ -118,3 +127,17 @@ test('removeAllListeners', t => {
   eventEmitter.emit(events.DESTROYED, 1);
   t.is(invokes2, m);
 });
+
+test('eventNames', t => {
+  const originatedValue = Symbol('data');
+
+  const eventEmitter = new EventEmitter<typeof originatedValue>();
+
+  eventEmitter.once(events.CREATED, value => {
+    t.is(value, originatedValue);
+  });
+
+  eventEmitter.emit(events.CREATED, originatedValue);
+
+  t.is(eventEmitter.eventNames().length, 0);
+});

src/lib/utils/event-emitter/event-emitter.ts

@@ -84,6 +84,10 @@ export class EventEmitter<T> implements EmitEvents<T>, ListenEvents<T> {
       container.splice(index, 1);
     }
 
+    if (container.length === 0) {
+      this.deleteEventContainer(eventName);
+    }
+
     return this;
   }
 
@@ -100,11 +104,14 @@ export class EventEmitter<T> implements EmitEvents<T>, ListenEvents<T> {
       return false;
     }
 
+    let emittedOnce = false;
+
     for (const [cb] of container) {
       cb(value, ...args);
+      emittedOnce = true;
     }
 
-    return container.length > 0;
+    return emittedOnce;
   }
 
   /**