Przeglądaj źródła

p2p/discover: attempt to deflake TestUDP_responseTimeouts

The test expected the timeout to fire after a matcher for the response
was added, but the timeout is random and fired sooner sometimes.
Felix Lange 10 lat temu
rodzic
commit
b3f1f4c673
1 zmienionych plików z 2 dodań i 1 usunięć
  1. 2 1
      p2p/discover/udp_test.go

+ 2 - 1
p2p/discover/udp_test.go

@@ -167,16 +167,17 @@ func TestUDP_responseTimeouts(t *testing.T) {
 		binary.BigEndian.PutUint64(p.from[:], uint64(i))
 		if p.ptype <= 128 {
 			p.errc = timeoutErr
+			test.udp.addpending <- p
 			nTimeouts++
 		} else {
 			p.errc = nilErr
+			test.udp.addpending <- p
 			time.AfterFunc(randomDuration(60*time.Millisecond), func() {
 				if !test.udp.handleReply(p.from, p.ptype, nil) {
 					t.Logf("not matched: %v", p)
 				}
 			})
 		}
-		test.udp.addpending <- p
 		time.Sleep(randomDuration(30 * time.Millisecond))
 	}