classification
Title: Dev Guide should say how to run tests in 2.7
Type: enhancement Stage: resolved
Components: Devguide, Documentation Versions:
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: ezio.melotti Nosy List: chris.jerdonek, docs@python, ezio.melotti, python-dev
Priority: normal Keywords: easy, patch

Created on 2012-08-01 09:07 by chris.jerdonek, last changed 2012-08-02 07:23 by ezio.melotti. This issue is now closed.

Files
File name Uploaded Description Edit
issue-15521-1.patch chris.jerdonek, 2012-08-01 09:58
issue-15521-2.patch chris.jerdonek, 2012-08-02 00:36
Messages (8)
msg167099 - (view) Author: Chris Jerdonek (chris.jerdonek) * (Python committer) Date: 2012-08-01 09:07
The Dev Guide should say the preferred way (or even a way) to run tests using the Python 2.7 build:

http://docs.python.org/devguide/runtests.html

This will help submitters who want to check that their patch works with 2.7.

In Python 2.7, this did not work (for obvious reasons):

$ ./python.exe -m test
.../python.exe: No module named test.__main__; 'test' is a package and cannot be directly executed

But this did:

$ ./python.exe -m test.regrtest

Is the latter the preferred way?
msg167103 - (view) Author: Ezio Melotti (ezio.melotti) * (Python committer) Date: 2012-08-01 09:17
FWIW this is already mentioned in the quick start [0].
Do you want to propose a patch for runtest.rst?

[0]: http://docs.python.org/devguide/#quick-start
msg167104 - (view) Author: Chris Jerdonek (chris.jerdonek) * (Python committer) Date: 2012-08-01 09:19
> FWIW this is already mentioned in the quick start [0].

I didn't catch that, thanks.

> Do you want to propose a patch for runtest.rst?

Sure.
msg167108 - (view) Author: Chris Jerdonek (chris.jerdonek) * (Python committer) Date: 2012-08-01 09:58
Attaching a patch for review.
msg167140 - (view) Author: Ezio Melotti (ezio.melotti) * (Python committer) Date: 2012-08-01 15:40
Thanks for the patch.
I'm not sure it's necessary to add a new section though, unless you are planning to add more differences there.

+base command instead of the one described above: ::

You can just use "... above::" here.
msg167183 - (view) Author: Chris Jerdonek (chris.jerdonek) * (Python committer) Date: 2012-08-02 00:36
It is a relatively large section (nine paragraphs + six code snippets) and is primarily about the in-development branch, so I did not want to interrupt the flow with a remark specific to 2.7.  I also liked that the instructions in a subsection could be linked to.

But here is a new patch.
msg167195 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2012-08-02 07:21
New changeset 851adff2d434 by Ezio Melotti in branch 'default':
#15521: mention that test.regrtest is necessary to run tests on 2.7.  Patch by Chris Jerdonek.
http://hg.python.org/devguide/rev/851adff2d434
msg167196 - (view) Author: Ezio Melotti (ezio.melotti) * (Python committer) Date: 2012-08-02 07:23
Fixed, thanks for the patch!
History
Date User Action Args
2012-08-02 07:23:44ezio.melottisetstatus: open -> closed
messages: + msg167196

assignee: docs@python -> ezio.melotti
resolution: fixed
stage: patch review -> resolved
2012-08-02 07:21:42python-devsetnosy: + python-dev
messages: + msg167195
2012-08-02 00:36:40chris.jerdoneksetfiles: + issue-15521-2.patch

messages: + msg167183
2012-08-01 15:40:15ezio.melottisetmessages: + msg167140
2012-08-01 09:58:43chris.jerdoneksetfiles: + issue-15521-1.patch
keywords: + patch
messages: + msg167108

stage: needs patch -> patch review
2012-08-01 09:19:38chris.jerdoneksetmessages: + msg167104
2012-08-01 09:17:48ezio.melottisettype: enhancement
messages: + msg167103
2012-08-01 09:07:47chris.jerdonekcreate