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

Need new Mapbox Tilejson Unit tests #450

Open
justb4 opened this issue Jun 19, 2023 · 0 comments
Open

Need new Mapbox Tilejson Unit tests #450

justb4 opened this issue Jun 19, 2023 · 0 comments

Comments

@justb4
Copy link
Member

justb4 commented Jun 19, 2023

Is your feature request related to a problem? Please describe.
PR #449 had to remove a stale URL that was a public "TileJSON" endpoint for MapBox Vector Tiles in the Dutch SDI PDOK:
https://geodata.nationaalgeoregister.nl/beta/topotiles-viewer/styles/tilejson.json
There is no direct replacement URL as PDOK has changed to OGC API Vector Tiles APIs.

Describe the solution you'd like
Somehow we need an endpoint to test the GHC Probe GeoHealthCheck.plugins.probe.mapbox.TileJSON.

Describe alternatives you've considered
MapTiler has some public endpoints but these require a key. Maybe we can still use.

Additional context
The unit tests were removed by removing these fixtures from tests/data/fixtures.json:

In fixtures.json


Resource
    "PDOK MAPBOX TILEJSON": {
      "owner": "admin",
      "resource_type": "Mapbox:TileJSON",
      "active": true,
      "title": "PDOK BRT+BGT Mapbox TileJSON",
      "url": "https://geodata.nationaalgeoregister.nl/beta/topotiles-viewer/styles/tilejson.json",
      "tags": [
        "pdok",
        "tiling"
      ]
    }
Probe vars
   "PDOK - MAPBOX TILEJSON": {
      "resource": "PDOK MAPBOX TILEJSON",
      "probe_class": "GeoHealthCheck.plugins.probe.mapbox.TileJSON",
      "parameters": {
        "lat_4326": "52",
        "lon_4326": "5"
      }
    }
Check vars

    "PDOK MAPBOX TILEJSON - HTTP - NoError": {
      "probe_vars": "PDOK - MAPBOX TILEJSON",
      "check_class": "GeoHealthCheck.plugins.check.checks.HttpStatusNoError",
      "parameters": {}
    },

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant