test: add test for concurrent statistics sampling

The test currently fails as `PacketsSent` is not protected by `statsMu` in `sendICMP`. Signed-off-by: Per Hallgren <[email protected]>
prometheus-community · Feb 3, 2025 · 38075b8 · 38075b8
1 parent 50c55f1
commit 38075b8
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/ping_test.go b/ping_test.go
@@ -870,3 +870,11 @@ func TestSetResolveTimeout(t *testing.T) {
 	err = p.SetAddr("127.0.0.1")
 	AssertNoError(t, err)
 }
+
+func TestRunStatisticsConcurrent(t *testing.T) {
+	p := New("www.google.com")
+	p.Count = 1
+	p.Interval = time.Millisecond
+	go p.Statistics()
+	p.Run()
+}