opengeospatial / ogcapi-tiles

OGC API - Tiles draft specification
https://ogcapi.ogc.org/tiles
Other
38 stars 18 forks source link

Issues in current conformance overview, Abstract Test 6 #90

Closed jerstlouis closed 2 years ago

jerstlouis commented 3 years ago

(In Clause 2 - Conformance) "Core": no path is prescribed for this conformance class so no fixed path resource exists for this. The only requirement is that a URL template could be specified where 3 different variables correspond to tile matrix, tile row and tile column.

"TileSet": The resource path is wrong. Normally it is…​/tiles/{tileMatrixSet}, …​/tiles/{tileMatrixSet}/{tileMatrix} does not return anything. However, again the path is not fixed yet for this conformance class, only the content must conform to TileSet Metadata standard.

"TileSets List": The resource path is also wrong. Normally it is…​/tiles. However again the path is not fixed, only the content must be a list of TileSets.

"DataSet TileSets": I think it would be easier to simply state that /tiles at the root of the API provides a TileSets List as defined in the other conformance classes. "GeoData TileSets": I think it would be easier to simply state that /collections/{collectionId}/tiles for the collections provides a TileSets List as defined in the other conformance classes.

NOTE: The path to individual tiles might actually not be fixed at all. Only Recommendation 1 A says something about this:

A tiles implementation should consider to use the tiles URI template variables in the following common order and form: {tileMatrix}/{tileRow}/{tileCol}

Abstract Test 6 may therefore be wrong, it should use a URL template rather than issue a GET to a fixed path:

Issue an HTTP GET request to the URL with pattern /tiles/{tileMatrixSetId}/{tileMatrix}/{tileRow}/{tileCol}.

In fact the Core conformance might be difficult to test by itself, since the URL template is only provided by the TileSet conformance class.

To also support testing the Core, Tileset and Tilesets List conformance classes by themselves, the OGC API - Tiles CITE test would need to be able to accept different types of URLs as well of a selection of what type of URL it is:

joanma747 commented 3 years ago

CITE will only be able to test the core class if there is a way to provide to the test suite the template URL (that contains the endpoint for the tiles) and a range of valid values for the parameters in the template to execute the test. we recommend to start by the second conformance class and them build a user interface (with a form asking for URI templates ...) for the core.

The other comment has been applied in the section 2 conformance.

joanma747 commented 2 years ago

A note is added in the ATS to remember about this way of testing the core.