chore: apply suggestions from code review

Co-Authored-By: Jacob Heun <jacobheun@gmail.com>

src/peer-store/index.js

@@ -65,7 +65,6 @@ class PeerStore extends EventEmitter {
 
   /**
    * Updates an already known peer.
-   * If already exist, updates ids info if outdated.
    * @param {PeerInfo} peerInfo
    */
   update (peerInfo) {
@@ -131,7 +130,7 @@ class PeerStore extends EventEmitter {
   }
 
   /**
-   * Remove the info of the peer with the given id.
+   * Removes the Peer with the matching `peerId` from the PeerStore
    * @param {string} peerId b58str id
    * @returns {boolean} true if found and removed
    */
@@ -140,7 +139,7 @@ class PeerStore extends EventEmitter {
   }
 
   /**
-   * Replace the info stored of the given peer.
+   * Completely replaces the existing peers metadata with the given `peerInfo`
    * @param {PeerInfo} peerInfo
    * @returns {void}
    */

test/peer-store/peer-store.spec.js

@@ -109,7 +109,7 @@ describe('peer-store', () => {
     peerStore.on('change:multiaddrs', () => {
       throw new Error('should not emit change:multiaddrs')
     })
-    // When updating, "change:protocols" event must not be emitted
+    // When updating, "change:protocols" event must be emitted
     peerStore.on('change:protocols', ({ peerInfo, protocols }) => {
       expect(peerInfo).to.exist()
       expect(protocols).to.exist()
@@ -152,10 +152,11 @@ describe('peer-store', () => {
     // Put the peer in the store
     peerStore.put(peerInfo)
 
-    const peers = peerStore.getAllArray()
+    const peers = peerStore.peers.size
     expect(peers).to.have.lengthOf(1)
 
     removed = peerStore.remove(id)
     expect(removed).to.eql(true)
+    expect(peerStore.peers.size).to.equal(0)
   })
 })