Opened 6 months ago

Last modified 8 weeks ago

#34981 assigned New feature

Promote `--pair` and `--bisect` from `runtest.py` to be available as flags for the Django `test` command

Reported by: Natalia Bidart Owned by: Sulabh Katila
Component: Testing framework Version: dev
Severity: Normal Keywords:
Cc: Adam Johnson Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

Following ​a recent conversation in the forum, it was suggested that we promote the flags --pair and --bisect to be available in the Django test command. Currently these are only implemented in the Django custom runtests.py helper script.

Change History (12)

comment:1 by Claude Paroz, 6 months ago

Triage Stage: Unreviewed → Accepted

Makes sense!

comment:3 by Tim Graham, 5 months ago

Easy pickings: unset
Type: Uncategorized → New feature

Looking at how intertwined the code for these options is with other code in runtests.py (even ​hardcoding certain Django tests apps), I doubt this is "easy pickings."

comment:5 by Sulabh Katila, 5 months ago

Owner: set to Sulabh Katila
Status: new → assigned

comment:6 by Adam Johnson, 5 months ago

Cc: Adam Johnson added

comment:7 by Sulabh Katila, 4 months ago

Has patch: set

comment:8 by Sulabh Katila, 4 months ago

Patch needs improvement: set

comment:9 by Sulabh Katila, 4 months ago

Patch needs improvement: unset

comment:10 by Tim Graham, 4 months ago

Needs documentation: set
Needs tests: set
Patch needs improvement: set

Hi Sulabh, tests and documentation are also required. Ideally, we wouldn't duplicate all the similar code in runtests.py as well.

Also, I didn't test it much, but when I tried --pair with the tutorial project, I see "Check test pairing with unittest.loader" which seems like a bug.

Last edited 4 months ago by Tim Graham (previous) (diff)

comment:11 by Sulabh Katila, 3 months ago

Hi Tim, I have made some changes and added the tests and documentation. I tested it with a sample project, and it seems to be working now.

comment:12 by Sulabh Katila, 3 months ago

Patch needs improvement: unset

comment:13 by Sulabh Katila, 2 months ago

Needs documentation: unset
Needs tests: unset

comment:14 by Mariusz Felisiak, 8 weeks ago

Patch needs improvement: set
Note: See TracTickets for help on using tickets.
Back to Top