TEST: Modify test/recipes/80-test_ssl_new.t to leave artifacts behind
authorRichard Levitte <levitte@openssl.org>
Fri, 14 Feb 2020 07:42:05 +0000 (08:42 +0100)
committerRichard Levitte <levitte@openssl.org>
Tue, 18 Feb 2020 08:45:53 +0000 (09:45 +0100)
Reviewed-by: Paul Dale <paul.dale@oracle.com>
(Merged from https://github.com/openssl/openssl/pull/11080)

test/recipes/80-test_ssl_new.t

index 04a0c13394dca3b7bf059b82f4ef0823bcd98b3b..01a49173d670d5e35d5576103050254a85c2124b 100644 (file)
@@ -120,17 +120,16 @@ sub test_conf {
     my ($conf, $check_source, $skip) = @_;
 
     my $conf_file = srctop_file("test", "ssl-tests", $conf);
-    my $tmp_file = "${conf}.$$.tmp";
+    my $input_file = $conf_file . ".in";
+    my $output_file = $conf;
     my $run_test = 1;
 
   SKIP: {
       # "Test" 1. Generate the source.
-      my $input_file = $conf_file . ".in";
-
       skip 'failure', 2 unless
         ok(run(perltest(["generate_ssl_tests.pl", $input_file],
                         interpreter_args => [ "-I", srctop_dir("util", "perl")],
-                        stdout => $tmp_file)),
+                        stdout => $output_file)),
            "Getting output from generate_ssl_tests.pl.");
 
     SKIP: {
@@ -138,7 +137,7 @@ sub test_conf {
         skip "Skipping generated source test for $conf", 1
           if !$check_source;
 
-        $run_test = is(cmp_text($tmp_file, $conf_file), 0,
+        $run_test = is(cmp_text($output_file, $conf_file), 0,
                        "Comparing generated sources.");
       }
 
@@ -146,10 +145,8 @@ sub test_conf {
       skip "No tests available; skipping tests", 1 if $skip;
       skip "Stale sources; skipping tests", 1 if !$run_test;
 
-      ok(run(test(["ssl_test", $tmp_file])), "running ssl_test $conf");
+      ok(run(test(["ssl_test", $output_file])), "running ssl_test $conf");
     }
-
-    unlink glob $tmp_file;
 }
 
 sub cmp_text {