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

Fix issue with running only selected tests #2155

Merged
merged 1 commit into from
Feb 3, 2025
Merged

Conversation

grusev
Copy link
Collaborator

@grusev grusev commented Jan 30, 2025

The job allows to run a selected test only takingan argument. However the quotes are making it not useful and cannot fulful purpos.

Now with this fix it is possible to run tests with such additional line:

-s -k myTest

That saves enormous time and efforts especially with tests with real storages

Reference Issues/PRs

What does this implement or fix?

Any other comments?

Checklist

Checklist for code changes...
  • Have you updated the relevant docstrings, documentation and copyright notice?
  • Is this contribution tested against all ArcticDB's features?
  • Do all exceptions introduced raise appropriate error messages?
  • Are API changes highlighted in the PR description?
  • Is the PR labelled as enhancement or bug so it appears in autogenerated release notes?

The job allows to run a selected test only takingan argument. However the quotes are making it not useful and cannot fulful purpos.

Now with this fix it is possible to run tests with such additional line:

-s -k myTest

That saves enormous time and efforts especially with tests with real storages
@grusev grusev merged commit 4a2fc47 into master Feb 3, 2025
151 of 152 checks passed
@grusev grusev deleted the fix/parallel_test_run branch February 3, 2025 07:00
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.

3 participants