Skip to content

Fix FeatureCollection item validation#252

Open
lihouwenbin wants to merge 1 commit into
jazzband:mainfrom
lihouwenbin:codex/validate-featurecollection-features
Open

Fix FeatureCollection item validation#252
lihouwenbin wants to merge 1 commit into
jazzband:mainfrom
lihouwenbin:codex/validate-featurecollection-features

Conversation

@lihouwenbin
Copy link
Copy Markdown

Summary

  • require FeatureCollection.features items to be Feature instances during validation
  • keep validating nested Feature errors after the item type check
  • add regression tests for geometry items and nested FeatureCollections in FeatureCollection.features

Fixes #220.

Validation

  • .venv/bin/python -m pytest tests/test_validation.py::TestValidationFeatureCollection
  • .venv/bin/python -m pytest
  • git diff --check

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] FeatureCollection is valid when it contains any geojson object that is valid

1 participant