Prevent 006_transfer_modes.pl from leaving files behind.
authorNathan Bossart <nathan@postgresql.org>
Tue, 8 Apr 2025 15:57:31 +0000 (10:57 -0500)
committerNathan Bossart <nathan@postgresql.org>
Tue, 8 Apr 2025 15:57:31 +0000 (10:57 -0500)
This test was leaving files like delete_old_cluster.{sh,bat} in the
source directory for VPATH and meson builds.  To fix, change the
directory to tmp_check before running the test, as was done in
commits 15b6d215538af917be6b, and c462b054ba.

Oversight in commit af0d4901c1.

Reported-by: Andrew Dunstan <andrew@dunslane.net> (on Discord)
Reviewed-by: Michael Paquier <michael@paquier.xyz>
Reviewed-by: Andrew Dunstan <andrew@dunslane.net>
Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/Z_RHkG770w3SE0yU%40nathan

src/bin/pg_upgrade/t/006_transfer_modes.pl

index 34fddbcdab57b239f997e6ed89c6096d230a7d79..550a63fdf7d477a727b5821fdce57c77ca91b5c0 100644 (file)
@@ -102,6 +102,11 @@ sub test_mode
    $new->clean_node();
 }
 
+# Run pg_upgrade in tmp_check to avoid leaving files like
+# delete_old_cluster.{sh,bat} in the source directory for VPATH and meson
+# builds.
+chdir ${PostgreSQL::Test::Utils::tmp_check};
+
 test_mode('--clone');
 test_mode('--copy');
 test_mode('--copy-file-range');