fixup! fixup! Fix data races in unit tests

Author: pchila

internal/pkg/agent/application/upgrade/watcher_test.go

@@ -1223,20 +1223,4 @@ func isProcessRunning(t *testing.T, cmd *exec.Cmd) (bool, error) {
 	}
 
 	return isProcessLive(cmd.Process)
-
-	//// if process is not nil we need to split between unix and non-unix OSes
-	//if runtime.GOOS == "windows" {
-	//	t.Logf("pid %d is running because os.GetProcess returned a not nil process on windows", cmd.Process.Pid)
-	//	return true, nil
-	//} else {
-	//	// on unix system we always get a process back, we need to do some further checks
-	//	signalErr := cmd.Process.Signal(syscall.Signal(0))
-	//	if signalErr != nil {
-	//		t.Logf("pid %d is not running because Signal(0) returned an error on unix", cmd.Process.Pid)
-	//		return false, nil //nolint:nilerr // if we receive an error it means that the process is not running, so the check completed without errors
-	//	} else {
-	//		t.Logf("pid %d is running because Signal(0) succeeded on unix", cmd.Process.Pid)
-	//		return true, nil
-	//	}
-	//}
 }