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

Integration tests #63

Closed
soxofaan opened this issue Aug 3, 2022 · 6 comments
Closed

Integration tests #63

soxofaan opened this issue Aug 3, 2022 · 6 comments

Comments

@soxofaan
Copy link
Member

soxofaan commented Aug 3, 2022

From #52

We currently only have unit tests (some of which actually go pretty far to do integration test kind of checks against dummy back-ends), but it would be good if there was also a bit of real integration tests against real back-ends

@soxofaan
Copy link
Member Author

Idea to be covered:
merging of collections (#5) could trigger some warnings or even errors (e.g. incompatible metadata from upstream providers). An integration test could check for these warnings/errors

@soxofaan
Copy link
Member Author

#76 added initial reporting tool for collection and process metadata merging issues. This should be integrated in the integration tests

@soxofaan
Copy link
Member Author

soxofaan commented Apr 4, 2023

To start this task simple:

JohanKJSchreurs added a commit that referenced this issue Apr 7, 2023
JohanKJSchreurs added a commit that referenced this issue Apr 11, 2023
Minimal test implementations for GET /processes and GET /collections

Issue #63
JohanKJSchreurs added a commit that referenced this issue Apr 11, 2023
Check that GET /processes and GET /collections actually returned some processes or collections.

Issue #63
JohanKJSchreurs added a commit that referenced this issue Apr 13, 2023
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
JohanKJSchreurs added a commit that referenced this issue Apr 13, 2023
Moved a general paragraph about pytest options, as suggested in code review.

Issue #63
soxofaan added a commit that referenced this issue Apr 17, 2023
Issue #63: add initial integration test skeleton
@soxofaan
Copy link
Member Author

just merged initial skeleton of PR #104

next steps:

  • Figure out how to do get authenticated connection (without exposing secrets in this repo)
  • set up jenkins flow to run this regularly or trigger based (e.g. pre-release check)

@soxofaan
Copy link
Member Author

I've set up basic configs for running integration tests in jenkins (triggered at end of normal openeo-aggregator run)

@soxofaan
Copy link
Member Author

soxofaan commented May 12, 2023

next step of working with authenticated connection is probably best done through:

and getting client credentials from Vault (which will be easier than messing around with private repo's)

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

No branches or pull requests

3 participants