@superhero/eventflow-hub 4.0.3 → 4.0.4

package/index.js

@@ -24,7 +24,11 @@ export function locate(locator)
 export default class Hub
 {
   #hubID
-  channel = new Channel()
+
+  abortion    = new AbortController()
+  channel     = new Channel()
+  spokes      = new SpokesManager()
+  subscribers = new SubscribersManager()
 
   get hubID()
   {
@@ -45,11 +49,8 @@ export default class Hub
     this.#hubID       = (new IdNameGenerator().generateId() + '.' + config.NAME).toUpperCase()
     this.config       = config
     this.db           = db
-    this.abortion     = new AbortController()
     this.log          = new Log({ label: `[${config.NAME}]` })
     this.certificates = new CertificatesManager(config.NAME, this.#hubID, config.certificates, db, this.log)
-    this.spokes       = new SpokesManager()
-    this.subscribers  = new SubscribersManager()
 
     this.channel.on('record', this.#onRecord.bind(this))
   }
@@ -63,11 +64,20 @@ export default class Hub
 
   async destroy()
   {
-    this.abortion.abort()
+    const reason = new Error('hub is destroyed')
+    reason.code  = 'E_EVENTFLOW_HUB_DESTROYED'
+
+    this.abortion.abort(reason)
     this.server?.close()
+
+    for(const socket of this.spokes.all)
+    {
+      socket.end()
+    }
+
     this.spokes.destroy()
     this.subscribers.destroy()
-    this.log.warn`hub has quit`
+    this.log.warn`destroyed`
     await this.db.updateHubToQuit(this.#hubID)
     setTimeout(() => this.db.close(), 500)
   }
@@ -269,7 +279,7 @@ export default class Hub
 
   async #attemptToConsumeAndBroadcastPublishedMessage(domain, id, name, pid)
   {
-    const consumed = await this.db.updateEventPublishedToConsumedByHub(domain, id, this.#hubID)
+    const consumed = await this.db.updateEventPublishedToConsumedByHub(id, this.#hubID)
 
     if(consumed)
     {

package/manager/spokes.js

@@ -8,10 +8,7 @@ export default class SpokesManager
 
   destroy()
   {
-    for(const socket of this.#set.values())
-    {
-      socket.end()
-    }
+    this.#set.clear()
   }
 
   add(socket)

package/manager/spokes.test.js

@@ -34,15 +34,14 @@ suite('@superhero/eventflow-hub/manager/spokes', () =>
   test('Destroy all sockets', async () =>
   {
     const manager = new SpokesManager()
-    const socket1 = { id: 'socket1', end: () => socket1.destroyed = true }
-    const socket2 = { id: 'socket2', end: () => socket2.destroyed = true }
+    const socket1 = { id: 'socket1' }
+    const socket2 = { id: 'socket2' }
 
     manager.add(socket1)
     manager.add(socket2)
     manager.destroy()
 
-    assert.strictEqual(socket1.destroyed, true)
-    assert.strictEqual(socket2.destroyed, true)
+    assert.strictEqual(manager.all.length, 0)
   })
 
   test('Handle deleting non-existent socket gracefully', async () =>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@superhero/eventflow-hub",
-  "version": "4.0.3",
+  "version": "4.0.4",
   "description": "Eventflow hub is the central server component in the eventflow ecosystem.",
   "keywords": [
     "eventflow",