Skip to content

test: fix pytest-xdist and pytest-timeout working together - #1596

Merged
ZohebShaikh merged 6 commits into
mainfrom
pytest--fix-timeout
Aug 3, 2026
Merged

test: fix pytest-xdist and pytest-timeout working together#1596
ZohebShaikh merged 6 commits into
mainfrom
pytest--fix-timeout

Conversation

@ZohebShaikh

@ZohebShaikh ZohebShaikh commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

When I had done the PR for pyxdist I increased the timeout to 10 seconds as they where not playing nice with eachother.

I think PR I have gone through the docs/issues and found out that we can use the commands as in PR to make them work nicely with each other

@ZohebShaikh
ZohebShaikh requested a review from a team as a code owner July 15, 2026 08:56
@ZohebShaikh ZohebShaikh changed the title tests: fix pytest-xdist and pytest-timeout working together test: fix pytest-xdist and pytest-timeout working together Jul 15, 2026
@codecov

codecov Bot commented Jul 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.96%. Comparing base (30fdb32) to head (7dfd956).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1596   +/-   ##
=======================================
  Coverage   95.96%   95.96%           
=======================================
  Files          45       45           
  Lines        3317     3317           
=======================================
  Hits         3183     3183           
  Misses        134      134           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread pyproject.toml
Comment thread pyproject.toml
testpaths = "docs src tests"
asyncio_mode = "auto"
# Timeout settings for pytest-timeout
timeout = 10

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I had a look at ophyd-async they are keeping timeout at 60 seconds
So I think I kept this timeout can stay at 10 seconds

@ZohebShaikh
ZohebShaikh requested a review from tpoliaw August 3, 2026 12:47
@ZohebShaikh
ZohebShaikh added this pull request to the merge queue Aug 3, 2026
Merged via the queue into main with commit 9a145e9 Aug 3, 2026
19 checks passed
@ZohebShaikh
ZohebShaikh deleted the pytest--fix-timeout branch August 3, 2026 16:25
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