postgres.git / summary / log / commit / refs

commit    f44b9b625bedd8e0bca67b3b42ba10ce482fa31b
Author:   Thomas Munro <tmunro@postgresql.org>
Date:     Mon Aug 03 00:39:15 2020 +0000
First released in: 14.0, 11.9, 12.4, 13.0

    Fix rare failure in LDAP tests.
    
    Instead of writing a query to psql's stdin, use -c.  This avoids a
    failure where psql exits before we write, seen a few times on the build
    farm.  Thanks to Tom Lane for the suggestion.
    
    Back-patch to 11, where the LDAP tests arrived.
    
    Reviewed-by: Noah Misch <noah@leadboat.com>
    Discussion: https://postgr.es/m/CA%2BhUKGLFmW%2BHQYPeKiwSp5sdFFHtFViCpw4Mh6yAgEx74r5-Cw%40mail.gmail.com


 src/test/ldap/t/001_auth.pl | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/test/ldap/t/001_auth.pl b/src/test/ldap/t/001_auth.pl
index f8941144f5e..3bc7672451e 100644
--- a/src/test/ldap/t/001_auth.pl
+++ b/src/test/ldap/t/001_auth.pl
@@ -165,7 +165,8 @@ sub test_access
 	my ($node, $role, $expected_res, $test_name) = @_;
 
 	my $res =
-	  $node->psql('postgres', 'SELECT 1', extra_params => [ '-U', $role ]);
+	  $node->psql('postgres', undef,
+				  extra_params => [ '-U', $role, '-c', 'SELECT 1' ]);
 	is($res, $expected_res, $test_name);
 	return;
 }


[parent: 63e9aa6879cc]