lourenco/mgmt
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

engine: resources: exec: Fix wrong err variable being checked in test

Browse Source
This commit is contained in:
Edward Toroshchyn
2025-02-23 13:51:13 +01:00
committed by James Shubin
parent d56896cb0d
commit 63d7b8e51e
1 changed files with 2 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
engine/resources/exec_test.go
View File

@@ -412,7 +412,7 @@ func TestExecTimeoutBehaviour(t *testing.T) {
} }
exitErr, ok := err.(*exec.ExitError) // embeds an os.ProcessState exitErr, ok := err.(*exec.ExitError) // embeds an os.ProcessState
if err != nil && ok { if ok {
pStateSys := exitErr.Sys() // (*os.ProcessState) Sys pStateSys := exitErr.Sys() // (*os.ProcessState) Sys
wStatus, ok := pStateSys.(syscall.WaitStatus) wStatus, ok := pStateSys.(syscall.WaitStatus)
if !ok { if !ok {
@@ -432,13 +432,8 @@ func TestExecTimeoutBehaviour(t *testing.T) {
t.Logf("exit status: %d", wStatus.ExitStatus()) t.Logf("exit status: %d", wStatus.ExitStatus())
return return
} else if err != nil {
t.Errorf("general cmd error")
return
} }
t.Errorf("general cmd error")
// no error
} }
func TestExecAutoEdge1(t *testing.T) { func TestExecAutoEdge1(t *testing.T) {