Address data race in pkg/healtcheck

pkg/healthcheck/check.go

@@ -6,6 +6,7 @@ import (
 	"fmt"
 	"io"
 	"strings"
+	"sync"
 )
 
 type (
@@ -28,6 +29,7 @@ type (
 
 	results []*result
 	checks  struct {
+		l  sync.RWMutex
 		cc []*check
 	}
 )
@@ -50,10 +52,16 @@ func New() *checks {
 
 // Add appends new check
 func (c *checks) Add(fn checkFn, label string, description ...string) {
+	c.l.Lock()
+	defer c.l.Unlock()
+
 	c.cc = append(c.cc, &check{fn, &Meta{Label: label, Description: strings.Join(description, "")}})
 }
 
-func (c checks) Run(ctx context.Context) results {
+func (c *checks) Run(ctx context.Context) results {
+	c.l.RLock()
+	defer c.l.RUnlock()
+
 	var rr = make([]*result, len(c.cc))
 
 	for i, c := range c.cc {

pkg/healthcheck/check_test.go

@@ -66,3 +66,17 @@ func Test_Healthy(t *testing.T) {
 		})
 	}
 }
+
+// tested with
+// go test -count 10 -race -run Test_DataRace ./pkg/healthcheck/...
+func Test_DataRace(t *testing.T) {
+	hc := New()
+
+	go func() {
+		hc.Add(func(ctx context.Context) error { return nil }, "hc1")
+		hc.Add(func(ctx context.Context) error { return nil }, "hc2")
+		hc.Add(func(ctx context.Context) error { return nil }, "hc3")
+	}()
+	go hc.Run(context.Background())
+
+}