search

geopandas / xyzservices

Source of XYZ tiles providers
https://xyzservices.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
169 stars 32 forks source link

TST: test openly accessible tiles and mark broken with `status="broken"` flag #78

Closed martinfleis closed 3 years ago

martinfleis commented 3 years ago

Splitting test suite into 2 files and adding tests that check all providers that do not require any API key.

I will also add a machinery marking broken sources coming from leaflet-providers and removing them from the compressed json. Just want to run CI before that to show how it behaves with broken tiles.

Some (Stadia) do not require API key according to our metadata but the domain needs to be whitelisted to make them work. Those give 403 error and are xfailed.

martinfleis commented 3 years ago

As shown in CI (https://github.com/geopandas/xyzservices/pull/78/checks?check_run_id=3274570828), we have a few broken sources:

I will not remove them but add status="broken" attribute to them. That way we do not deviate from the original leaflet-providers set but clearly mark that they are broken. Since all are now reported it is likely that it will be resolved in the upstream repo.