TST: add PGPORT env var, rewrite setup_postgres.sh to scripts dir (#1932)

.github/workflows/tests.yaml

@@ -110,10 +110,11 @@ jobs:
           PGUSER: postgres
           PGPASSWORD: postgres
           PGHOST: "127.0.0.1"
+          PGPORT: 5432
         run: |
           source activate test
           conda install postgis -c conda-forge
-          source ci/envs/setup_postgres.sh
+          sh ci/scripts/setup_postgres.sh
           pytest -v -r s --color=yes --cov=geopandas --cov-append --cov-report term-missing --cov-report xml geopandas/io/tests/test_sql.py | tee /dev/stderr | if grep SKIPPED >/dev/null;then echo "TESTS SKIPPED, FAILING" && exit 1;fi
 
       - name: Test docstrings

ci/envs/setup_postgres.sh

@@ -1,21 +0,0 @@
-#!/bin/bash -e
-
-echo "Setting up Postgresql"
-
-mkdir -p ${HOME}/var
-rm -rf ${HOME}/var/db
-
-pg_ctl initdb -D ${HOME}/var/db
-pg_ctl start -D ${HOME}/var/db
-
-echo -n 'waiting for postgres'
-while [ ! -e /tmp/.s.PGSQL.5432 ]; do
-    sleep 1
-    echo -n '.'
-done
-
-createuser -U ${USER} -s postgres
-createdb --owner=postgres test_geopandas
-psql -d test_geopandas -q -c "CREATE EXTENSION postgis"
-
-echo "Done setting up Postgresql"

ci/scripts/setup_postgres.sh

@@ -0,0 +1,33 @@
+#!/bin/sh
+set -e
+
+if [ -z "${PGUSER}" ] || [ -z "${PGPORT}" ]; then
+    echo "Environment variables PGUSER and PGPORT must be set"
+    exit 1
+fi
+
+PGDATA=$(mktemp -d /tmp/postgres.XXXXXX)
+echo "Setting up PostgreSQL in ${PGDATA} on port ${PGPORT}"
+
+pg_ctl -D ${PGDATA} initdb
+pg_ctl -D ${PGDATA} start
+
+SOCKETPATH="/tmp/.s.PGSQL.${PGPORT}"
+echo -n 'waiting for postgres'
+while [ ! -e ${SOCKETPATH} ]; do
+    sleep 1
+    echo -n '.'
+done
+echo
+
+echo "Done setting up PostgreSQL. When finished, stop and cleanup using:"
+echo
+echo "    pg_ctl -D ${PGDATA} stop"
+echo "    rm -rf ${PGDATA}"
+echo
+
+createuser -U ${USER} -s ${PGUSER}
+createdb --owner=${PGUSER} test_geopandas
+psql -d test_geopandas -q -c "CREATE EXTENSION postgis"
+
+echo "PostGIS server ready."