op-e2e: Don't double peer (ethereum-optimism#9423)

trianglesphere · web-flow · commit 37647587b59e · 2024-02-07T21:13:28.000Z
diff --git a/op-e2e/actions/sync_test.go b/op-e2e/actions/sync_test.go
@@ -187,7 +187,6 @@ func TestELSync(gt *testing.T) {
 	}
 
 	// Wait longer to peer. This tests flakes or takes a long time when the op-geth instances are not able to peer.
-	seqEng.AddPeers(verEng.Enode())
 	verEng.AddPeers(seqEng.Enode())
 
 	// Insert it on the verifier
@@ -201,7 +200,7 @@ func TestELSync(gt *testing.T) {
 		func() bool {
 			return seqEng.PeerCount() > 0 && verEng.PeerCount() > 0
 		},
-		60*time.Second, 1500*time.Millisecond,
+		120*time.Second, 1500*time.Millisecond,
 		"Sequencer & Verifier must peer with each other for snap sync to work",
 	)
 

Original file line number	Diff line number	Diff line change
`@@ -187,7 +187,6 @@ func TestELSync(gt *testing.T) {`
`187`	`187`	`}`
`188`	`188`
`189`	`189`	`// Wait longer to peer. This tests flakes or takes a long time when the op-geth instances are not able to peer.`
`190`		`- seqEng.AddPeers(verEng.Enode())`
`191`	`190`	`verEng.AddPeers(seqEng.Enode())`
`192`	`191`
`193`	`192`	`// Insert it on the verifier`
`@@ -201,7 +200,7 @@ func TestELSync(gt *testing.T) {`
`201`	`200`	`func() bool {`
`202`	`201`	`return seqEng.PeerCount() > 0 && verEng.PeerCount() > 0`
`203`	`202`	`},`
`204`		`- 60time.Second, 1500time.Millisecond,`
	`203`	`+ 120time.Second, 1500time.Millisecond,`
`205`	`204`	`"Sequencer & Verifier must peer with each other for snap sync to work",`
`206`	`205`	`)`
`207`	`206`