Repository navigation
Support Django Test Runs with Coverage #24199
Description
make sure Coverage works while running Django and that the scenario is supported.
originally raised in #22827 (comment)
{{ message }}
make sure Coverage works while running Django and that the scenario is supported.
originally raised in #22827 (comment)
Activity
slank commentedon Feb 16, 2025
The problem occurs because coverage is run in-process using
coverage.start()(here), but the test runs under a subprocess and is unobservable to coverage.Here are some approaches to fixing it.
Keep
coveragein-process. Also runmanage.pyin-process.Three variations on this:
exec()to run manage.py in the same interpreter. This has a bad security smell but it's hard to say if there's really any difference from ...main()(here). Because manage.py is hard-coded to read its arguments fromsys.argv, this approach would require manipulation of that variable.Don't run
coveragein-process. Callmanage.pyas designedIn this case, we'd simply replace
[sys.executable](here) with["coverage", "run"]. The test execution code would need to be cleaned up.I think all of these have their plusses and minuses. This is my first time here and I'm just working on a weekend project so I don't know the backstory on the current implementation. I like the
execute_from_command_line()approach for simplicity.mcobalchinisoftfocus commentedon Mar 7, 2025
I read the documentation about testing and it doesn't mention that coverage in Django tests is not yet supported, so I was lost in the dark until I found this issue right here, I think it would be nice if this information was added as a disclaimer
eleanorjboyd commentedon Mar 7, 2025
great point Matheus Eduardo Cobalchini (@mcobalchinisoftfocus)! Courtney Webster (@cwebster-99), could we include this in our docs update?
danila-grobov commentedon Mar 23, 2025
Hi,
Eleanor Boyd (@eleanorjboyd) Could you take a look at my PR #24927? I think I've fixed the issue.
Django Test Runs with Coverage (#24927)
Test Coverage for older django versions (#24968)