chore: Improve raster integration tests #397
Conversation
| seeded_collection = "noaa-emergency-response" | ||
| seeded_id = "20200307aC0853900w361030" | ||
| z, x, y = 15, 8589, 12849 |
There was a problem hiding this comment.
Thanks for implementing this - would this PR be a good time to define these constants as fixtures? Later on, when we implement more automated integration tests, we could use these to test live environments by overriding the endpoint and items for the tests.
There was a problem hiding this comment.
Thanks for the guidance. I think the updates are there now
.github/workflows/tests/conftest.py
Outdated
| STAC_HEALTH_ENDPOINT = "http://0.0.0.0:8081/_mgmt/ping" | ||
| SEARCH_ROUTE = "search" | ||
|
|
||
| RASTER_ENDPOINT = "http://0.0.0.0:8082/searches" |
There was a problem hiding this comment.
nit: I think we only had like one test for other prefixes in the raster API (i.e. /cog, /searches, stac /collections/{collection_id}/items) but maybe we should either make this RASTER_SEARCHES_ENDPOINT or remove the /searches and add it to the tests on searches? I think there was previously one stac search test /collections/{collection_id}/items{item_id} but we haven't had all that much coverage.
|
nit: can we bring these item tests back next time we are working in raster API? I'm pretty sure the dashboard uses these |
…ltiple domain names for prod, Improve raster integration tests, remove moto from setup. leave in requirements_dev (#423) ### 🪙 Changed/Updated - [chore: Improve raster integration tests #397](#397) - [chore: remove moto from setup. leave in requirements_dev #421](#421) ### 🩹 Fixed - [fix: update domain names to support multiple domain names for prod](#418 ) <details> <summary>Authors</summary> - Co-authored-by: Jennifer Tran - Co-authored-by: Stephen Kilbourn </details>
Issue
#384
What?
This updates the existing raster integration tests to cover more of the endpoints listed in the docs. It also pulls down the Openapi Schemas from https://api.stacspec.org/v1.0.0 and validates the feature and collection results match those schemas.
Testing?
this will run with the existing script: ./scripts/run-local-tests.sh