[Python-checkins] cpython (2.7): Issue #18919: Fixed resource leaks in audio tests.

serhiy.storchaka python-checkins at python.org
Mon Oct 14 19:10:42 CEST 2013


http://hg.python.org/cpython/rev/fd7752db1e54
changeset:   86370:fd7752db1e54
branch:      2.7
parent:      86367:164a60cce934
user:        Serhiy Storchaka <storchaka at gmail.com>
date:        Mon Oct 14 20:09:30 2013 +0300
summary:
  Issue #18919: Fixed resource leaks in audio tests.

files:
  Lib/test/audiotests.py |  24 ++++++++++++------------
  1 files changed, 12 insertions(+), 12 deletions(-)


diff --git a/Lib/test/audiotests.py b/Lib/test/audiotests.py
--- a/Lib/test/audiotests.py
+++ b/Lib/test/audiotests.py
@@ -145,18 +145,18 @@
                           self.sndfilenframes, self.comptype, self.compname)
 
     def test_close(self):
-        testfile = open(self.sndfilepath, 'rb')
-        f = self.f = self.module.open(testfile)
-        self.assertFalse(testfile.closed)
-        f.close()
-        self.assertEqual(testfile.closed, self.close_fd)
-        testfile = open(TESTFN, 'wb')
-        fout = self.module.open(testfile, 'wb')
-        self.assertFalse(testfile.closed)
-        with self.assertRaises(self.module.Error):
-            fout.close()
-        self.assertEqual(testfile.closed, self.close_fd)
-        fout.close() # do nothing
+        with open(self.sndfilepath, 'rb') as testfile:
+            f = self.f = self.module.open(testfile)
+            self.assertFalse(testfile.closed)
+            f.close()
+            self.assertEqual(testfile.closed, self.close_fd)
+        with open(TESTFN, 'wb') as testfile:
+            fout = self.fout = self.module.open(testfile, 'wb')
+            self.assertFalse(testfile.closed)
+            with self.assertRaises(self.module.Error):
+                fout.close()
+            self.assertEqual(testfile.closed, self.close_fd)
+            fout.close() # do nothing
 
     def test_read(self):
         framesize = self.nchannels * self.sampwidth

-- 
Repository URL: http://hg.python.org/cpython


More information about the Python-checkins mailing list