adding better error reporting for bin tests (#1578)

forgot a few things

Signed-off-by: cdleu430 <[email protected]>

src/test/bin/test_exr2aces.py

@@ -16,31 +16,31 @@
 # no args = usage message, error
 result = run ([exr2aces], stdout=PIPE, stderr=PIPE, universal_newlines=True)
 print(" ".join(result.args))
-assert(result.returncode != 0)
-assert(result.stderr.startswith ("Usage: "))
+assert(result.returncode != 0), "\n"+result.stderr
+assert(result.stderr.startswith ("Usage: ")), "\n"+result.stderr
 
 # -h = usage message
 result = run ([exr2aces, "-h"], stdout=PIPE, stderr=PIPE, universal_newlines=True)
 print(" ".join(result.args))
-assert(result.returncode == 0)
-assert(result.stdout.startswith ("Usage: "))
+assert(result.returncode == 0), "\n"+result.stderr
+assert(result.stdout.startswith ("Usage: ")), "\n"+result.stdout
 
 result = run ([exr2aces, "--help"], stdout=PIPE, stderr=PIPE, universal_newlines=True)
 print(" ".join(result.args))
-assert(result.returncode == 0)
-assert(result.stdout.startswith ("Usage: "))
+assert(result.returncode == 0), "\n"+result.stderr
+assert(result.stdout.startswith ("Usage: ")), "\n"+result.stdout
 
 # --version
 result = run ([exr2aces, "--version"], stdout=PIPE, stderr=PIPE, universal_newlines=True)
 print(" ".join(result.args))
-assert(result.returncode == 0)
-assert(result.stdout.startswith ("exr2aces"))
-assert(version in result.stdout)
+assert(result.returncode == 0), "\n"+result.stderr
+assert(result.stdout.startswith ("exr2aces")), "\n"+result.stdout
+assert(version in result.stdout), "\n"+result.stdout
 
 # invalid arguments
 result = run ([exr2aces, "foo.exr", "bar.exr"], stdout=PIPE, stderr=PIPE, universal_newlines=True)
 print(" ".join(result.args))
-assert(result.returncode != 0)
+assert(result.returncode != 0), "\n"+result.stderr
 
 def find_line(keyword, lines):
     for line in lines:
@@ -58,14 +58,14 @@ def cleanup():
 image = f"{image_dir}/TestImages/GrayRampsHorizontal.exr"
 result = run ([exr2aces, "-v", image, outimage], stdout=PIPE, stderr=PIPE, universal_newlines=True)
 print(" ".join(result.args))
-assert(result.returncode == 0)
+assert(result.returncode == 0), "\n"+result.stderr
 
 result = run ([exrinfo, "-v", outimage], stdout=PIPE, stderr=PIPE, universal_newlines=True)
 print(" ".join(result.args))
-assert(result.returncode == 0)
+assert(result.returncode == 0), "\n"+result.stderr
 
 # confirm the output has the proper chromaticities
-assert("chromaticities: chromaticities r[0.7347, 0.2653] g[0, 1] b[0.0001, -0.077] w[0.32168, 0.33767]" in result.stdout)
+assert("chromaticities: chromaticities r[0.7347, 0.2653] g[0, 1] b[0.0001, -0.077] w[0.32168, 0.33767]" in result.stdout), "\n"+result.stdout
 
 print("success")
 

src/test/bin/test_exrcheck.py

@@ -7,7 +7,7 @@
 from subprocess import PIPE, run
 
 print(f"testing exrcheck: {' '.join(sys.argv)}")
-      
+
 exrcheck = sys.argv[1]
 image_dir = sys.argv[2]
 
@@ -17,28 +17,23 @@
 
     result = run ([exrcheck, exr_path], stdout=PIPE, stderr=PIPE, universal_newlines=True)
     print(" ".join(result.args))
-    print(result.stderr)
-    assert(result.returncode == 0)
+    assert(result.returncode == 0), "\n"+result.stderr
 
     result = run ([exrcheck, "-m", exr_path], stdout=PIPE, stderr=PIPE, universal_newlines=True)
     print(" ".join(result.args))
-    print(result.stderr)
-    assert(result.returncode == 0)
+    assert(result.returncode == 0), "\n"+result.stderr
 
     result = run ([exrcheck, "-t", exr_path], stdout=PIPE, stderr=PIPE, universal_newlines=True)
     print(" ".join(result.args))
-    print(result.stderr)
-    assert(result.returncode == 0)
+    assert(result.returncode == 0), "\n"+result.stderr
 
     result = run ([exrcheck, "-s", exr_path], stdout=PIPE, stderr=PIPE, universal_newlines=True)
     print(" ".join(result.args))
-    print(result.stderr)
-    assert(result.returncode == 0)
+    assert(result.returncode == 0), "\n"+result.stderr
 
     result = run ([exrcheck, "-c", exr_path], stdout=PIPE, stderr=PIPE, universal_newlines=True)
     print(" ".join(result.args))
-    print(result.stderr)
-    assert(result.returncode == 0)
+    assert(result.returncode == 0), "\n"+result.stderr
 
 
 print("success.")