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

Improve handling of long workgraph tests for use case parameters #81

Open
agoscinski opened this issue Jan 9, 2025 · 0 comments
Open

Comments

@agoscinski
Copy link
Collaborator

In PR #74 we reduced the test size as much as possible without affecting coverage but it still takes considerable amount of time to run the test (on my machine 70s). This can be still improved by doing one of the following things:

  • We could split up the test case into smaller cases to not affect coverage
  • We could run it concurrently using submit instead of run (even though I think this will not help so much with the CI because it only has two processes running)
  • The overall test time could be reduced by executing them in parallel with pytest-xdist or pytest-run-parallel (again I am not sure how much this will help with CI)
  • Mark it as long test so it can be turned of easily when developing
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

No branches or pull requests

1 participant