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

Add parallel tests to Github actions workflow #164

Merged
merged 3 commits into from
Sep 30, 2024

Conversation

shanejbrown
Copy link
Contributor

What does this PR do?

Speed up the workflow tests.

What issues does this PR fix or reference?

Previous Behavior

New Behavior

Merge requirements satisfied?

  • I have updated the documentation or no documentation changes are required.
  • I have added tests to cover my changes.
  • I have updated the base version in setup.py (if appropriate).

@shanejbrown shanejbrown changed the title Parallel tests Add parallel tests and change source image removal failure to be a warning Sep 30, 2024
@shanejbrown shanejbrown changed the title Add parallel tests and change source image removal failure to be a warning Add parallel tests to Github actions workflow Sep 30, 2024
@@ -50,7 +55,10 @@ jobs:
chmod 700 ~/.ssh
chmod 600 ~/.ssh/buildrunner-deploy-*
- name: Test with pytest
run: pytest -v --junitxml=test-reports/test-results.xml
run: |
pytest -v -m "not serial" --numprocesses=auto --junitxml=test-reports/serial-test-results.xml
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't these file names be switched?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. Only if you don't want to confuse people.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@shanejbrown shanejbrown merged commit 122d7fd into adobe:main Sep 30, 2024
9 checks passed
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