Skip to content
This repository has been archived by the owner on Aug 19, 2022. It is now read-only.

Commit

Permalink
fix flaky EWMA test (#205)
Browse files Browse the repository at this point in the history
  • Loading branch information
marten-seemann committed Aug 12, 2022
1 parent caf30df commit 320a84f
Showing 1 changed file with 10 additions and 11 deletions.
21 changes: 10 additions & 11 deletions metrics_test.go
Expand Up @@ -2,7 +2,6 @@ package peerstore

import (
"fmt"
"math"
"math/rand"
"testing"
"time"
Expand Down Expand Up @@ -44,21 +43,21 @@ func TestLatencyEWMA(t *testing.T) {
t.Fatal(err)
}

exp := 100.0
mu := exp
sig := 10.0
next := func() time.Duration {
mu := (rand.NormFloat64() * sig) + mu
return time.Duration(mu)
}
const exp = 100
const mu = exp
const sig = 10
next := func() time.Duration { return time.Duration(rand.Intn(20) - 10 + mu) }

for i := 0; i < 10; i++ {
time.Sleep(200 * time.Millisecond)
m.RecordLatency(id, next())
}

lat := m.LatencyEWMA(id)
if math.Abs(exp-float64(lat)) > sig {
t.Fatal("latency outside of expected range: ", exp, lat, sig)
diff := exp - lat
if diff < 0 {
diff = -diff
}
if diff > sig {
t.Fatalf("latency outside of expected range. expected %d ± %d, got %d", exp, sig, lat)
}
}

0 comments on commit 320a84f

Please sign in to comment.