mapbox / vector-tile-cs

Parses vector tiles with C#.
BSD 3-Clause "New" or "Revised" License
57 stars 37 forks source link

Use mvt-fixtures for test data #5

Closed springmeyer closed 7 years ago

springmeyer commented 7 years ago

https://github.com/mapbox/mvt-fixtures exists, thanks to @mapsam, to provide very small and explicit test fixtures for writing a vtile decoder. We should be using https://github.com/mapbox/mvt-fixtures/tree/master/fixtures/valid and adding more fixtures to https://github.com/mapbox/mvt-fixtures/tree/master/fixtures/valid if things are missing that are needed (instead of custom data).

/cc @BergWerkGIS

springmeyer commented 7 years ago

I recommend pulling in via submodule like: git submodule add https://github.com/mapbox/mvt-fixtures.git test/mvt-fixtures

wilhelmberg commented 7 years ago

First iteration of tests uses https://github.com/mapbox/mvt-fixtures/:

https://github.com/mapbox/vector-tile-cs/blob/816f55615720905598bea4ee9f8128d9560195a0/src/VectorTiles.Tests/TestAllMVTs.cs

wilhelmberg commented 7 years ago

Done - using valid as well as invalid fixtures.

springmeyer commented 7 years ago

\o/

wilhelmberg commented 7 years ago

AppVeyor does a good job of displaying the test results: https://ci.appveyor.com/project/Mapbox/vector-tile-cs/build/1.0.5/job/yqccevo2wf87wpn9/tests

image