From 3dad610860dc025f339795574a6a1be8e2b5bb1e Mon Sep 17 00:00:00 2001
From: Yash Mehrotra <yashmehrotra95@gmail.com>
Date: Mon, 27 Oct 2025 22:07:45 +0530
Subject: [PATCH 1/3] chore: fix test suite

---
 .github/workflows/build-and-test.yml | 79 ++++++++++++++++------------
 1 file changed, 44 insertions(+), 35 deletions(-)

diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml
index 150adb7..1530ea6 100644
--- a/.github/workflows/build-and-test.yml
+++ b/.github/workflows/build-and-test.yml
@@ -202,43 +202,52 @@ jobs:
             -v $PWD/test-data/${{ matrix.test.from }}:/var/lib/postgresql/data \
             -e POSTGRES_PASSWORD=testpass \
             postgres:${{ matrix.test.to }}-bookworm \
-            bash -c "
-              set -e
-              set -o pipefail
-
-              docker-entrypoint.sh postgres &
-              PGPID=\$!
-              sleep 10
-
-              # Verify data
-              psql -U testuser -d testdb -v ON_ERROR_STOP=1 <<'EOF'
-          -- Check table exists
-          SELECT COUNT(*) as record_count FROM test_data;
-
-          -- Check index exists
-          SELECT indexname FROM pg_indexes WHERE tablename = 'test_data';
-
-          -- Verify data integrity - this will fail if count is not 1000
-          DO $$
-          DECLARE
-            record_count INT;
-          BEGIN
-            SELECT COUNT(*) INTO record_count FROM test_data;
-            IF record_count != 10000 THEN
-              RAISE EXCEPTION 'FAIL: Expected 1000 records, found %', record_count;
-            END IF;
-            RAISE NOTICE 'PASS: All 1000 records present';
-          END $$;
+            bash <<'EOF'
+          set -e
+          set -o pipefail
+
+          docker-entrypoint.sh postgres &
+          PGPID=$!
+          sleep 10
+
+          # Check record count
+          echo 'Checking record count...'
+          RECORD_COUNT=$(psql -U testuser -d testdb -t -A -c 'SELECT COUNT(*) FROM test_data;')
+          echo "Found $RECORD_COUNT records"
+          if [ "$RECORD_COUNT" != "10000" ]; then
+            echo "FAIL: Expected 1000 records, found $RECORD_COUNT"
+            exit 1
+          fi
+          echo 'PASS: All 1000 records present'
+
+          # Check index exists
+          echo 'Checking if index exists...'
+          INDEX_NAME=$(psql -U testuser -d testdb -t -A -c "SELECT indexname FROM pg_indexes WHERE tablename = 'test_data' AND indexname = 'idx_test_data';")
+          if [ -z "$INDEX_NAME" ]; then
+            echo 'FAIL: Index idx_test_data not found'
+            exit 1
+          fi
+          echo "PASS: Index $INDEX_NAME exists"
+
+          # Verify we can query the data
+          echo 'Verifying data integrity...'
+          SAMPLE_DATA=$(psql -U testuser -d testdb -t -A -c "SELECT data FROM test_data WHERE id = 1;")
+          echo "Sample record: $SAMPLE_DATA"
+          EXPECTED_DATA='Test record 1'
+          if [ "$SAMPLE_DATA" != "$EXPECTED_DATA" ]; then
+            echo "FAIL: Expected '$EXPECTED_DATA', found '$SAMPLE_DATA'"
+            exit 1
+          fi
+          echo 'PASS: Data is accessible and intact'
+
+          # Stop PostgreSQL
+          su postgres -c 'pg_ctl -D /var/lib/postgresql/data stop -m smart -w'
+          sleep 5
+
+          # Fix permissions for GitHub Actions runner
+          chown -R $(id -u):$(id -g) /var/lib/postgresql/data
           EOF
 
-              # Stop PostgreSQL
-              su postgres -c 'pg_ctl -D /var/lib/postgresql/data stop -m smart -w'
-              sleep 5
-
-              # Fix permissions for GitHub Actions runner
-              chown -R $(id -u):$(id -g) /var/lib/postgresql/data
-            "
-
       - name: Fix permissions after test
         if: always()
         run: |

From 30f3d6d77dd0c06e57e0238ccfdaf486f353611e Mon Sep 17 00:00:00 2001
From: Yash Mehrotra <yashmehrotra95@gmail.com>
Date: Mon, 27 Oct 2025 22:19:38 +0530
Subject: [PATCH 2/3] chore: update bash

---
 .github/workflows/build-and-test.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml
index 1530ea6..be0b355 100644
--- a/.github/workflows/build-and-test.yml
+++ b/.github/workflows/build-and-test.yml
@@ -198,11 +198,11 @@ jobs:
 
       - name: Verify upgraded data
         run: |
-          docker run --rm \
+          cat <<'EOF' | docker run --rm -i \
             -v $PWD/test-data/${{ matrix.test.from }}:/var/lib/postgresql/data \
             -e POSTGRES_PASSWORD=testpass \
             postgres:${{ matrix.test.to }}-bookworm \
-            bash <<'EOF'
+            bash
           set -e
           set -o pipefail
 

From c12e9ad83f466e99e92851b197b299098a3ee482 Mon Sep 17 00:00:00 2001
From: Yash Mehrotra <yashmehrotra95@gmail.com>
Date: Mon, 27 Oct 2025 22:25:42 +0530
Subject: [PATCH 3/3] chore: update where pg bin is set

---
 .github/workflows/build-and-test.yml | 2 +-
 Dockerfile                           | 1 -
 docker-entrypoint.sh                 | 2 ++
 3 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml
index be0b355..4920fbc 100644
--- a/.github/workflows/build-and-test.yml
+++ b/.github/workflows/build-and-test.yml
@@ -214,7 +214,7 @@ jobs:
           echo 'Checking record count...'
           RECORD_COUNT=$(psql -U testuser -d testdb -t -A -c 'SELECT COUNT(*) FROM test_data;')
           echo "Found $RECORD_COUNT records"
-          if [ "$RECORD_COUNT" != "10000" ]; then
+          if [ "$RECORD_COUNT" != "1000" ]; then
             echo "FAIL: Expected 1000 records, found $RECORD_COUNT"
             exit 1
           fi
diff --git a/Dockerfile b/Dockerfile
index a480bef..d1f012c 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -91,7 +91,6 @@ ENV PG17BIN=/usr/lib/postgresql/17/bin
 
 # Data directory
 ENV PGDATA=/var/lib/postgresql/data
-ENV PGBIN=/usr/lib/postgresql/${PG_VERSION}/bin
 
 # PostgreSQL default configuration
 ENV POSTGRES_DB=postgres
diff --git a/docker-entrypoint.sh b/docker-entrypoint.sh
index b1b7623..aa89c6b 100755
--- a/docker-entrypoint.sh
+++ b/docker-entrypoint.sh
@@ -37,5 +37,7 @@ if [ "$AUTO_UPGRADE" = "true" ] && [ "$UPGRADE_ONLY" = "true" ]; then
     exit 0
 fi
 
+export PGBIN=/usr/lib/postgresql/${PG_VERSION}/bin
+
 # Start PostgreSQL server
 exec "$PGBIN/postgres" "$@"