stacklok / toolhive / 20855252142

50%

push

github

Fix race condition in testing (#3239)

Root Cause: The NewBackendWatcher() function was calling ctrl.SetLogger() on every invocation at line 109 in manager.go. Since tests run in parallel (using t.Parallel()), multiple goroutines were concurrently setting the global controller-runtime logger, causing a data race.

Solution: Used sync.Once to ensure the controller-runtime logger is set exactly once across all invocations, even when multiple BackendWatcher instances are created concurrently.

3 of 3 new or added lines in 1 file covered. (100.0%)

29141 of 58202 relevant lines covered (50.07%)

58.14 hits per line