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

Commit8df601b

Browse files
committed
Make new authentication test case more robust.
I happened to notice that the new test case I added in b55b4dad9falls over if one runs "make check" repeatedly; though not in branchesafter v10. That's because it was assuming that tmp_check/pgpasswouldn't exist already. However, it's only been since v11 that theMakefiles forcibly remove all of tmp_check/ before starting a TAP run.This fix to unlink the file is therefore strictly necessary only inv10 ... but it seems wisest to do it across the board, rather thanlet the test rely on external logic to get the conditions right.
1 parent9b81a30 commit8df601b

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

‎src/test/authentication/t/001_password.pl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -106,6 +106,7 @@ sub test_role
106106
delete$ENV{"PGCHANNELBINDING"};
107107
$ENV{"PGPASSFILE"} =$pgpassfile;
108108

109+
unlink($pgpassfile);
109110
append_to_file($pgpassfile,qq!
110111
# This very long comment is just here to exercise handling of long lines in the file. This very long comment is just here to exercise handling of long lines in the file. This very long comment is just here to exercise handling of long lines in the file. This very long comment is just here to exercise handling of long lines in the file. This very long comment is just here to exercise handling of long lines in the file.
111112
*:*:postgres:scram_role:pass:this is not part of the password.

0 commit comments

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