Possible Error in TestGossiper_Download_Parallel_Multiple

Possible Error in TestGossiper_Download_Parallel_Multiple

par Sacha Adrien Bartholmé,
Nombre de réponses : 3

There is a possible mistake in this test that made my tests fail. The line

nB.gossiper.AddRoute(nB.id, nB.addr) // B --> A

Should be replaced by 

nB.gossiper.AddRoute(nA.id, nA.addr) // B --> A
En réponse à Sacha Adrien Bartholmé

Re: Possible Error in TestGossiper_Download_Parallel_Multiple

par Sacha Adrien Bartholmé,

Never mind, apparently it has already been fixed.

En réponse à Sacha Adrien Bartholmé

Re: Possible Error in TestGossiper_Download_Parallel_Multiple

par Clement Luc Irenee Blaudeau,

I actually have the exact same issue. Where did the fix came from ? I pulled the master branch of gitlab but it did not change anything

En réponse à Clement Luc Irenee Blaudeau

Re: Possible Error in TestGossiper_Download_Parallel_Multiple

par Kirill Nikitin,

Hi,

The issue was fixed with commit 302f477e67b6d15ec980153507b85d192cf0cf46 when we also added the integration tests. I have checked and I can see it on your master branch.

Or are you referring to some similar issue in another test?

Kirill