#12929 closed (wontfix)
runtests.py should reference the checkout it is part of
| Reported by: | termie | Owned by: | nobody |
|---|---|---|---|
| Component: | Testing framework | Version: | dev |
| Severity: | Keywords: | ||
| Cc: | jbronn@… | Triage Stage: | Accepted |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
At current runtests.py attempts to test the first installation of django it finds on the path, which, without any modifications, will generally mean it tests the version of django installed on the system rather than the local checkout.
My argument is that there is very little reason for runtests.py to test anything other than the local checkout it is part of.
The easiest solution would appear to be to prepend "../" to sys.path at the beginning of the script if it is run from the tests directory.
Change History (4)
comment:1 by , 15 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:2 by , 15 years ago
comment:3 by , 15 years ago
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
I'm only a mild +0, so if there is someone in the core who has a use for the existing behavior, I'm happy to call this a wontfix.
Note:
See TracTickets
for help on using tickets.
Not that I haven't been bitten by it, but I actually do use the current behavior to run new tests from patches against old level code. I'm not particularly eager to see the described change implemented.