]> granicus.if.org Git - python/commitdiff
Issue #18919: Fixed resource leaks in audio tests.
authorSerhiy Storchaka <storchaka@gmail.com>
Mon, 14 Oct 2013 17:09:47 +0000 (20:09 +0300)
committerSerhiy Storchaka <storchaka@gmail.com>
Mon, 14 Oct 2013 17:09:47 +0000 (20:09 +0300)
Lib/test/audiotests.py

index 147cda002a555a7c1e84fb9cce700bbd1e037e3c..59e9928796dc9e132731f9762fd1dd16f55804bc 100644 (file)
@@ -141,18 +141,18 @@ class AudioTestsWithSourceFile(AudioTests):
                           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