Open Bug 1433941 Opened 2 years ago Updated 2 years ago

Add --no-capture pass-through for pytests

Categories

(Testing :: Python Test, enhancement)

Version 3
enhancement
Not set

Tracking

(Not tracked)

People

(Reporter: whimboo, Unassigned)

Details

For debugging purposes it would be good to have support for --no-capture in `mach python-test`. Right now this option doesn't seem to be supported.
How may i proceed to solve this issue ?
Hi Kamit! You can find the pytest options here:

https://dxr.mozilla.org/mozilla-central/source/config/mozunit.py#229

And you would have to add the capture option like here:

https://searchfox.org/mozilla-central/source/testing/web-platform/tests/tools/wptrunner/wptrunner/executors/pytestrunner/runner.py#54-61

Andrew, would it make sense to also take the other options as used by wdspec tests?
Flags: needinfo?(ahalberstadt)
So to clarify, we're just adding the ability to specify this on the command line right? Not making it the default.

I was thinking we could use argparse.REMAINDER to get all unrecognized args in |mach python-test| (defined in python/mach_commands.py), then just forward those on to the test subprocess and have the subprocess append them to the pytest args.
Flags: needinfo?(ahalberstadt)
You need to log in before you can comment on or make changes to this bug.