Add functional test cases for spatial type support in PostgreSQL

This commit adds functional testing for PostGIS spatial types (GEOMETRY and GEOGRAPHY) with schema introspection and CI integration.

The implementation leverages PostgreSQL's native type system for introspection, making it compatible with any PostgreSQL instance without requiring PostGIS system tables to be accessible during schema operations.

Author: ddegasperi

.github/workflows/continuous-integration.yml

@@ -133,6 +133,30 @@ jobs:
             extension: "pdo_pgsql"
             config-file-suffix: "-stringify_fetches"
 
+  phpunit-postgis:
+    name: "PHPUnit with PostGIS"
+    needs: "phpunit-smoke-check"
+    uses: ./.github/workflows/phpunit-postgis.yml
+    with:
+      php-version: ${{ matrix.php-version }}
+      postgis-version: ${{ matrix.postgis-version }}
+      extension: ${{ matrix.extension }}
+
+    strategy:
+      matrix:
+        php-version:
+          - "8.3"
+          - "8.4"
+        postgis-version:
+          - "17-3.5"
+        extension:
+          - "pgsql"
+          - "pdo_pgsql"
+        include:
+          - php-version: "8.2"
+            postgis-version: "17-3.5"
+            extension: "pgsql"
+
   phpunit-mariadb:
     name: "PHPUnit with MariaDB"
     needs: "phpunit-smoke-check"
@@ -288,6 +312,7 @@ jobs:
       - "phpunit-smoke-check"
       - "phpunit-oracle"
       - "phpunit-postgres"
+      - "phpunit-postgis"
       - "phpunit-mariadb"
       - "phpunit-mysql"
       - "phpunit-mssql"

.github/workflows/phpunit-postgis.yml

@@ -0,0 +1,64 @@
+name: PHPUnit with PostGIS
+
+on:
+  workflow_call:
+    inputs:
+      php-version:
+        required: true
+        type: string
+      postgis-version:
+        required: true
+        type: string
+      extension:
+        required: true
+        type: string
+      postgres-locale-provider:
+        required: true
+        type: string
+      config-file-suffix:
+        required: false
+        type: string
+        default: ''
+
+jobs:
+  phpunit-postgis:
+    runs-on: ubuntu-24.04
+
+    services:
+      postgres:
+        image: postgis/postgis:${{ inputs.postgis-version }}
+        ports:
+          - '5432:5432'
+        env:
+          POSTGRES_PASSWORD: postgres
+          POSTGRES_INITDB_ARGS: ${{ inputs.postgres-locale-provider == 'icu' && '--locale-provider=icu --icu-locale=en-US' || '' }}
+        options: >-
+          --health-cmd pg_isready
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Install PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ inputs.php-version }}
+          extensions: ${{ inputs.extension }}
+          coverage: pcov
+          ini-values: zend.assertions=1
+        env:
+          fail-fast: true
+
+      - name: Install dependencies with Composer
+        uses: ramsey/composer-install@v3
+        with:
+          composer-options: '--ignore-platform-req=php+'
+
+      - name: Run PHPUnit
+        run: vendor/bin/phpunit -c ci/github/phpunit/${{ inputs.extension }}${{ inputs.config-file-suffix }}.xml --coverage-clover=coverage.xml
+
+      - name: Upload coverage file
+        uses: actions/upload-artifact@v4
+        with:
+          name: ${{ github.job }}-${{ inputs.postgis-version }}-php-${{ inputs.php-version }}-${{ inputs.extension }}${{ inputs.config-file-suffix }}.coverage
+          path: coverage.xml

src/Schema/PostgreSQLSchemaManager.php

@@ -313,7 +313,7 @@ protected function _getPortableTableColumnDefinition(array $tableColumn): Column
     /**
      * Parses the parameters between parenthesis in the data type.
      *
-     * @return list<int>
+     * @return list<int|string>
      */
     private function parseColumnTypeParameters(string $type): array
     {
@@ -328,7 +328,7 @@ private function parseColumnTypeParameters(string $type): array
         }
 
         // Cast numeric strings to int automatically
-        $parameters = array_map(function ($param) {
+        $parameters = array_map(static function ($param) {
             return ctype_digit($param) ? (int) $param : $param;
         }, $parameters);