refactoring, fix handling of PG_BIN & PG_CONFIG, fix test_reload()

Author: funbringer

run_tests.sh

@@ -12,6 +12,11 @@ else
 	PIP=pip3
 fi
 
+
+# check that pg_config exists
+command -v pg_config
+
+
 # prepare environment
 VENV_PATH=/tmp/testgres_venv
 rm -rf $VENV_PATH
@@ -29,12 +34,32 @@ $PIP install .
 # test code quality
 flake8 .
 
-# run tests
-coverage run tests/test_simple.py
+
+# remove existing coverage file
+export COVERAGE_FILE=.coverage
+rm -f $COVERAGE_FILE
+
+
+# run tests (PATH)
+time coverage run -a tests/test_simple.py
+
+
+# run tests (PG_BIN)
+export PG_BIN=$(dirname $(which pg_config))
+time coverage run -a tests/test_simple.py
+unset PG_BIN
+
+
+# run tests (PG_CONFIG)
+export PG_CONFIG=$(which pg_config)
+time coverage run -a tests/test_simple.py
+unset PG_CONFIG
+
 
 # show coverage
 coverage report
 
+
 # attempt to fix codecov
 set +eux
 

testgres/backup.py

@@ -12,6 +12,7 @@
 from .exceptions import BackupException
 
 from .utils import \
+    get_bin_path, \
     default_username as _default_username, \
     execute_utility as _execute_utility, \
     explain_exception as _explain_exception
@@ -48,10 +49,11 @@ def __init__(self,
 
         data_dir = os.path.join(self.base_dir, _DATA_DIR)
         _params = [
+            get_bin_path("pg_basebackup"),
             "-D{}".format(data_dir), "-p{}".format(node.port),
             "-U{}".format(username), "-X{}".format(xlog_method)
         ]
-        _execute_utility("pg_basebackup", _params, self.log_file)
+        _execute_utility(_params, self.log_file)
 
     def __enter__(self):
         return self