From b5cb644f521498e317a547686ff465a9ffcf80d8 Mon Sep 17 00:00:00 2001
From: Rust Saiargaliev <fly.amureki@gmail.com>
Date: Sun, 16 Oct 2022 23:13:16 +0200
Subject: [PATCH] Revert "Unify env var names"

This reverts commit 007520acad04c01826b53a4fdb9b5e11090c781a.
---
 .github/workflows/tests.yml |  4 +++-
 postgis-tests.sh            | 10 +++++-----
 postgres-docker             |  4 ++--
 tests/conftest.py           | 12 ++++++------
 tox.ini                     |  3 +--
 5 files changed, 17 insertions(+), 16 deletions(-)

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index 2dcf43c6..bc24156d 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -16,8 +16,9 @@ jobs:
         image: postgis/postgis
         env:
           POSTGRES_DB: postgres
-          POSTGRES_USER: postgres
           POSTGRES_PASSWORD: postgres
+          PGPASSWORD: postgres
+          POSTGRES_USER: postgres
         ports:
           - 5432:5432
         options: --health-cmd pg_isready --health-interval 10s --health-timeout 5s --health-retries 5
@@ -33,6 +34,7 @@ jobs:
         run: |
           sudo apt-get update
           sudo apt-get install -y gdal-bin
+          psql -c "CREATE ROLE runner SUPERUSER LOGIN CREATEDB;" -U postgres -h localhost -p 5432
           psql template1 -c "CREATE EXTENSION citext;" -U postgres -h localhost -p 5432
           psql template1 -c "CREATE EXTENSION hstore;" -U postgres -h localhost -p 5432
           psql template1 -c "CREATE EXTENSION postgis;" -U postgres -h localhost -p 5432
diff --git a/postgis-tests.sh b/postgis-tests.sh
index 8bd00711..a7ff5c68 100755
--- a/postgis-tests.sh
+++ b/postgis-tests.sh
@@ -11,7 +11,7 @@ set -euo pipefail
 ### so if you are already running PostgreSQL locally, instead of using this
 ### script, simply run:
 ###
-###     # TEST_DB=postgis POSTGRES_USER=postgres python -m pytest
+###     # TEST_DB=postgis PGUSER=postgres python -m pytest
 ###
 ### This script uses the `python` on the current `$PATH`, but can be overridden
 ### by setting the `PYTHON_CLI` environment variable.
@@ -48,9 +48,9 @@ PYTHON=${PYTHON_CLI:-python}
 DEPS="$PYTHON"
 
 # Arg defaults
-export POSTGRES_PORT=${POSTGRES_PORT:-5432}
-export POSTGRES_USER=${POSTGRES_USER:-postgres}
-export POSTGRES_PASSWORD=${POSTGRES_PASSWORD:-postgres}
+export PGPORT=${PGPORT:-5432}
+export PGUSER=${PGUSER:-postgres}
+export PGPASSWORD=${PGPASSWORD:-postgres}
 export TEST_DB=${TEST_DB:-postgis}
 
 # Argument parsing using "getopt"
@@ -110,7 +110,7 @@ for dep in $DEPS; do
 done
 
 # Run the postgres container with all extensions installed
-$PROJECT_ROOT/postgres-docker --port POSTGRES_PORT
+$PROJECT_ROOT/postgres-docker --port $PGPORT
 
 # Run the tests
 $PYTHON -m pytest
diff --git a/postgres-docker b/postgres-docker
index 8933fcc3..d503d5a0 100755
--- a/postgres-docker
+++ b/postgres-docker
@@ -48,7 +48,7 @@ DEPS="$DOCKER"
 # Arg defaults
 KILL=false
 CONTAINER_NAME=model-bakery-postgres
-PORT=${POSTGRES_PORT:-5432}
+PORT=${PGPORT:-5432}
 
 # Argument parsing using "getopt"
 OPTIONS=h,k,p
@@ -66,7 +66,7 @@ eval set -- "$PARSED"
 while true; do
     case "$1" in
         -p|--port)
-            POSTGRES_PORT=$2
+            PGPORT=$2
             shift
             ;;
         -k|--kill)
diff --git a/tests/conftest.py b/tests/conftest.py
index 9b91322e..ab46503f 100644
--- a/tests/conftest.py
+++ b/tests/conftest.py
@@ -48,18 +48,18 @@ def pytest_configure():
                 "NAME": db_name,
                 "HOST": "localhost",
                 # The following DB settings are only used for `postgresql` and `postgis`
-                "PORT": os.environ.get("POSTGRES_PORT", ""),
-                "USER": os.environ.get("POSTGRES_USER", ""),
-                "PASSWORD": os.environ.get("POSTGRES_PASSWORD", ""),
+                "PORT": os.environ.get("PGPORT", ""),
+                "USER": os.environ.get("PGUSER", ""),
+                "PASSWORD": os.environ.get("PGPASSWORD", ""),
             },
             # Extra DB used to test multi database support
             EXTRA_DB: {
                 "ENGINE": db_engine,
                 "NAME": extra_db_name,
                 "HOST": "localhost",
-                "PORT": os.environ.get("POSTGRES_PORT", ""),
-                "USER": os.environ.get("POSTGRES_USER", ""),
-                "PASSWORD": os.environ.get("POSTGRES_PASSWORD", ""),
+                "PORT": os.environ.get("PGPORT", ""),
+                "USER": os.environ.get("PGUSER", ""),
+                "PASSWORD": os.environ.get("PGPASSWORD", ""),
             },
         },
         INSTALLED_APPS=installed_apps,
diff --git a/tox.ini b/tox.ini
index 7e49e989..8595863a 100644
--- a/tox.ini
+++ b/tox.ini
@@ -18,8 +18,7 @@ python =
 setenv =
     PYTHONPATH={toxinidir}
     postgresql: TEST_DB=postgis
-    postgresql: POSTGRES_USER=postgres
-    postgresql: POSTGRES_PASSWORD=postgres
+    postgresql: PGUSER=postgres
     sqlite: TEST_DB=sqlite
     sqlite: USE_TZ=True
 deps =