From f5317bdf87d8e86dc55c19bfce99036142fee0ca Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ond=C5=99ej=20Sloup?= Date: Thu, 11 Aug 2022 12:20:48 +0200 Subject: [PATCH] Fix permission issue with tmp folder Due to permission issues with creating a temporary folder, the two custom tests from PostgreSQL documentation failed as they could not write their results to the temp log file. The issue was resolved by changing the owner of the temporary folder while creating it. While looking for the issue, the varDump command was commented out as it was not needed and not even found. Also, two tests taken from the documentation were edited; therefore, the slight changes which were made to them to work in our scenario can be highlighted. --- Sanity/libecpg/runtest.sh | 3 ++- Sanity/libecpg/testecpg-decimal.pgc | 4 +++- Sanity/libecpg/testecpg-interval.pgc | 4 +++- 3 files changed, 8 insertions(+), 3 deletions(-) diff --git a/Sanity/libecpg/runtest.sh b/Sanity/libecpg/runtest.sh index b89d652..114ee51 100755 --- a/Sanity/libecpg/runtest.sh +++ b/Sanity/libecpg/runtest.sh @@ -43,11 +43,12 @@ rlJournalStart done rlRun "TestDir=$PWD" rlRun "TmpDir=\$(mktemp -d)" 0 "Creating tmp directory" + rlRun "chown postgres:postgres $TmpDir" rlRun "pushd $TmpDir" rlPhaseEnd rlPhaseStartSetup "Database setup" - varDump postgresqlDataDir; + # varDump postgresqlDataDir; [[ -d $postgresqlDataDir ]] && rlFileBackup $postgresqlDataDir; postgresqlCleanup; postgresqlInitDB; diff --git a/Sanity/libecpg/testecpg-decimal.pgc b/Sanity/libecpg/testecpg-decimal.pgc index d22a77c..7bc7151 100644 --- a/Sanity/libecpg/testecpg-decimal.pgc +++ b/Sanity/libecpg/testecpg-decimal.pgc @@ -1,4 +1,4 @@ -// Taken from https://www.postgresql.org/docs/13/ecpg-variables.html +// Taken from https://www.postgresql.org/docs/14/ecpg-variables.html #include #include @@ -15,7 +15,9 @@ EXEC SQL BEGIN DECLARE SECTION; decimal *dec; EXEC SQL END DECLARE SECTION; +// Add "USER postgres;" EXEC SQL CONNECT TO testdb USER postgres; +// EXEC SQL SELECT pg_catalog.set_config('search_path', '', false); EXEC SQL COMMIT; num = PGTYPESnumeric_new(); dec = PGTYPESdecimal_new(); diff --git a/Sanity/libecpg/testecpg-interval.pgc b/Sanity/libecpg/testecpg-interval.pgc index d065911..5fa5b75 100644 --- a/Sanity/libecpg/testecpg-interval.pgc +++ b/Sanity/libecpg/testecpg-interval.pgc @@ -1,4 +1,4 @@ -// Taken from https://www.postgresql.org/docs/13/ecpg-variables.html +// Taken from https://www.postgresql.org/docs/14/ecpg-variables.html #include #include @@ -11,7 +11,9 @@ EXEC SQL BEGIN DECLARE SECTION; interval *in; EXEC SQL END DECLARE SECTION; +// Add "USER postgres;" EXEC SQL CONNECT TO testdb USER postgres; +// EXEC SQL SELECT pg_catalog.set_config('search_path', '', false); EXEC SQL COMMIT; in = PGTYPESinterval_new(); EXEC SQL SELECT '1 min'::interval INTO :in;