Possible Error in TestGossiper_Download_Parallel_Multiple

Possible Error in TestGossiper_Download_Parallel_Multiple

by Sacha Adrien Bartholmé -
Number of replies: 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
In reply to Sacha Adrien Bartholmé

Re: Possible Error in TestGossiper_Download_Parallel_Multiple

by Sacha Adrien Bartholmé -

Never mind, apparently it has already been fixed.

In reply to Sacha Adrien Bartholmé

Re: Possible Error in TestGossiper_Download_Parallel_Multiple

by 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

In reply to Clement Luc Irenee Blaudeau

Re: Possible Error in TestGossiper_Download_Parallel_Multiple

by 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