diff options
author | Ed Santiago <santiago@redhat.com> | 2022-06-13 14:24:12 -0600 |
---|---|---|
committer | Matthew Heon <mheon@redhat.com> | 2022-06-14 16:12:11 -0400 |
commit | 0e535962714aede07bc67f10503a8a5746e72920 (patch) | |
tree | c547166f591900972fd3d483831afc8474994019 /vendor/github.com/davecgh | |
parent | 96ccd6934b8f23d5af51514b136ed802647dc47a (diff) | |
download | podman-0e535962714aede07bc67f10503a8a5746e72920.tar.gz podman-0e535962714aede07bc67f10503a8a5746e72920.tar.bz2 podman-0e535962714aede07bc67f10503a8a5746e72920.zip |
buildah-bud tests: add arg sanity check
Fix bad design decision (mine) by adding a simple usage check to 'skip'
and 'skip_if_remote' functions: if invoked without test-name args,
fail loudly and immediately.
Background: yeah, their usage is not intuitive. Making the first arg
be a comment helps with _reading_ the code, but not _writing_ new
additions. A developer in a hurry could write "skip this-test" and,
until now, that would be a silent NOP.
Tested by adding broken skip/skip_if_remote calls inline; I confirm
that the line number and funcname usage is correct.
Signed-off-by: Ed Santiago <santiago@redhat.com>
Diffstat (limited to 'vendor/github.com/davecgh')
0 files changed, 0 insertions, 0 deletions