fix: runCommand returns non-zero exit code for non-existing exes

[miki725]

• Followed the steps in the contributor's guide: https://crashoverride.com/docs/other/contributing#filing-the-pull-request
• PR title uses semantic commit messages
• Filled out the template to a useful degree
• Updated CHANGELOG.md if necessary

Issue

crashappsec/nimutils#65

Description

Running chalk docker ... when docker is not installed would not exit with appropriate exit code.

Testing

run chalk docker ... without docker being on PATH

[ee7]

If it's trivial, could you add a test? Looks good to me otherwise.

Before this PR:

$ git log -1 --oneline
bafac54 (HEAD -> main, crashappsec/main) fix: error docker inspect logs during chalk exec (#248)
$ PATH='' chalk docker --version
error: docker command is missing. chalk requires docker binary installed to wrap docker commands.
$ echo $?
0 

With this PR:

$ git log -1 --oneline
81d5d9c (HEAD -> ms/exit, crashappsec/ms/exit) fix: runCommand returns non-zero exit code for non-existing exes
$ PATH='' chalk docker --version
error: docker command is missing. chalk requires docker binary installed to wrap docker commands.
docker: No such file or directory
$ echo $?
1

[miki725]

this was an omission in nimutils. we probably need a whole bunch of unit tests there. will add simple integration tests here for now I guess then

[ee7]

Yeah - just asserting the exit code of e.g. chalk docker --version when docker isn't in the PATH would be great, thanks.