Merge pull request #14 from ruxwez/implement-runtime-tests

Refactor Dockerfile and workflows for build and runtime testing

Author: ruxwez

.docker/Dockerfile

@@ -1,6 +1,6 @@
 ARG POSTGRES_VERSION=17
 
-FROM docker.io/golang:1.26-alpine AS builder
+FROM mirror.gcr.io/golang:1.26-alpine AS builder
 
 WORKDIR /app
 
@@ -11,7 +11,7 @@ COPY . .
 
 RUN go build -o main .
 
-FROM docker.io/postgres:${POSTGRES_VERSION}
+FROM mirror.gcr.io/postgres:${POSTGRES_VERSION}
 
 WORKDIR /opt/app
 

.docker/runtime-test.sh

@@ -0,0 +1,24 @@
+#!/usr/bin/env bash
+
+set -euo pipefail
+
+POSTGRES_VERSION="${POSTGRES_VERSION:-18}"
+IMAGE="${IMAGE:-postgres-extensor-runtime-test}"
+CONTAINER_NAME="${CONTAINER_NAME:-postgres-extensor-runtime-test-$$}"
+
+trap 'docker rm -f "${CONTAINER_NAME}" >/dev/null 2>&1 || true' EXIT
+
+if [ "${SKIP_BUILD:-false}" != "true" ]; then
+  docker build -f .docker/Dockerfile -t "${IMAGE}" --build-arg POSTGRES_VERSION="${POSTGRES_VERSION}" .
+fi
+
+docker run -d --rm --name "${CONTAINER_NAME}" -e POSTGRES_PASSWORD=postgres "${IMAGE}" >/dev/null
+
+echo "Waiting for PostgreSQL to start..."
+until docker exec "${CONTAINER_NAME}" pg_isready -U postgres >/dev/null 2>&1; do
+  sleep 1
+done
+
+echo "Running SQL tests..."
+docker exec -i "${CONTAINER_NAME}" psql -v ON_ERROR_STOP=1 -U postgres < .docker/runtime-test.sql
+echo "Tests passed successfully!"

.docker/runtime-test.sql

@@ -0,0 +1,6 @@
+CREATE EXTENSION IF NOT EXISTS vector;
+CREATE EXTENSION IF NOT EXISTS pgmq;
+CREATE EXTENSION IF NOT EXISTS postgis;
+
+SELECT extname FROM pg_extension WHERE extname IN ('vector','pgmq','postgis');
+SELECT name FROM pg_available_extensions WHERE name IN ('pg_stat_statements');

.github/workflows/docker-build.yml

@@ -11,7 +11,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        postgres_version: ["latest", "18", "17", "16", "15", "14", "13"]
+        postgres_version: ["latest", "18", "17", "16", "15", "14"]
       fail-fast: false
 
     steps:

.github/workflows/pr-test.yml

@@ -18,7 +18,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        postgres_version: ["latest", "18", "17", "16", "15", "14", "13"]
+        postgres_version: ["latest", "18", "17", "16", "15", "14"]
         platform: ["linux/amd64"]
       fail-fast: false
 
@@ -41,7 +41,7 @@ jobs:
           file: .docker/Dockerfile
           platforms: ${{ matrix.platform }}
           push: false
-          load: false
+          load: true
           tags: |
             pr-test/${{ github.repository }}:postgres-${{ matrix.postgres_version }}-pr${{ github.event.pull_request.number }}
           build-args: |
@@ -55,6 +55,12 @@ jobs:
           echo "Built (multi-arch) for POSTGRES_VERSION=${{ matrix.postgres_version }} PLATFORM=${{ matrix.platform }}"
           echo "Tag used: pr-test/${{ github.repository }}:postgres-${{ matrix.postgres_version }}-pr${{ github.event.pull_request.number }}"
 
+      - name: Run runtime test
+        env:
+          SKIP_BUILD: "true"
+          IMAGE: pr-test/${{ github.repository }}:postgres-${{ matrix.postgres_version }}-pr${{ github.event.pull_request.number }}
+        run: bash .docker/runtime-test.sh
+
   # Job resumen que agrupa todos los builds de la matriz
   build-complete:
     name: All builds passed

Makefile

@@ -1,10 +1,13 @@
 PG_VERSION ?= 18
 
-.PHONY: test build
+.PHONY: test build runtime-test
 
 test:
 	docker build -f .docker/Dockerfile.test -t postgres-extensor-test .
 	-docker rmi postgres-extensor-test > /dev/null
 
 build:
 	docker build -f .docker/Dockerfile -t ruxwez/postgres:$(PG_VERSION) --build-arg POSTGRES_VERSION=$(PG_VERSION) .
+
+runtime-test:
+	POSTGRES_VERSION=$(PG_VERSION) bash .docker/runtime-test.sh

extensions/builddeps_install.go

@@ -10,7 +10,7 @@ import (
 var buildDepsInstall = plyd.New(
 	"build-deps-install",
 	plyd.NewHandlersList(
-		plyd.NewHandler(">=13.0.0, <19.0.0", func(ins *instance.InstanceInfo) error {
+		plyd.NewHandler(">=14.0.0, <19.0.0", func(ins *instance.InstanceInfo) error {
 			if err := plyd.Run("apt-get update -qq"); err != nil {
 				return err
 			}

extensions/builddeps_remove.go

@@ -10,7 +10,7 @@ import (
 var buildDepsRemove = plyd.New(
 	"build-deps-remove",
 	plyd.NewHandlersList(
-		plyd.NewHandler(">=13.0.0, <19.0.0", func(ins *instance.InstanceInfo) error {
+		plyd.NewHandler(">=14.0.0, <19.0.0", func(ins *instance.InstanceInfo) error {
 			if err := plyd.Run(fmt.Sprintf(
 				"apt-get remove -y -qq git make gcc postgresql-server-dev-%s",
 				ins.PostgreSQL.Mayor,

extensions/pgvector.go

@@ -10,7 +10,7 @@ import (
 var pgvector = plyd.New(
 	"pgvector",
 	plyd.NewHandlersList(
-		plyd.NewHandler(">=13.0.0, <19.0.0", func(ins *instance.InstanceInfo) error {
+		plyd.NewHandler(">=14.0.0, <19.0.0", func(ins *instance.InstanceInfo) error {
 			err := plyd.Run(
 				"git clone --branch v0.8.2 --depth 1 " +
 					"https://github.com/pgvector/pgvector.git /tmp/pgvector",

extensions/postgis.go

@@ -10,7 +10,7 @@ import (
 var postgis = plyd.New(
 	"postgis",
 	plyd.NewHandlersList(
-		plyd.NewHandler(">=13.0.0, <19.0.0", func(ins *instance.InstanceInfo) error {
+		plyd.NewHandler(">=14.0.0, <19.0.0", func(ins *instance.InstanceInfo) error {
 			pkg := fmt.Sprintf("postgresql-%s-postgis-3", ins.PostgreSQL.Mayor)
 			if err := plyd.Run(fmt.Sprintf("apt-get install -y -qq %s", pkg)); err != nil {
 				return err