Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit4b15868

Browse files
committed
Make PG_TEST_NOCLEAN work for temporary directories in TAP tests
When set, this environment variable was only effective for datadirectories but not for all the other temporary files created byPostgreSQL::Test::Utils. Keeping the temporary files after a successfulrun can be useful for debugging purposes.The documentation is updated to reflect the new behavior, with contentsavailable in doc/ since v16 and in src/test/perl/README since v15.Author: Jacob ChampionReviewed-by: Daniel GustafssonDiscussion:https://postgr.es/m/CAAWbhmgHtDH1SGZ+Fw05CsXtE0mzTmjbuUxLB9mY9iPKgM6cUw@mail.gmail.comDiscussion:https://postgr.es/m/YyPd9unV14SX2bLF@paquier.xyzBackpatch-through: 11
1 parentf50200c commit4b15868

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

‎src/test/perl/PostgreSQL/Test/Utils.pm‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -268,7 +268,7 @@ sub all_tests_passing
268268
269269
Securely create a temporary directory insideC<$tmp_check>, likeC<mkdtemp>,
270270
and return its name. The directory will be removed automatically at the
271-
end of the tests.
271+
end of the tests, unless the environment variable PG_TEST_NOCLEAN is provided.
272272
273273
IfC<prefix> is given, the new directory is templated asC<${prefix}_XXXX>.
274274
Otherwise the template isC<tmp_test_XXXX>.
@@ -282,7 +282,7 @@ sub tempdir
282282
return File::Temp::tempdir(
283283
$prefix .'_XXXX',
284284
DIR=>$tmp_check,
285-
CLEANUP=>1);
285+
CLEANUP=>notdefined$ENV{'PG_TEST_NOCLEAN'});
286286
}
287287

288288
=pod
@@ -297,7 +297,8 @@ name, to avoid path length issues.
297297
subtempdir_short
298298
{
299299

300-
return File::Temp::tempdir(CLEANUP=> 1);
300+
return File::Temp::tempdir(
301+
CLEANUP=>notdefined$ENV{'PG_TEST_NOCLEAN'});
301302
}
302303

303304
=pod

‎src/test/perl/README‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,9 @@ some lesser number of seconds.
3131

3232
Data directories will also be left behind for analysis when a test fails;
3333
they are named according to the test filename. But if the environment
34-
variable PG_TEST_NOCLEAN is set, data directories will be retained
35-
regardless of test status.
34+
variable PG_TEST_NOCLEAN is set, the data directories will be retained
35+
regardless of test status. This environment variable also prevents the
36+
test's temporary directories from being removed.
3637

3738

3839
Writing tests

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp