Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#12929 closed (wontfix)

runtests.py should reference the checkout it is part of

Reported by: termie Owned by: nobody
Component: Testing framework Version: master
Severity: Keywords:
Cc: jbronn@… Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

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 Changed 5 years ago by russellm

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Accepted

comment:2 Changed 5 years ago by kmtracey

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.

comment:3 Changed 5 years ago by russellm

  • Resolution set to wontfix
  • Status changed from new to 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.

comment:4 Changed 5 years ago by david

Just for reference: #9947.

Note: See TracTickets for help on using tickets.
Back to Top