WMS Facade updates (#2303)

* WMS Facade updates:

- added unit test for wms facade
- removed trailing comma
- prevent errors with non existing keys

* - fixed flak8 errors

* - removed bbox validation

* - removed invalid bbox test

* - fixed flak8

Author: doublebyte1

pygeoapi/api/maps.py

@@ -69,7 +69,7 @@
     'http://www.opengis.net/def/crs/EPSG/0/4326': 'http://www.opengis.net/def/crs/EPSG/0/4326',  # noqa
     'http://www.opengis.net/def/crs/EPSG/0/3857': 'http://www.opengis.net/def/crs/EPSG/0/3857',  # noqa
     'EPSG:4326': 'http://www.opengis.net/def/crs/EPSG/0/4326',
-    'EPSG:3857': 'http://www.opengis.net/def/crs/EPSG/0/3857',
+    'EPSG:3857': 'http://www.opengis.net/def/crs/EPSG/0/3857'
 }
 
 

pygeoapi/provider/wms_facade.py

@@ -86,7 +86,7 @@ def query(self, style=None, bbox=[-180, -90, 180, 90], width=500,
 
         version = self.options.get('version', '1.3.0')
 
-        if version == '1.3.0' and CRS_CODES[bbox_crs] == 'EPSG:4326':
+        if version == '1.3.0' and CRS_CODES.get('bbox_crs') == 'EPSG:4326':
             bbox = [bbox[1], bbox[0], bbox[3], bbox[2]]
         bbox2 = ','.join(map(str, bbox))
 
@@ -99,7 +99,7 @@ def query(self, style=None, bbox=[-180, -90, 180, 90], width=500,
             'service': 'WMS',
             'request': 'GetMap',
             'bbox': bbox2,
-            crs_param: CRS_CODES[crs],
+            crs_param: CRS_CODES.get(crs) or 'EPSG:4326',
             'layers': self.options['layer'],
             'styles': self.options.get('style', 'default'),
             'width': width,

tests/provider/test_wms_facade_provider.py

@@ -0,0 +1,64 @@
+# =================================================================
+#
+# Authors: Joana Simoes <doublebyte@doublebyte.net>
+#
+#
+# Copyright (c) 2026 Joana Simoes
+#
+# Permission is hereby granted, free of charge, to any person
+# obtaining a copy of this software and associated documentation
+# files (the "Software"), to deal in the Software without
+# restriction, including without limitation the rights to use,
+# copy, modify, merge, publish, distribute, sublicense, and/or sell
+# copies of the Software, and to permit persons to whom the
+# Software is furnished to do so, subject to the following
+# conditions:
+#
+# The above copyright notice and this permission notice shall be
+# included in all copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+# EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
+# OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+# NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+# HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
+# WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+# FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+# OTHER DEALINGS IN THE SOFTWARE.
+#
+# =================================================================
+
+import pytest
+
+from pygeoapi.provider.wms_facade import WMSFacadeProvider
+
+
+@pytest.fixture()
+def config():
+    return {
+        'name': 'WMSFacade',
+        'type': 'map',
+        'data': 'https://demo.mapserver.org/cgi-bin/msautotest',
+        'options': {
+            'layer': 'world_latlong',
+            'style': 'default'
+        },
+        'format': {
+            'name': 'png',
+            'mimetype': 'image/png'
+        }
+    }
+
+
+def test_query(config):
+    p = WMSFacadeProvider(config)
+
+    results = p.query()
+    assert len(results) > 0
+
+    # an invalid CRS should return the default bbox (4326)
+    results2 = p.query(crs='http://www.opengis.net/def/crs/EPSG/0/1111')
+    assert len(results2) == len(results)
+
+    results3 = p.query(crs='http://www.opengis.net/def/crs/EPSG/0/3857')
+    assert len(results3) != len(results)