#27028 closed Cleanup/optimization (fixed)
Clarify failed parallel tests run message
| Reported by: | Adam Zapletal | Owned by: | Adam Zapletal |
|---|---|---|---|
| Component: | Testing framework | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
If the test suite fails while running with more than one process, and the traceback is cannot be pickled, this message is printed:
You should re-run this test without the --parallel option to reproduce the failure and get a correct traceback.
The CPU count on my machine is 4, and the test suite runs with 4 processors if I don't provide the --parallel option or if I provide it with no argument. So the message is misleading.
Perhaps the error message should say:
You should re-run this test with the "--parallel 1" option to reproduce the failure and get a correct traceback.
I'll be happy to submit a PR if this ticket is accepted.
Change History (6)
comment:1 by , 8 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:2 by , 8 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:4 by , 8 years ago
| Has patch: | set |
|---|
Note:
See TracTickets
for help on using tickets.
I believe this message can show up:
The message is appropriate for the latter case but not for the former.
--parallel=1covers both use cases. That's a good solution.