You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
👋 we've recently released mvt-fixtures as an attempt to centralize test fixtures for decoding mvt buffers. I'd love to see vector-tile-js using this library at some point in the future. Presumably mvt-fixtures doesn't contain every unit-test fixture required for this library, so I'll work on updates there to make sure there is feature parity before working on a branch in this project.
mvt-fixtures comes with a method for testing all fixtures relatively quickly - .each() and this would allow us to run vector-tile-js through every fixture plus test whether it should decode the fixture successfully or not. Example in the README.
It also comes with a selection of real-world tiles that can be used for benchmarks and tests like this one.
I'll plan on being point here for prototyping what this can look like, but if anyone has thoughts or concerns please feel free to raise them here!
👋 we've recently released mvt-fixtures as an attempt to centralize test fixtures for decoding mvt buffers. I'd love to see vector-tile-js using this library at some point in the future. Presumably mvt-fixtures doesn't contain every unit-test fixture required for this library, so I'll work on updates there to make sure there is feature parity before working on a branch in this project.
mvt-fixtures comes with a method for testing all fixtures relatively quickly -
.each()
and this would allow us to run vector-tile-js through every fixture plus test whether it should decode the fixture successfully or not. Example in the README.It also comes with a selection of real-world tiles that can be used for benchmarks and tests like this one.
I'll plan on being point here for prototyping what this can look like, but if anyone has thoughts or concerns please feel free to raise them here!
cc @mapbox/core-tech @jfirebaugh @lbud @mollymerp @mourner @anandthakker @lucaswoj
The text was updated successfully, but these errors were encountered: