fix: lint violations

Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
This commit is contained in:
Mark Sagi-Kazar 2024-02-15 12:07:55 +01:00 committed by Márk Sági-Kazár
parent 6c6bd7c0b5
commit 030b77a720

View file

@ -2427,7 +2427,7 @@ func TestWatchFile(t *testing.T) {
wg := sync.WaitGroup{} wg := sync.WaitGroup{}
wg.Add(1) wg.Add(1)
var wgDoneOnce sync.Once // OnConfigChange is called twice on Windows var wgDoneOnce sync.Once // OnConfigChange is called twice on Windows
v.OnConfigChange(func(in fsnotify.Event) { v.OnConfigChange(func(_ fsnotify.Event) {
t.Logf("config file changed") t.Logf("config file changed")
wgDoneOnce.Do(func() { wgDoneOnce.Do(func() {
wg.Done() wg.Done()
@ -2450,7 +2450,7 @@ func TestWatchFile(t *testing.T) {
v, watchDir, _ := newViperWithSymlinkedConfigFile(t) v, watchDir, _ := newViperWithSymlinkedConfigFile(t)
wg := sync.WaitGroup{} wg := sync.WaitGroup{}
v.WatchConfig() v.WatchConfig()
v.OnConfigChange(func(in fsnotify.Event) { v.OnConfigChange(func(_ fsnotify.Event) {
t.Logf("config file changed") t.Logf("config file changed")
wg.Done() wg.Done()
}) })
@ -2689,8 +2689,6 @@ func BenchmarkGetBoolFromMap(b *testing.B) {
} }
// Skip some tests on Windows that kept failing when Windows was added to the CI as a target. // Skip some tests on Windows that kept failing when Windows was added to the CI as a target.
//
//nolint:gocritic // sloppyTestFuncName
func skipWindows(t *testing.T) { func skipWindows(t *testing.T) {
if runtime.GOOS == "windows" { if runtime.GOOS == "windows" {
t.Skip("Skip test on Windows") t.Skip("Skip test on Windows")