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

Endpoint tests with Sumo data #710

Merged
merged 23 commits into from
Jan 28, 2025
Merged

Endpoint tests with Sumo data #710

merged 23 commits into from
Jan 28, 2025

Conversation

HansKallekleiv
Copy link
Collaborator

@HansKallekleiv HansKallekleiv commented Sep 3, 2024

Closes #548

  • Added new github actions workflow for testing against Sumo prod with shared key
  • Added some endpoint tests for timeseries and explore
  • Rewrote explore router to be similar as other routers
  • Bump pytest and added pytest-timeout and pytest-asyncio plugins
  • Removed non-working parameteraccess tests

Copy link
Collaborator

@jorgenherje jorgenherje left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was there any conclusion on the issue of regarding PRs has to come from the equinor/webviz repository in order to run the endpoint test actions?

@@ -106,7 +106,7 @@ jobs:
black --check primary/ tests/
pylint primary/ tests/
bandit --recursive primary/
mypy primary/ tests/
mypy primary/
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is test-folder excluded now?

.github/workflows/backend_sumo_prod.yml Outdated Show resolved Hide resolved
@HansKallekleiv HansKallekleiv force-pushed the integration-tests branch 3 times, most recently from 70b3d5f to 5eac9e3 Compare January 25, 2025 11:33
@HansKallekleiv HansKallekleiv merged commit d050598 into main Jan 28, 2025
12 checks passed
@HansKallekleiv HansKallekleiv deleted the integration-tests branch January 28, 2025 14:57
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.

Integration tests setup
3 participants