API v2 tests: usability improvements

* Allow for descriptive comment in 't' invocations, making it
  easier to distinguish similar requests
* Include test file basename (eg 40-pods) in 'ok/not ok' line
* Always symlink $TMPDIR/test-apiv2.log to latest YYMMDDetc file
* Include test result ('ok', 'not ok') in said log
* When curl results are JSON, filter them through jq into log

Signed-off-by: Ed Santiago <santiago@redhat.com>

1 files changed, 14 insertions, 10 deletions

diff --git a/test/apiv2/40-pods.at b/test/apiv2/40-pods.at
index 70d9f8203..982100396 100644
--- a/test/apiv2/40-pods.at
+++ b/test/apiv2/40-pods.at
@@ -3,7 +3,8 @@
 # test pod-related endpoints
 #
 
-t GET  libpod/pods/json            200 null
+t GET  "libpod/pods/json (clean slate at start)"   200 null
+
 t POST libpod/pods/create name=foo 201 .id~[0-9a-f]\\{64\\}
 pod_id=$(jq -r .id <<<"$output")
 t GET  libpod/pods/foo/exists      204
@@ -19,7 +20,8 @@ t GET  libpod/pods/json            200 \
   .[0].Containers\|length=1
 
 # Cannot create a dup pod with the same name
-t POST libpod/pods/create name=foo 409 .cause="pod already exists"
+t POST "libpod/pods/create (dup pod)" name=foo 409 \
+  .cause="pod already exists"
 
 #t POST libpod/pods/create a=b 400 .cause='bad parameter'  # FIXME: unimplemented
 
@@ -27,16 +29,18 @@ if root || have_cgroupsv2; then
     t POST libpod/pods/foo/pause   '' 200
 else
     # Rootless cgroupsv1 : unsupported
-    t POST libpod/pods/foo/pause '' 500 \
+    t POST "libpod/pods/foo/pause (rootless cgroups v1)" '' 500 \
       .cause="this container does not have a cgroup" \
       .message~".*pause pods containing rootless containers with cgroup V1"
 fi
-t POST libpod/pods/foo/unpause '' 200
-t POST libpod/pods/foo/unpause '' 200   # (2nd time)
-t POST libpod/pods/foo/stop    '' 304
-t POST libpod/pods/foo/restart '' 200
+t POST  libpod/pods/foo/unpause '' 200
+t POST "libpod/pods/foo/unpause (2nd unpause in a row)" '' 200
+t POST "libpod/pods/foo/stop (pod is already stopped)"  '' 304
+t POST  libpod/pods/foo/restart '' 200 \
+  .Errs=null \
+  .Id=$pod_id
 
-t POST libpod/pods/bar/restart '' 404
+t POST  "libpod/pods/bar/restart (restart on nonexistent pod)" '' 404
 
 # FIXME: I'm not sure what 'prune' is supposed to do; as of 20200224 it
 # just returns 200 (ok) with empty result list.
@@ -44,7 +48,7 @@ t POST libpod/pods/bar/restart '' 404
 #t POST libpod/pods/prune 'a=b' 400     # FIXME: 2020-02-24 returns 200
 
 # Clean up; and try twice, making sure that the second time fails
-t DELETE libpod/pods/foo 200
-t DELETE libpod/pods/foo 404
+t DELETE  libpod/pods/foo 200
+t DELETE "libpod/pods/foo (pod has already been deleted)" 404
 
 # vim: filetype=sh