Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix test-* make targets #230

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.

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

Merged
merged 4 commits into from
Aug 12, 2021
Merged

Fix test-* make targets #230

merged 4 commits into from
Aug 12, 2021

Conversation

duckontheweb
Copy link
Contributor

@duckontheweb duckontheweb commented Aug 12, 2021

Changes:

  • Adds a run-database make target to run just the database service for testing
    This was purely for convenience

  • Uses app-pgstac instead of app-sqlalchemy in test-pgstac target

  • Removes test-pgstac dependency on pgstac-install target
    This was causing problems when installing rasterio in my environment and we don't appear to need it since we are running the tests inside the stac-fastapi-pgstac container.

  • UPDATE: Adds run-joplin-sqlalchemy make target and adds it as a dependency to test-sqlalchemy to ensure test data is loaded prior to running unit tests.

Again, could be some user error here, so let me know if I'm off-base.

@duckontheweb duckontheweb changed the title Fix test-pgstac make target Fix test-* make targets Aug 12, 2021
@lossyrob lossyrob mentioned this pull request Aug 12, 2021
4 tasks
@lossyrob lossyrob merged commit 8e3eb99 into stac-utils:master Aug 12, 2021
@duckontheweb duckontheweb deleted the fix/make-test-pgstac branch August 13, 2021 01:13
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.

2 participants