fluencelabs/js-libp2p
1
0
Fork 0
mirror of https://github.com/fluencelabs/js-libp2p synced 2025-06-17 19:21:22 +00:00
Code Issues Projects Releases Wiki Activity

fix: do not add abort signals to useless addresses (#913)

Browse Source
This commit is contained in:
Vasco Santos
2021-04-20 09:31:16 +02:00
committed by GitHub
parent 3f7dde3e13
commit 06e8f3dd42
3 changed files with 31 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
test/dialing/direct.node.js
View File

@ -98,8 +98,8 @@ describe('Dialing (direct, TCP)', () => {
const dialer = new Dialer({ transportManager: localTM, peerStore })
await expect(dialer.connectToPeer(unsupportedAddr))
.to.eventually.be.rejectedWith(AggregateError)
.and.to.have.nested.property('._errors[0].code', ErrorCodes.ERR_TRANSPORT_UNAVAILABLE)
.to.eventually.be.rejectedWith(Error)
.and.to.have.nested.property('.code', ErrorCodes.ERR_NO_VALID_ADDRESSES)
})
it('should fail to connect if peer has no known addresses', async () => {
@ -139,8 +139,28 @@ describe('Dialing (direct, TCP)', () => {
const peerId = await PeerId.createFromJSON(Peers[0])
await expect(dialer.connectToPeer(peerId))
.to.eventually.be.rejectedWith(AggregateError)
.and.to.have.nested.property('._errors[0].code', ErrorCodes.ERR_TRANSPORT_UNAVAILABLE)
.to.eventually.be.rejectedWith(Error)
.and.to.have.nested.property('.code', ErrorCodes.ERR_NO_VALID_ADDRESSES)
})
it('should only try to connect to addresses supported by the transports configured', async () => {
const remoteAddrs = remoteTM.getAddrs()
const dialer = new Dialer({
transportManager: localTM,
peerStore: {
addressBook: {
add: () => { },
getMultiaddrsForPeer: () => [...remoteAddrs, unsupportedAddr]
}
}
})
const peerId = await PeerId.createFromJSON(Peers[0])
sinon.spy(localTM, 'dial')
const connection = await dialer.connectToPeer(peerId)
expect(localTM.dial.callCount).to.equal(remoteAddrs.length)
expect(connection).to.exist()
await connection.close()
})
it('should abort dials on queue task timeout', async () => {