test: group tests

libp2p · Aug 22, 2017 · 3c16d0a · 3c16d0a
1 parent 8b53921
commit 3c16d0a
Show file tree

Hide file tree

Showing 2 changed files with 59 additions and 56 deletions.
diff --git a/test/nodejs-bundle/circuit.js b/test/nodejs-bundle/circuit.js
@@ -133,68 +133,72 @@ describe(`circuit`, function () {
     ], done)
   })
 
-  it('should dial from WS1 to TCP1 over any R', function (done) {
-    nodeWS1.dial(nodeTCP1.peerInfo, '/echo/1.0.0', (err, conn) => {
-      expect(err).to.not.exist()
-      expect(conn).to.exist()
-
-      pull(
-        pull.values(['hello']),
-        conn,
-        pull.collect((e, result) => {
-          expect(e).to.not.exist()
-          expect(result[0].toString()).to.equal('hello')
-          done()
-        })
-      )
+  describe(`any relay`, function () {
+    it('should dial from WS1 to TCP1 over any R', function (done) {
+      nodeWS1.dial(nodeTCP1.peerInfo, '/echo/1.0.0', (err, conn) => {
+        expect(err).to.not.exist()
+        expect(conn).to.exist()
+
+        pull(
+          pull.values(['hello']),
+          conn,
+          pull.collect((e, result) => {
+            expect(e).to.not.exist()
+            expect(result[0].toString()).to.equal('hello')
+            done()
+          })
+        )
+      })
     })
-  })
 
-  it('should dial from WS1 to TCP1 over R1', function (done) {
-    nodeWS1.dial(nodeTCP1.peerInfo, '/echo/1.0.0', (err, conn) => {
-      expect(err).to.not.exist()
-      expect(conn).to.exist()
-
-      pull(
-        pull.values(['hello']),
-        conn,
-        pull.collect((e, result) => {
-          expect(e).to.not.exist()
-          expect(result[0].toString()).to.equal('hello')
-
-          const addr = multiaddr(handlerSpies[0].args[2][0].dstPeer.addrs[0]).toString()
-          expect(addr).to.equal(`/ipfs/${nodeTCP1.peerInfo.id.toB58String()}`)
-          done()
-        })
-      )
+    it(`should not dial - no R from WS2 to TCP1`, function (done) {
+      nodeWS2.dial(nodeTCP2.peerInfo, '/echo/1.0.0', (err, conn) => {
+        expect(err).to.exist()
+        expect(conn).to.not.exist()
+        done()
+      })
     })
   })
 
-  it(`should dial from WS1 to TCP2 over R2`, function (done) {
-    nodeWS1.dial(nodeTCP2.peerInfo, '/echo/1.0.0', (err, conn) => {
-      expect(err).to.not.exist()
-      expect(conn).to.exist()
-
-      pull(
-        pull.values(['hello']),
-        conn,
-        pull.collect((e, result) => {
-          expect(e).to.not.exist()
-          expect(result[0].toString()).to.equal('hello')
-
-          const addr = multiaddr(handlerSpies[1].args[2][0].dstPeer.addrs[0]).toString()
-          expect(addr).to.equal(`/ipfs/${nodeTCP2.peerInfo.id.toB58String()}`)
-          done()
-        })
-      )
+  describe(`explicit relay`, function () {
+    it('should dial from WS1 to TCP1 over R1', function (done) {
+      nodeWS1.dial(nodeTCP1.peerInfo, '/echo/1.0.0', (err, conn) => {
+        expect(err).to.not.exist()
+        expect(conn).to.exist()
+
+        pull(
+          pull.values(['hello']),
+          conn,
+          pull.collect((e, result) => {
+            expect(e).to.not.exist()
+            expect(result[0].toString()).to.equal('hello')
+
+            const addr = multiaddr(handlerSpies[0].args[2][0].dstPeer.addrs[0]).toString()
+            expect(addr).to.equal(`/ipfs/${nodeTCP1.peerInfo.id.toB58String()}`)
+            done()
+          })
+        )
+      })
     })
-  })
 
-  it(`should not dial - no R from WS2 to TCP1`, function (done) {
-    nodeWS2.dial(nodeTCP2.peerInfo, '/echo/1.0.0', (err, conn) => {
-      expect(err).to.exist()
-      expect(conn).to.not.exist()
-      done()
+    it(`should dial from WS1 to TCP2 over R2`, function (done) {
+      nodeWS1.dial(nodeTCP2.peerInfo, '/echo/1.0.0', (err, conn) => {
+        expect(err).to.not.exist()
+        expect(conn).to.exist()
+
+        pull(
+          pull.values(['hello']),
+          conn,
+          pull.collect((e, result) => {
+            expect(e).to.not.exist()
+            expect(result[0].toString()).to.equal('hello')
+
+            const addr = multiaddr(handlerSpies[1].args[2][0].dstPeer.addrs[0]).toString()
+            expect(addr).to.equal(`/ipfs/${nodeTCP2.peerInfo.id.toB58String()}`)
+            done()
+          })
+        )
+      })
     })
   })
 })
diff --git a/test/nodejs-bundle/utils.js b/test/nodejs-bundle/utils.js
@@ -10,7 +10,6 @@ const waterfall = require('async/waterfall')
 const pull = require('pull-stream')
 
 function createNode (multiaddrs, options, callback) {
-  options = options || {}
   if (typeof options === 'function') {
     callback = options
     options = {}