Opened 4 years ago

Closed 3 years ago

#31370 closed New feature (fixed)

Make parallel test runner work with buffer option.

Reported by: Adam Johnson Owned by: Adam Johnson
Component: Testing framework Version: dev
Severity: Normal Keywords:
Cc: Baptiste Mispelon Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

#27430 added support for unittest's --buffer option, but only when not using --parallel. Since --parallel is normally very useful to speed up tests, it would be good to make the options work together. I think this can be done by copying some code from unittest into the RemoteTestResult class.

Change History (12)

comment:1 by Mariusz Felisiak, 4 years ago

Cc: Baptiste Mispelon added
Resolution: needsinfo
Status: assignedclosed
Summary: Make parallel test runner work with buffer optionMake parallel test runner work with buffer option.

I'm not sure if it's feasible (see comment). I think we need a proof of concept, before we will accept this. Would you like to prepare a patch?

comment:2 by Adam Johnson, 4 years ago

Has patch: set

comment:3 by Adam Johnson, 4 years ago

Resolution: needsinfo
Status: closednew

comment:4 by Mariusz Felisiak, 4 years ago

Owner: changed from nobody to Adam Johnson
Status: newassigned
Triage Stage: UnreviewedAccepted

comment:5 by Mariusz Felisiak, 4 years ago

Patch needs improvement: set

comment:6 by Adam Johnson, 3 years ago

Patch needs improvement: unset

comment:7 by Mariusz Felisiak, 3 years ago

Patch needs improvement: set

comment:8 by Adam Johnson, 3 years ago

Patch needs improvement: unset

comment:9 by Mariusz Felisiak, 3 years ago

Triage Stage: AcceptedReady for checkin

comment:10 by Mariusz Felisiak <felisiak.mariusz@…>, 3 years ago

In 92975bcd:

Refs #31370 -- Added test for pickling RemoteTestResult.

comment:11 by Mariusz Felisiak <felisiak.mariusz@…>, 3 years ago

In e3bca22e:

Refs #31370 -- Made RemoteTestResult subclass unittest.TestResult.

comment:12 by Mariusz Felisiak <felisiak.mariusz@…>, 3 years ago

Resolution: fixed
Status: assignedclosed

In f6713cda:

Fixed #31370 -- Added support for parallel tests with --buffer.

Note: See TracTickets for help on using tickets.
Back to Top