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.

Sign up for GitHub

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

Jump to bottom

Add unit tests for carto-cloud-native #143

Merged

alasarr merged 1 commit into carto-cloud-native from tests/carto-cloud-native

May 25, 2021

Contributor

AdriSolid commented May 24, 2021

For: https://app.clubhouse.io/cartoteam/story/155287/adapt-tests

A little fix was implemented in viewportFeaturesGeoJSON. It was not correctly handling the uniqueIdProperty accessibility.
Current coverage is 82.28%


          feat: add test for geojson && minor viewport features fix

bb83066

Contributor Author

AdriSolid commented May 24, 2021

CI is failing because of the local link of @deck.gl/carto

alasarr approved these changes

View reviewed changes

alasarr merged commit ffc9cac into carto-cloud-native

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

Labels

None yet