fix: enableCircuitRelay is async and therefore needs a callback

Author: daviddias

README.md

@@ -135,7 +135,7 @@ const secio = require('libp2p-secio')
 switch.connection.crypto(secio.tag, secio.encrypt)
 ```
 
-##### `switch.connection.enableCircuitRelay(options)`
+##### `switch.connection.enableCircuitRelay(options, callback)`
 
 Enable circuit relaying.
 
@@ -144,6 +144,7 @@ Enable circuit relaying.
     - hop - an object with two properties
         - enabled - enables circuit relaying
         - active - is it an active or passive relay (default false)
+- `callback`
 
 ### Internal Transports API
 

test/circuit-relay.node.js

@@ -55,14 +55,17 @@ describe(`circuit`, function () {
   })
 
   it('.enableCircuitRelay', () => {
-    swarmA.connection.enableCircuitRelay({ enabled: true })
-    expect(Object.keys(swarmA.transports).length).to.equal(3)
-
-    swarmB.connection.enableCircuitRelay({ enabled: true })
-    expect(Object.keys(swarmB.transports).length).to.equal(2)
+    parallel([
+      (cb) => swarmA.connection.enableCircuitRelay({ enabled: true }, cb),
+      (cb) => swarmB.connection.enableCircuitRelay({ enabled: true }, cb)
+    ], (err) => {
+      expect(err).to.not.exist()
+      expect(Object.keys(swarmA.transports).length).to.equal(3)
+      expect(Object.keys(swarmB.transports).length).to.equal(2)
+    })
   })
 
-  it('add circuit to the transports lists', () => {
+  it('listed on the transports map', () => {
     expect(swarmA.transports['Circuit']).to.exist()
     expect(swarmB.transports['Circuit']).to.exist()
   })

test/swarm-muxing.node.js

@@ -178,13 +178,8 @@ describe('Switch (everything all together)', () => {
     let i = 0
 
     function check (err) {
-      if (err) {
-        return done(err)
-      }
-
-      if (i++ === 2) {
-        done()
-      }
+      expect(err).to.not.exist()
+      if (i++ === 2) { done() }
     }
 
     switchC.handle('/mamao/1.0.0', (protocol, conn) => {
@@ -205,8 +200,8 @@ describe('Switch (everything all together)', () => {
         expect(peerInfo).to.exist()
         check()
       })
-      expect(Object.keys(switchA.muxedConns).length).to.equal(2)
 
+      expect(Object.keys(switchA.muxedConns).length).to.equal(2)
       expect(switchC._peerInfo.isConnected).to.exist()
       expect(switchA._peerInfo.isConnected).to.exist()