testutil: Fix non-standard subtest output
authorRichard Levitte <levitte@openssl.org>
Thu, 11 May 2017 17:32:43 +0000 (19:32 +0200)
committerRichard Levitte <levitte@openssl.org>
Thu, 11 May 2017 18:42:01 +0000 (20:42 +0200)
In some cases, testutil outputs subtests like this:

    1..6 # Subtest: progname

The standard set by Test::More (because there really is no actual
standard yet) gives this display:

    # Subtest: progname
    1..6

Until the standard is actually agreed upon, let's do it like
Test::More.

Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/3446)

test/testutil/driver.c

index 660db7b3270c660da4a83113a1d5d45f5f55b2a3..7be6d2f66431864f5a31801494b1a0479388d5fb 100644 (file)
@@ -134,14 +134,14 @@ int run_tests(const char *test_prog_name)
     char *verdict = NULL;
     int i, j;
 
-    if (num_tests < 1)
+    if (num_tests < 1) {
         test_printf_stdout("%*s1..0 # Skipped: %s\n", level, "",
                            test_prog_name);
-    else if (level > 0)
-        test_printf_stdout("%*s1..%d # Subtest: %s\n", level, "", num_tests,
-                           test_prog_name);
-    else
+    } else {
+        if (level > 0)
+            test_printf_stdout("%*s# Subtest: %s\n", level, "", test_prog_name);
         test_printf_stdout("%*s1..%d\n", level, "", num_tests);
+    }
     test_flush_stdout();
 
     for (i = 0; i != num_tests; ++i) {