Test against GeoServer supported releases

Author: avvertix

.github/workflows/run-tests.yml

@@ -13,10 +13,16 @@ jobs:
       fail-fast: true
       matrix:
         os: [ubuntu-latest]
-        php: [7.4, 8.0, 8.1] 
-        stability: [prefer-stable] # prefer-lowest
+        php: [7.4, 8.0, 8.1]
+        geoserver: ["2.20.4", "2.21.0"]
+        exclude:
+        - geoserver: "2.20.4"
+          php: 7.4
 
-    name: P${{ matrix.php }} - ${{ matrix.stability }} - ${{ matrix.os }}
+    name: P${{ matrix.php }} - G${{ matrix.geoserver }}
+
+    env:
+      GEOSERVER_TAG: ${{ matrix.geoserver }}
 
     steps:
       - name: Checkout code
@@ -36,7 +42,7 @@ jobs:
 
       - name: Install dependencies
         run: |
-          composer update --${{ matrix.stability }} --prefer-dist --no-interaction
+          composer update --prefer-dist --no-interaction
       
       - name: Create Environment
         run: |
@@ -46,8 +52,8 @@ jobs:
       - name: Execute tests
         env:
           GEOSERVER_URL: "http://127.0.0.1:8600/geoserver/"
-          GEOSERVER_USER: "admin"
-          GEOSERVER_PASSWORD: "geoserver"
+          GEOSERVER_USER: "GEOSERVER_ADMIN_USER"
+          GEOSERVER_PASSWORD: "GEOSERVER_ADMIN_PASSWORD"
         run: vendor/bin/phpunit
 
       - name: Stop Environment

docs/tests.md

@@ -8,7 +8,8 @@ vendor/bin/phpunit
 
 By default integration tests are not executed unless in the phpunit.xml file a GeoServer instance is specified.
 
-The `phpunit.xml.dist` define the `GEOSERVER_URL`, `GEOSERVER_USER`, `GEOSERVER_PASSWORD` for that purpose. 
+The `phpunit.xml.dist` define the `GEOSERVER_URL`, `GEOSERVER_USER`,
+`GEOSERVER_PASSWORD` for that purpose. 
 If you want you can copy `phpunit.xml.dist` to `phpunit.xml` and edit those variables in place 
 or define them in your environment variables.
 

tests/Integration/GeoServerDataStoresTest.php

@@ -252,10 +252,10 @@ public function test_geopackage_can_be_uploaded_and_deleted()
         $this->assertNotNull($feature->nativeBoundingBox);
         $this->assertNotNull($feature->boundingBox);
         $this->assertEquals("EPSG:4326", $feature->boundingBox->crs);
-        $this->assertEquals(-165.18163033675, $feature->boundingBox->minX);
-        $this->assertEquals(-50.786361166949, $feature->boundingBox->minY);
-        $this->assertEquals(176.5891535779, $feature->boundingBox->maxX);
-        $this->assertEquals(73.578794485117, $feature->boundingBox->maxY);
+        $this->assertEquals(-180.00000000000003, $feature->boundingBox->minX);
+        $this->assertEquals(-85, $feature->boundingBox->minY);
+        $this->assertEquals(180.00000000000003, $feature->boundingBox->maxX);
+        $this->assertEquals(85, $feature->boundingBox->maxY);
 
         $this->assertTrue($this->geoserver->exist($data), "Data not existing after upload");
 

tests/docker-compose.yml

@@ -8,7 +8,7 @@ networks:
 
 services:
   geoserver:
-    image: "kartoza/geoserver:${GEOSERVER_TAG:-2.17.0}"
+    image: "kartoza/geoserver:${GEOSERVER_TAG:-2.21.0}"
     env_file:
         - geoserver.env
     ports: