]> 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:30 +0000 (20:09 +0300)
committerSerhiy Storchaka <storchaka@gmail.com>
Mon, 14 Oct 2013 17:09:30 +0000 (20:09 +0300)
Lib/test/audiotests.py

index 3e18df14c8f5477e541417bdf8c8d61642bf793c..9c4ce6cd119b862efd8d4567212a8744743e7314 100644 (file)
@@ -145,18 +145,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