Skip to content

Commit d0cf961

Browse files
committed
device: limit allowedip fuzzer a to 4 times through
Trying this for every peer winds up being very slow and precludes it from acceptable runtime in the CI, so reduce this to 4. Signed-off-by: Jason A. Donenfeld <[email protected]>
1 parent 841756e commit d0cf961

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

device/allowedips_rand_test.go

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,10 @@ import (
1313
)
1414

1515
const (
16-
NumberOfPeers = 100
17-
NumberOfAddresses = 250
18-
NumberOfTests = 10000
16+
NumberOfPeers = 100
17+
NumberOfPeerRemovals = 4
18+
NumberOfAddresses = 250
19+
NumberOfTests = 10000
1920
)
2021

2122
type SlowNode struct {
@@ -103,7 +104,8 @@ func TestTrieRandom(t *testing.T) {
103104
slow6 = slow6.Insert(addr6[:], cidr, peers[index])
104105
}
105106

106-
for p := 0; ; p++ {
107+
var p int
108+
for p = 0; ; p++ {
107109
for n := 0; n < NumberOfTests; n++ {
108110
var addr4 [4]byte
109111
rand.Read(addr4[:])
@@ -121,13 +123,16 @@ func TestTrieRandom(t *testing.T) {
121123
t.Errorf("Trie did not match naive implementation, for %v: want %p, got %p", net.IP(addr6[:]), peer1, peer2)
122124
}
123125
}
124-
if p >= len(peers) {
126+
if p >= len(peers) || p >= NumberOfPeerRemovals {
125127
break
126128
}
127129
allowedIPs.RemoveByPeer(peers[p])
128130
slow4 = slow4.RemoveByPeer(peers[p])
129131
slow6 = slow6.RemoveByPeer(peers[p])
130132
}
133+
for ; p < len(peers); p++ {
134+
allowedIPs.RemoveByPeer(peers[p])
135+
}
131136

132137
if allowedIPs.IPv4 != nil || allowedIPs.IPv6 != nil {
133138
t.Error("Failed to remove all nodes from trie by peer")

0 commit comments

Comments
 (0)