aboutsummaryrefslogtreecommitdiff
path: root/utils/ports.go
diff options
context:
space:
mode:
authorEd Santiago <santiago@redhat.com>2022-06-13 14:24:12 -0600
committerEd Santiago <santiago@redhat.com>2022-06-14 08:56:28 -0600
commit232b3e342a6fdf14c22dab679a330aa32d8d2829 (patch)
treeb5569b6504b42cb2ef25f251e061025d20434eb2 /utils/ports.go
parentcffed099b8404fa7f136643116020edac2336155 (diff)
downloadpodman-232b3e342a6fdf14c22dab679a330aa32d8d2829.tar.gz
podman-232b3e342a6fdf14c22dab679a330aa32d8d2829.tar.bz2
podman-232b3e342a6fdf14c22dab679a330aa32d8d2829.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 'utils/ports.go')
0 files changed, 0 insertions, 0 deletions