fix: dispatch peer event on adding new addresses as well as set (libp…

…2p#74)
maschad · Apr 11, 2023 · f6d7658 · f6d7658
1 parent 3df533c
commit f6d7658
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 2 deletions.
diff --git a/src/address-book.ts b/src/address-book.ts
@@ -246,7 +246,7 @@ export class PeerStoreAddressBook {
     const release = await this.store.lock.writeLock()
     log.trace('add got write lock')
 
-    let hasPeer
+    let hasPeer = false
     let peer: Peer | undefined
     let updatedPeer
 
@@ -291,7 +291,7 @@ export class PeerStoreAddressBook {
     }))
 
     // Notify the existence of a new peer
-    if (hasPeer === true) {
+    if (!hasPeer) {
       this.dispatchEvent(new CustomEvent<PeerInfo>('peer', {
         detail: {
           id: peerId,

diff --git a/test/address-book.spec.ts b/test/address-book.spec.ts
@@ -201,6 +201,32 @@ describe('addressBook', () => {
       await defer.promise
     })
 
+    it('emits peer event if new peer has addresses added', async () => {
+      const defer = pDefer()
+
+      peerStore.addEventListener('peer', () => {
+        defer.resolve()
+      })
+
+      await ab.add(await createEd25519PeerId(), [
+        multiaddr('/ip4/42.14.53.21/tcp/3981')
+      ])
+      await defer.promise
+    })
+
+    it('emits peer event if new peer has addresses set', async () => {
+      const defer = pDefer()
+
+      peerStore.addEventListener('peer', () => {
+        defer.resolve()
+      })
+
+      await ab.set(await createEd25519PeerId(), [
+        multiaddr('/ip4/42.14.53.21/tcp/3981')
+      ])
+      await defer.promise
+    })
+
     it('adds the new content and emits change event', async () => {
       const defer = pDefer()