Skip to content
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

Issue63 add integration tests #104

Merged
merged 6 commits into from
Apr 17, 2023
Merged

Conversation

JohanKJSchreurs
Copy link
Contributor

Issue #63

Backend can be set with the environment variable ENDPOINT, like we have in the GeoPySpark driver's integration tests.

There is also a shell script to run the integration tests with the default backend, https://openeocloud.vito.be:

./scripts/run-integration-tests.sh

Instructions to run the integration tests have been added to the README.md

Minimal test implementations for GET /processes and GET /collections

Issue #63
Check that GET /processes and GET /collections actually returned some processes or collections.

Issue #63
@JohanKJSchreurs JohanKJSchreurs marked this pull request as ready for review April 11, 2023 07:35
integration-tests/conftest.py Outdated Show resolved Hide resolved
integration-tests/conftest.py Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
scripts/run-integration-tests.sh Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Simplified and removed some unnecessary things from conftest.py.

Better environment variable name: OPENEO_BACKEND_URL and default
backend in run-integration-tests.sh .

Issue #63
Moved a general paragraph about pytest options, as suggested in code review.

Issue #63
@JohanKJSchreurs
Copy link
Contributor Author

Currently some of the unit tests are failing, but that should be fixed by this PR: #107

The errors are related to the level parameter for log levels, which was recently introduced/changed in openeo-python-driver, openeo-python-client and openeo-geopyspark-driver for issue Open-EO/openeo-python-driver#170.

@soxofaan soxofaan merged commit 568cb01 into master Apr 17, 2023
@soxofaan soxofaan deleted the issue63_add-integration-tests branch April 17, 2023 10:11
@soxofaan soxofaan mentioned this pull request Apr 17, 2023
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