[New-bugs-announce] [issue6024] regrtest says refleaks are "ok"

[Collin Winter]

New submission from Collin Winter <collinw at gmail.com>:

Currently (r72643), regrtest.py -R:: says that a test passed even if it
leaked references:

trunk collinwinter$ ./python.exe Lib/test/regrtest.py -R::
test_urllib2_localnet
test_urllib2_localnet
beginning 9 repetitions
123456789
.........
test_urllib2_localnet leaked [3, 3, 3, 3] references, sum=12
1 test OK.
[50020 refs]
trunk collinwinter$

The attached patch turns that into

trunk collinwinter$ ./python.exe Lib/test/regrtest.py -R::
test_urllib2_localnet
test_urllib2_localnet
beginning 9 repetitions
123456789
.........
test_urllib2_localnet leaked [3, 3, 3, 3] references, sum=12
1 test failed:
    test_urllib2_localnet
[50010 refs]
trunk collinwinter$ echo $?
1
trunk collinwinter$

This makes it easier to run regrtest.py -R:: as part of a buildbot
installation.

I'll merge to py3k once this is reviewed. Any thoughts on merging to
release26-maint?

----------
components: Tests
files: refleak.patch
keywords: patch
messages: 87776
nosy: collinwinter, jyasskin
severity: normal
stage: patch review
status: open
title: regrtest says refleaks are "ok"
type: behavior
versions: Python 2.7
Added file: http://bugs.python.org/file13985/refleak.patch

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue6024>
_______________________________________