From 48b9237bf2d47c4b05b316a62aa0ec6970da38c0 Mon Sep 17 00:00:00 2001 From: Yusef Napora Date: Fri, 10 Feb 2017 14:19:30 -0500 Subject: [PATCH] make coveralls happy --- test/peer/directory_test.js | 11 +++++++++++ test/peer/node_info_test.js | 13 +++++++++---- 2 files changed, 20 insertions(+), 4 deletions(-) diff --git a/test/peer/directory_test.js b/test/peer/directory_test.js index 5db9eba..275805a 100644 --- a/test/peer/directory_test.js +++ b/test/peer/directory_test.js @@ -39,6 +39,17 @@ describe('Directory Node', function () { })) }) + it('node can set directory with string multiaddr', () => { + const dirInfo = dir.p2p.peerInfo + const dirId = dirInfo.id + const addrString = dirInfo.multiaddrs[0].toString() + '/p2p/' + dirId.toB58String() + expect(() => + node.setDirectory(addrString) + ).to.not.throw() + + node.setDirectory(dirInfo) + }) + it('responds to lookup requests for known peers', function () { // just stuff the node's id into the directory manually dir.peerBook.put(node.peerInfo) diff --git a/test/peer/node_info_test.js b/test/peer/node_info_test.js index 6d5a4b8..35cba43 100644 --- a/test/peer/node_info_test.js +++ b/test/peer/node_info_test.js @@ -3,15 +3,18 @@ const assert = require('assert') const { before, describe, it } = require('mocha') const { makeNode } = require('../util') +const { PublisherId } = require('../../src/peer/identity') describe('Node Info', function () { const infoMessage = 'tests are great!' let p1, p2 - before(() => Promise.all([ - makeNode({listenAddresses: ['/ip4/127.0.0.1/tcp/9090/ws']}).then(_p1 => { p1 = _p1 }), - makeNode({listenAddresses: ['/ip4/127.0.0.1/tcp/9091/ws']}).then(_p2 => { p2 = _p2; p2.setInfoMessage(infoMessage) }) - ])) + before(() => PublisherId.generate() + .then(publisherId => Promise.all([ + makeNode({listenAddresses: ['/ip4/127.0.0.1/tcp/9090/ws']}).then(_p1 => { p1 = _p1 }), + makeNode({publisherId, listenAddresses: ['/ip4/127.0.0.1/tcp/9091/ws']}).then(_p2 => { p2 = _p2; p2.setInfoMessage(infoMessage) }) + ])) + ) it('retrieves the ids and info message from another node', () => { return Promise.all([p1.start(), p2.start()]) // start both peers @@ -21,6 +24,8 @@ describe('Node Info', function () { 'node info response should include correct peer id') assert.equal(result.info, infoMessage, 'node info response should include correct info message') + assert.equal(result.publisher, p2.publisherId.id58, + 'if remote node has a publisher id, it should be included') }) .then(() => Promise.all([p1.stop(), p2.stop()])) })