Merged revisions 76934-76935 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk ........ r76934 | r.david.murray | 2009-12-20 11:24:46 -0500 (Sun, 20 Dec 2009) | 2 lines Fix comment typo. ........ r76935 | r.david.murray | 2009-12-20 11:46:06 -0500 (Sun, 20 Dec 2009) | 10 lines Issue #7376: When called with no arguments doctest was running a self-test. Because of a change to the way tracebacks are printed, this self-test was failing. The test is run (and passes) during normal regression testing. So instead of running the failing self-test this patch makes doctest emit a usage message. This is better behavior anyway since passing in arguments is the real reason to run doctest as a command. Bug discovery and initial patch by Florent Xicluna. ........
This commit is contained in:
parent
1554b18b5a
commit
445448ca2d
|
@ -2611,14 +2611,20 @@ __test__ = {"_TestClass": _TestClass,
|
|||
""",
|
||||
}
|
||||
|
||||
|
||||
def _test():
|
||||
testfiles = [arg for arg in sys.argv[1:] if arg and arg[0] != '-']
|
||||
if testfiles:
|
||||
if not testfiles:
|
||||
name = os.path.basename(sys.argv[0])
|
||||
if '__loader__' in globals() and name.endswith('.py'): # python -m
|
||||
name, _ = os.path.splitext(name)
|
||||
print("usage: {0} [-v] file ...".format(name))
|
||||
return 2
|
||||
for filename in testfiles:
|
||||
if filename.endswith(".py"):
|
||||
# It is a module -- insert its dir into sys.path and try to
|
||||
# import it. If it is part of a package, that possibly won't work
|
||||
# because of package imports.
|
||||
# import it. If it is part of a package, that possibly
|
||||
# won't work because of package imports.
|
||||
dirname, filename = os.path.split(filename)
|
||||
sys.path.insert(0, dirname)
|
||||
m = __import__(filename[:-3])
|
||||
|
@ -2628,10 +2634,8 @@ def _test():
|
|||
failures, _ = testfile(filename, module_relative=False)
|
||||
if failures:
|
||||
return 1
|
||||
else:
|
||||
r = unittest.TextTestRunner()
|
||||
r.run(DocTestSuite())
|
||||
return 0
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
sys.exit(_test())
|
||||
|
|
|
@ -125,7 +125,7 @@ def _run_module_as_main(mod_name, alter_argv=True):
|
|||
|
||||
Note that the executed module will have full access to the
|
||||
__main__ namespace. If this is not desirable, the run_module()
|
||||
function sbould be used to run the module code in a fresh namespace.
|
||||
function should be used to run the module code in a fresh namespace.
|
||||
|
||||
At the very least, these variables in __main__ will be overwritten:
|
||||
__name__
|
||||
|
|
|
@ -511,6 +511,9 @@ Documentation
|
|||
Tests
|
||||
-----
|
||||
|
||||
- Issue #7376: instead of running a self-test (which was failing) when called
|
||||
with no arguments, doctest.py now gives a usage message.
|
||||
|
||||
- Issue #7396: fix regrtest -s, which was broken by the -j enhancement.
|
||||
|
||||
- Issue #7498: test_multiprocessing now uses test.support.find_unused_port
|
||||
|
|
Loading…
Reference in New Issue