diff options
author | John Szakmeister <john@szakmeister.net> | 2013-01-08 10:47:22 -0500 |
---|---|---|
committer | John Szakmeister <john@szakmeister.net> | 2013-01-20 08:06:23 -0500 |
commit | 663deb1e3d046d1dc085183c6581b728aa37f2e6 (patch) | |
tree | 61c6a0871b16a631012d4924310fd0d4cbaa6b15 /functional_tests | |
parent | e28ee9dddd000ff3079bd5a79aabb7a4f365e5a9 (diff) | |
download | nose-663deb1e3d046d1dc085183c6581b728aa37f2e6.tar.gz |
Fix a ResourceWarning in test_shared.
Thanks to Arfrever for the patch. This fixes #593.
Diffstat (limited to 'functional_tests')
-rw-r--r-- | functional_tests/doc_tests/test_multiprocess/support/test_shared.py | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/functional_tests/doc_tests/test_multiprocess/support/test_shared.py b/functional_tests/doc_tests/test_multiprocess/support/test_shared.py index d8617f8..d798aac 100644 --- a/functional_tests/doc_tests/test_multiprocess/support/test_shared.py +++ b/functional_tests/doc_tests/test_multiprocess/support/test_shared.py @@ -17,9 +17,14 @@ def _clear(): if os.path.isfile(flag): os.unlink(flag) - + def logged(): - return [line for line in open(flag, 'r')] + flag_file = open(flag, 'r') + try: + lines = [line for line in flag_file] + finally: + flag_file.close() + return lines def setup(): @@ -31,7 +36,7 @@ def teardown(): print >> sys.stderr, "teardown called" _clear() - + def test_a(): assert len(logged()) == 1, "len(%s) !=1" % called |