Merge pull request #24545 from karalabe/beacon-sync-fix-test

eth/downloader: fix off-by-one error in test causing 50% fails
ethereum · Mar 16, 2022 · 2352c72 · 2352c72
2 parents dbfd397 + 6b8718c
commit 2352c72
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/eth/downloader/skeleton_test.go b/eth/downloader/skeleton_test.go
@@ -742,13 +742,13 @@ func TestSkeletonSyncRetrievals(t *testing.T) {
 			head: chain[2*requestHeaders],
 			peers: []*skeletonTestPeer{
 				newSkeletonTestPeerWithHook("peer-1", chain, func(origin uint64) []*types.Header {
-					if origin == chain[2*requestHeaders+2].Number.Uint64() {
+					if origin == chain[2*requestHeaders+1].Number.Uint64() {
 						time.Sleep(100 * time.Millisecond)
 					}
 					return nil // Fallback to default behavior, just delayed
 				}),
 				newSkeletonTestPeerWithHook("peer-2", chain, func(origin uint64) []*types.Header {
-					if origin == chain[2*requestHeaders+2].Number.Uint64() {
+					if origin == chain[2*requestHeaders+1].Number.Uint64() {
 						time.Sleep(100 * time.Millisecond)
 					}
 					return nil // Fallback to default behavior, just delayed