summaryrefslogtreecommitdiff
path: root/contrib/script
diff options
context:
space:
mode:
authorEd Santiago <santiago@redhat.com>2019-06-18 11:18:53 -0600
committerEd Santiago <santiago@redhat.com>2019-06-18 15:07:53 -0600
commit40eb0453adb85649daacf822ccbb81ade33a97e2 (patch)
tree5ac86aab983235d83db3eb3c3c9e7e2efc7a5674 /contrib/script
parent265d55edbf698b624adee081a48f47b3b73b6164 (diff)
downloadpodman-40eb0453adb85649daacf822ccbb81ade33a97e2.tar.gz
podman-40eb0453adb85649daacf822ccbb81ade33a97e2.tar.bz2
podman-40eb0453adb85649daacf822ccbb81ade33a97e2.zip
fix broken healthcheck tests
Four of the healthcheck tests were completely broken. They were written with the option '--healthcheck-cmd' which is not an option (it should be '--healthcheck-command', with 'command' as a full word). The tests were merely checking exit code, not error message, so of course they failed. I have fixed the command line and added checks for the expected diagnostic. (Side note: do not write tests that check exit code but nothing else. This should not need to be said). One of the four tests was invalid: --healthcheck-interval 0.5s. Per Brent: initially i was going to restrict sub one-second intervals That test has been removed. It would probably be a good idea for a future PR to add some validation such as preventing negative values, but that's left as an exercise for later. Also: grammar fix in an error message. Caught by my ginkgo log greasemonkey script, which highlights 'Error' messages and grabbed my attention. Signed-off-by: Ed Santiago <santiago@redhat.com>
Diffstat (limited to 'contrib/script')
0 files changed, 0 insertions, 0 deletions