tests/extra: add timeout= argument to run.communicate() - #886
Conversation
5c86b77 to
b3da251
Compare
(non-asyncio) subprocess.Popen.communicate() does have a timeout argument. Since the extra tests wrapper minic the non-asyncio version, add this parameter too.
b3da251 to
9d032e7
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #886 +/- ##
=======================================
Coverage 70.78% 70.78%
=======================================
Files 61 61
Lines 14358 14358
=======================================
Hits 10164 10164
Misses 4194 4194
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
OpenQA test summaryComplete test suite and dependencies: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2026091002-devel&flavor=pull-requests Test run included the following:
New failures, excluding unstableCompared to: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2026050504-devel&flavor=update
Failed tests12 failures
Fixed failuresCompared to: https://openqa.qubes-os.org/tests/176874#dependencies 37 fixed
Unstable testsDetails
Performance TestsPerformance degradation:No issues Remaining performance tests:72 tests
|
2 participants
(non-asyncio) subprocess.Popen.communicate() does have a timeout
argument. Since the extra tests wrapper minic the non-asyncio version,
add this parameter too.