Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OGCAPI intial tests #7926

Merged
merged 16 commits into from
Jul 13, 2023
Merged

OGCAPI intial tests #7926

merged 16 commits into from
Jul 13, 2023

Conversation

elpaso
Copy link
Collaborator

@elpaso elpaso commented Jun 12, 2023

Fix #7679

Initial implementation of tests for OGCAPI

@elpaso elpaso marked this pull request as draft June 12, 2023 15:45
autotest/gdrivers/ogcapi.py Outdated Show resolved Hide resolved
autotest/gdrivers/ogcapi.py Outdated Show resolved Hide resolved
autotest/gdrivers/ogcapi.py Outdated Show resolved Hide resolved
autotest/gdrivers/ogcapi.py Outdated Show resolved Hide resolved
autotest/gdrivers/ogcapi.py Outdated Show resolved Hide resolved
autotest/gdrivers/ogcapi.py Outdated Show resolved Hide resolved
autotest/gdrivers/ogcapi.py Outdated Show resolved Hide resolved
autotest/gdrivers/ogcapi.py Outdated Show resolved Hide resolved
@elpaso elpaso force-pushed the bugfix-gh7679-OGCAPI-tests branch 3 times, most recently from f54640b to 4d75961 Compare June 19, 2023 05:59
@rouault
Copy link
Member

rouault commented Jun 22, 2023

There's a specific error with the build-windows-conda (https://github.com/OSGeo/gdal/actions/runs/5321551897/jobs/9636788770?pr=7926)

36: FAILED gdrivers/ogcapi.py::test_ogr_ogcapi_features - RuntimeError: Unsupported HTTP version (2.0) in response
36: FAILED gdrivers/ogcapi.py::test_ogr_ogcapi_vector_tiles[AUTO] - RuntimeError: Unsupported HTTP version (2.0) in response
36: FAILED gdrivers/ogcapi.py::test_ogr_ogcapi_vector_tiles[GEOJSON] - RuntimeError: Unsupported HTTP version (2.0) in response
36: FAILED gdrivers/ogcapi.py::test_ogr_ogcapi_vector_tiles[GEOJSON_PREFERRED] - RuntimeError: Unsupported HTTP version (2.0) in response
36: FAILED gdrivers/ogcapi.py::test_ogr_ogcapi_vector_tiles[MVT] - RuntimeError: Unsupported HTTP version (2.0) in response
36: FAILED gdrivers/ogcapi.py::test_ogr_ogcapi_vector_tiles[MVT_PREFERRED] - RuntimeError: Unsupported HTTP version (2.0) in response
36: FAILED gdrivers/ogcapi.py::test_ogr_ogcapi_raster[MAP-Collection ne_10m_lakes_europe] - RuntimeError: Unsupported HTTP version (2.0) in response
36: FAILED gdrivers/ogcapi.py::test_ogr_ogcapi_raster[TILES-Collection ne_10m_lakes_europe] - RuntimeError: Unsupported HTTP version (2.0) in response
36: FAILED gdrivers/ogcapi.py::test_ogr_ogcapi_raster[COVERAGE-SRTM] - RuntimeError: Unsupported HTTP version (2.0) in response

We should probably use HTTP 1.1 to be safe

@rouault
Copy link
Member

rouault commented Jun 22, 2023

@elpaso I finally figured the reason of the failures of the vector side. The reason is that CMake had to be run twice to have OGR_ENABLE_DRIVER_GML defined. Fixed per #7993

@elpaso elpaso force-pushed the bugfix-gh7679-OGCAPI-tests branch from 1915fcb to 319c5de Compare June 26, 2023 06:31
@elpaso elpaso marked this pull request as ready for review June 26, 2023 09:41
@rouault rouault merged commit 687d150 into OSGeo:master Jul 13, 2023
jeffypooo pushed a commit to IgnisTechnologies/gdal that referenced this pull request Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OGCAPI driver: need regression tests
2 participants