fix: release providers resources (#23)

Author: richardschneider

src/index.js

@@ -120,6 +120,7 @@ class KadDHT {
   stop (callback) {
     this._running = false
     this.bootstrapStop()
+    this.providers.stop()
     this.network.stop(callback)
   }
 

src/providers.js

@@ -58,6 +58,18 @@ class Providers {
     this.providers = cache(this.lruCacheSize)
   }
 
+  /**
+   * Release any resources.
+   *
+   * @returns {undefined}
+   */
+  stop () {
+    if (this._cleaner) {
+      clearInterval(this._cleaner)
+      this._cleaner = null
+    }
+  }
+
   /**
    * Check all providers if they are still valid, and if not
    * delete them.

test/providers.spec.js

@@ -51,6 +51,7 @@ describe('Providers', () => {
       providers.getProviders(cid, (err, provs) => {
         expect(err).to.not.exist()
         expect(provs).to.be.eql([infos[0].id, infos[1].id])
+        providers.stop()
 
         done()
       })
@@ -82,6 +83,7 @@ describe('Providers', () => {
           provs.forEach((p) => {
             expect(p[0].id).to.be.eql(infos[0].id.id)
           })
+          providers.stop()
           cb()
         })
       }
@@ -111,6 +113,7 @@ describe('Providers', () => {
         providers.getProviders(cid, (err, provs) => {
           expect(err).to.not.exist()
           expect(provs).to.have.length(0)
+          providers.stop()
           done()
         })
       }, 300)