Description:
Currently, extensions that define additional top-level properties on collection objects (e.g. item-assets) are effectively unusable because the CollectionSerializer.db_to_stac only allows the 'core' properties to be passed through. This PR changes that to allow arbitrary additional props.
That method also includes code to populate the 'core' properties with default values if they are not present. At one point while working on this I had deleted that code and noticed that doing so did not break any tests. So I added test_collection_defaults to ensure that any changes to that behavior will be caught going forward.
PR Checklist:
[x] Code is formatted and linted (run pre-commit run --all-files)
[x] Tests pass (run make test)
~- [ ] Documentation has been updated to reflect changes, if applicable~
Description: Currently, extensions that define additional top-level properties on collection objects (e.g. item-assets) are effectively unusable because the
CollectionSerializer.db_to_staconly allows the 'core' properties to be passed through. This PR changes that to allow arbitrary additional props.That method also includes code to populate the 'core' properties with default values if they are not present. At one point while working on this I had deleted that code and noticed that doing so did not break any tests. So I added
test_collection_defaultsto ensure that any changes to that behavior will be caught going forward.PR Checklist:
pre-commit run --all-files)make test) ~- [ ] Documentation has been updated to reflect changes, if applicable~