[Python-checkins] r74011 - python/trunk/Lib/test/test_zipfile.py

ezio.melotti python-checkins at python.org
Wed Jul 15 19:07:04 CEST 2009


Author: ezio.melotti
Date: Wed Jul 15 19:07:04 2009
New Revision: 74011

Log:
methods' names pep8ification

Modified:
   python/trunk/Lib/test/test_zipfile.py

Modified: python/trunk/Lib/test/test_zipfile.py
==============================================================================
--- python/trunk/Lib/test/test_zipfile.py	(original)
+++ python/trunk/Lib/test/test_zipfile.py	Wed Jul 15 19:07:04 2009
@@ -39,7 +39,7 @@
         fp.write(self.data)
         fp.close()
 
-    def makeTestArchive(self, f, compression):
+    def make_test_archive(self, f, compression):
         # Create the ZIP archive
         zipfp = zipfile.ZipFile(f, "w", compression)
         zipfp.write(TESTFN, "another"+os.extsep+"name")
@@ -47,8 +47,8 @@
         zipfp.writestr("strfile", self.data)
         zipfp.close()
 
-    def zipTest(self, f, compression):
-        self.makeTestArchive(f, compression)
+    def zip_test(self, f, compression):
+        self.make_test_archive(f, compression)
 
         # Read the ZIP archive
         zipfp = zipfile.ZipFile(f, "r", compression)
@@ -105,12 +105,12 @@
         zipfp.testzip()
         zipfp.close()
 
-    def test_Stored(self):
+    def test_stored(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipTest(f, zipfile.ZIP_STORED)
+            self.zip_test(f, zipfile.ZIP_STORED)
 
-    def zipOpenTest(self, f, compression):
-        self.makeTestArchive(f, compression)
+    def zip_open_test(self, f, compression):
+        self.make_test_archive(f, compression)
 
         # Read the ZIP archive
         zipfp = zipfile.ZipFile(f, "r", compression)
@@ -134,11 +134,11 @@
         self.assertEqual(''.join(zipdata2), self.data)
         zipfp.close()
 
-    def test_OpenStored(self):
+    def test_open_stored(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipOpenTest(f, zipfile.ZIP_STORED)
+            self.zip_open_test(f, zipfile.ZIP_STORED)
 
-    def test_OpenViaZipInfo(self):
+    def test_open_via_zip_info(self):
         # Create the ZIP archive
         zipfp = zipfile.ZipFile(TESTFN2, "w", zipfile.ZIP_STORED)
         zipfp.writestr("name", "foo")
@@ -157,8 +157,8 @@
         self.assertTrue(data == "foobar" or data == "barfoo")
         zipfp.close()
 
-    def zipRandomOpenTest(self, f, compression):
-        self.makeTestArchive(f, compression)
+    def zip_random_open_test(self, f, compression):
+        self.make_test_archive(f, compression)
 
         # Read the ZIP archive
         zipfp = zipfile.ZipFile(f, "r", compression)
@@ -173,12 +173,12 @@
         self.assertEqual(''.join(zipdata1), self.data)
         zipfp.close()
 
-    def test_RandomOpenStored(self):
+    def test_random_open_stored(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipRandomOpenTest(f, zipfile.ZIP_STORED)
+            self.zip_random_open_test(f, zipfile.ZIP_STORED)
 
-    def zipReadlineTest(self, f, compression):
-        self.makeTestArchive(f, compression)
+    def zip_readline_test(self, f, compression):
+        self.make_test_archive(f, compression)
 
         # Read the ZIP archive
         zipfp = zipfile.ZipFile(f, "r")
@@ -189,8 +189,8 @@
 
         zipfp.close()
 
-    def zipReadlinesTest(self, f, compression):
-        self.makeTestArchive(f, compression)
+    def zip_readlines_test(self, f, compression):
+        self.make_test_archive(f, compression)
 
         # Read the ZIP archive
         zipfp = zipfile.ZipFile(f, "r")
@@ -200,8 +200,8 @@
 
         zipfp.close()
 
-    def zipIterlinesTest(self, f, compression):
-        self.makeTestArchive(f, compression)
+    def zip_iterlines_test(self, f, compression):
+        self.make_test_archive(f, compression)
 
         # Read the ZIP archive
         zipfp = zipfile.ZipFile(f, "r")
@@ -210,50 +210,50 @@
 
         zipfp.close()
 
-    def test_ReadlineStored(self):
+    def test_readline_stored(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipReadlineTest(f, zipfile.ZIP_STORED)
+            self.zip_readline_test(f, zipfile.ZIP_STORED)
 
-    def test_ReadlinesStored(self):
+    def test_readlines_stored(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipReadlinesTest(f, zipfile.ZIP_STORED)
+            self.zip_readlines_test(f, zipfile.ZIP_STORED)
 
-    def test_IterlinesStored(self):
+    def test_iterlines_stored(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipIterlinesTest(f, zipfile.ZIP_STORED)
+            self.zip_iterlines_test(f, zipfile.ZIP_STORED)
 
     @skipUnless(zlib, "requires zlib")
-    def test_Deflated(self):
+    def test_deflated(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipTest(f, zipfile.ZIP_DEFLATED)
+            self.zip_test(f, zipfile.ZIP_DEFLATED)
 
     @skipUnless(zlib, "requires zlib")
-    def test_OpenDeflated(self):
+    def test_open_deflated(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipOpenTest(f, zipfile.ZIP_DEFLATED)
+            self.zip_open_test(f, zipfile.ZIP_DEFLATED)
 
     @skipUnless(zlib, "requires zlib")
-    def test_RandomOpenDeflated(self):
+    def test_random_open_deflated(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipRandomOpenTest(f, zipfile.ZIP_DEFLATED)
+            self.zip_random_open_test(f, zipfile.ZIP_DEFLATED)
 
     @skipUnless(zlib, "requires zlib")
-    def test_ReadlineDeflated(self):
+    def test_readline_deflated(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipReadlineTest(f, zipfile.ZIP_DEFLATED)
+            self.zip_readline_test(f, zipfile.ZIP_DEFLATED)
 
     @skipUnless(zlib, "requires zlib")
-    def test_ReadlinesDeflated(self):
+    def test_readlines_deflated(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipReadlinesTest(f, zipfile.ZIP_DEFLATED)
+            self.zip_readlines_test(f, zipfile.ZIP_DEFLATED)
 
     @skipUnless(zlib, "requires zlib")
-    def test_IterlinesDeflated(self):
+    def test_iterlines_deflated(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipIterlinesTest(f, zipfile.ZIP_DEFLATED)
+            self.zip_iterlines_test(f, zipfile.ZIP_DEFLATED)
 
     @skipUnless(zlib, "requires zlib")
-    def test_LowCompression(self):
+    def test_low_compression(self):
         # Checks for cases where compressed data is larger than original
         # Create the ZIP archive
         zipfp = zipfile.ZipFile(TESTFN2, "w", zipfile.ZIP_DEFLATED)
@@ -266,7 +266,7 @@
         self.assertEqual(openobj.read(1), '1')
         self.assertEqual(openobj.read(1), '2')
 
-    def test_AbsoluteArcnames(self):
+    def test_absolute_arcnames(self):
         zipfp = zipfile.ZipFile(TESTFN2, "w", zipfile.ZIP_STORED)
         zipfp.write(TESTFN, "/absolute")
         zipfp.close()
@@ -275,7 +275,7 @@
         self.assertEqual(zipfp.namelist(), ["absolute"])
         zipfp.close()
 
-    def test_AppendToZipFile(self):
+    def test_append_to_zip_file(self):
         # Test appending to an existing zipfile
         zipfp = zipfile.ZipFile(TESTFN2, "w", zipfile.ZIP_STORED)
         zipfp.write(TESTFN, TESTFN)
@@ -285,7 +285,7 @@
         self.assertEqual(zipfp.namelist(), [TESTFN, "strfile"])
         zipfp.close()
 
-    def test_AppendToNonZipFile(self):
+    def test_append_to_non_zip_file(self):
         # Test appending to an existing file that is not a zipfile
         # NOTE: this test fails if len(d) < 22 because of the first
         # line "fpin.seek(-22, 2)" in _EndRecData
@@ -304,14 +304,14 @@
         zipfp.close()
         f.close()
 
-    def test_WriteDefaultName(self):
+    def test_write_default_name(self):
         # Check that calling ZipFile.write without arcname specified produces the expected result
         zipfp = zipfile.ZipFile(TESTFN2, "w")
         zipfp.write(TESTFN)
         self.assertEqual(zipfp.read(TESTFN), file(TESTFN).read())
         zipfp.close()
 
-    def test_PerFileCompression(self):
+    def test_per_file_compression(self):
         # Check that files within a Zip archive can have different compression options
         zipfp = zipfile.ZipFile(TESTFN2, "w")
         zipfp.write(TESTFN, 'storeme', zipfile.ZIP_STORED)
@@ -322,7 +322,7 @@
         self.assertEqual(dinfo.compress_type, zipfile.ZIP_DEFLATED)
         zipfp.close()
 
-    def test_WriteToReadonly(self):
+    def test_write_to_readonly(self):
         # Check that trying to call write() on a readonly ZipFile object
         # raises a RuntimeError
         zipf = zipfile.ZipFile(TESTFN2, mode="w")
@@ -332,7 +332,7 @@
         self.assertRaises(RuntimeError, zipf.write, TESTFN)
         zipf.close()
 
-    def test_Extract(self):
+    def test_extract(self):
         zipfp = zipfile.ZipFile(TESTFN2, "w", zipfile.ZIP_STORED)
         for fpath, fdata in SMALL_TEST_DATA:
             zipfp.writestr(fpath, fdata)
@@ -361,7 +361,7 @@
         # remove the test file subdirectories
         shutil.rmtree(os.path.join(os.getcwd(), 'ziptest2dir'))
 
-    def test_ExtractAll(self):
+    def test_extract_all(self):
         zipfp = zipfile.ZipFile(TESTFN2, "w", zipfile.ZIP_STORED)
         for fpath, fdata in SMALL_TEST_DATA:
             zipfp.writestr(fpath, fdata)
@@ -388,12 +388,12 @@
         # Make sure that writestr creates files with mode 0600,
         # when it is passed a name rather than a ZipInfo instance.
 
-        self.makeTestArchive(f, compression)
+        self.make_test_archive(f, compression)
         zipfp = zipfile.ZipFile(f, "r")
         zinfo = zipfp.getinfo('strfile')
         self.assertEqual(zinfo.external_attr, 0600 << 16)
 
-    def test_WritestrPermissions(self):
+    def test_writestr_permissions(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
             self.zip_test_writestr_permissions(f, zipfile.ZIP_STORED)
 
@@ -418,24 +418,24 @@
         fp.write(self.data)
         fp.close()
 
-    def largeFileExceptionTest(self, f, compression):
+    def large_file_exception_test(self, f, compression):
         zipfp = zipfile.ZipFile(f, "w", compression)
         self.assertRaises(zipfile.LargeZipFile,
                 zipfp.write, TESTFN, "another"+os.extsep+"name")
         zipfp.close()
 
-    def largeFileExceptionTest2(self, f, compression):
+    def large_file_exception_test2(self, f, compression):
         zipfp = zipfile.ZipFile(f, "w", compression)
         self.assertRaises(zipfile.LargeZipFile,
                 zipfp.writestr, "another"+os.extsep+"name", self.data)
         zipfp.close()
 
-    def test_LargeFileException(self):
+    def test_large_file_exception(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.largeFileExceptionTest(f, zipfile.ZIP_STORED)
-            self.largeFileExceptionTest2(f, zipfile.ZIP_STORED)
+            self.large_file_exception_test(f, zipfile.ZIP_STORED)
+            self.large_file_exception_test2(f, zipfile.ZIP_STORED)
 
-    def zipTest(self, f, compression):
+    def zip_test(self, f, compression):
         # Create the ZIP archive
         zipfp = zipfile.ZipFile(f, "w", compression, allowZip64=True)
         zipfp.write(TESTFN, "another"+os.extsep+"name")
@@ -500,16 +500,16 @@
 
         zipfp.close()
 
-    def test_Stored(self):
+    def test_stored(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipTest(f, zipfile.ZIP_STORED)
+            self.zip_test(f, zipfile.ZIP_STORED)
 
     @skipUnless(zlib, "requires zlib")
-    def test_Deflated(self):
+    def test_deflated(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipTest(f, zipfile.ZIP_DEFLATED)
+            self.zip_test(f, zipfile.ZIP_DEFLATED)
 
-    def test_AbsoluteArcnames(self):
+    def test_absolute_arcnames(self):
         zipfp = zipfile.ZipFile(TESTFN2, "w", zipfile.ZIP_STORED, allowZip64=True)
         zipfp.write(TESTFN, "/absolute")
         zipfp.close()
@@ -525,7 +525,7 @@
 
 
 class PyZipFileTests(unittest.TestCase):
-    def test_WritePyfile(self):
+    def test_write_pyfile(self):
         zipfp  = zipfile.PyZipFile(TemporaryFile(), "w")
         fn = __file__
         if fn.endswith('.pyc') or fn.endswith('.pyo'):
@@ -550,7 +550,7 @@
         self.assertTrue(bn + 'o' in zipfp.namelist() or bn + 'c' in zipfp.namelist())
         zipfp.close()
 
-    def test_WritePythonPackage(self):
+    def test_write_python_package(self):
         import email
         packagedir = os.path.dirname(email.__file__)
 
@@ -562,7 +562,7 @@
         self.assertTrue('email/__init__.pyo' in names or 'email/__init__.pyc' in names)
         self.assertTrue('email/mime/text.pyo' in names or 'email/mime/text.pyc' in names)
 
-    def test_WritePythonDirectory(self):
+    def test_write_python_directory(self):
         os.mkdir(TESTFN2)
         try:
             fp = open(os.path.join(TESTFN2, "mod1.py"), "w")
@@ -588,7 +588,7 @@
         finally:
             shutil.rmtree(TESTFN2)
 
-    def test_WriteNonPyfile(self):
+    def test_write_non_pyfile(self):
         zipfp  = zipfile.PyZipFile(TemporaryFile(), "w")
         file(TESTFN, 'w').write('most definitely not a python file')
         self.assertRaises(RuntimeError, zipfp.writepy, TESTFN)
@@ -596,7 +596,7 @@
 
 
 class OtherTests(unittest.TestCase):
-    def test_UnicodeFilenames(self):
+    def test_unicode_filenames(self):
         zf = zipfile.ZipFile(TESTFN, "w")
         zf.writestr(u"foo.txt", "Test for unicode filename")
         zf.writestr(u"\xf6.txt", "Test for unicode filename")
@@ -607,7 +607,7 @@
         self.assertEqual(zf.filelist[1].filename, u"\xf6.txt")
         zf.close()
 
-    def test_CreateNonExistentFileForAppend(self):
+    def test_create_non_existent_file_for_append(self):
         if os.path.exists(TESTFN):
             os.unlink(TESTFN)
 
@@ -627,7 +627,7 @@
         self.assertEqual(zf.read(filename), content)
         zf.close()
 
-    def test_CloseErroneousFile(self):
+    def test_close_erroneous_file(self):
         # This test checks that the ZipFile constructor closes the file object
         # it opens if there's an error in the file.  If it doesn't, the traceback
         # holds a reference to the ZipFile object and, indirectly, the file object.
@@ -642,7 +642,7 @@
         except zipfile.BadZipfile:
             pass
 
-    def test_IsZipErroneousFile(self):
+    def test_is_zip_erroneous_file(self):
         # This test checks that the is_zipfile function correctly identifies
         # a file that is not a zip file
 
@@ -664,7 +664,7 @@
         chk = zipfile.is_zipfile(fp)
         self.assertTrue(not chk)
 
-    def test_IsZipValidFile(self):
+    def test_is_zip_valid_file(self):
         # This test checks that the is_zipfile function correctly identifies
         # a file that is a zip file
 
@@ -689,7 +689,7 @@
         chk = zipfile.is_zipfile(fp)
         self.assertTrue(chk)
 
-    def test_NonExistentFileRaisesIOError(self):
+    def test_non_existent_file_raises_IOError(self):
         # make sure we don't raise an AttributeError when a partially-constructed
         # ZipFile instance is finalized; this tests for regression on SF tracker
         # bug #403871.
@@ -703,7 +703,7 @@
         # quickly.
         self.assertRaises(IOError, zipfile.ZipFile, TESTFN)
 
-    def test_ClosedZipRaisesRuntimeError(self):
+    def test_closed_zip_raises_RuntimeError(self):
         # Verify that testzip() doesn't swallow inappropriate exceptions.
         data = StringIO()
         zipf = zipfile.ZipFile(data, mode="w")
@@ -721,11 +721,11 @@
         file(TESTFN, 'w').write('zipfile test data')
         self.assertRaises(RuntimeError, zipf.write, TESTFN)
 
-    def test_BadConstructorMode(self):
+    def test_bad_constructor_mode(self):
         # Check that bad modes passed to ZipFile constructor are caught
         self.assertRaises(RuntimeError, zipfile.ZipFile, TESTFN, "q")
 
-    def test_BadOpenMode(self):
+    def test_bad_open_mode(self):
         # Check that bad modes passed to ZipFile.open are caught
         zipf = zipfile.ZipFile(TESTFN, mode="w")
         zipf.writestr("foo.txt", "O, for a Muse of Fire!")
@@ -736,7 +736,7 @@
         self.assertRaises(RuntimeError, zipf.open, "foo.txt", "q")
         zipf.close()
 
-    def test_Read0(self):
+    def test_read0(self):
         # Check that calling read(0) on a ZipExtFile object returns an empty
         # string and doesn't advance file pointer
         zipf = zipfile.ZipFile(TESTFN, mode="w")
@@ -749,30 +749,30 @@
         self.assertEqual(f.read(), "O, for a Muse of Fire!")
         zipf.close()
 
-    def test_OpenNonexistentItem(self):
+    def test_open_non_existent_item(self):
         # Check that attempting to call open() for an item that doesn't
         # exist in the archive raises a RuntimeError
         zipf = zipfile.ZipFile(TESTFN, mode="w")
         self.assertRaises(KeyError, zipf.open, "foo.txt", "r")
 
-    def test_BadCompressionMode(self):
+    def test_bad_compression_mode(self):
         # Check that bad compression methods passed to ZipFile.open are caught
         self.assertRaises(RuntimeError, zipfile.ZipFile, TESTFN, "w", -1)
 
-    def test_NullByteInFilename(self):
+    def test_null_byte_in_filename(self):
         # Check that a filename containing a null byte is properly terminated
         zipf = zipfile.ZipFile(TESTFN, mode="w")
         zipf.writestr("foo.txt\x00qqq", "O, for a Muse of Fire!")
         self.assertEqual(zipf.namelist(), ['foo.txt'])
 
-    def test_StructSizes(self):
+    def test_struct_sizes(self):
         # check that ZIP internal structure sizes are calculated correctly
         self.assertEqual(zipfile.sizeEndCentDir, 22)
         self.assertEqual(zipfile.sizeCentralDir, 46)
         self.assertEqual(zipfile.sizeEndCentDir64, 56)
         self.assertEqual(zipfile.sizeEndCentDir64Locator, 20)
 
-    def test_Comments(self):
+    def test_comments(self):
         # This test checks that comments on the archive are handled properly
 
         # check default comment is empty
@@ -860,19 +860,19 @@
         self.zip2.close()
         os.unlink(TESTFN2)
 
-    def test_NoPassword(self):
+    def test_no_password(self):
         # Reading the encrypted file without password
         # must generate a RunTime exception
         self.assertRaises(RuntimeError, self.zip.read, "test.txt")
         self.assertRaises(RuntimeError, self.zip2.read, "zero")
 
-    def test_BadPassword(self):
+    def test_bad_password(self):
         self.zip.setpassword("perl")
         self.assertRaises(RuntimeError, self.zip.read, "test.txt")
         self.zip2.setpassword("perl")
         self.assertRaises(RuntimeError, self.zip2.read, "zero")
 
-    def test_GoodPassword(self):
+    def test_good_password(self):
         self.zip.setpassword("python")
         self.assertEquals(self.zip.read("test.txt"), self.plain)
         self.zip2.setpassword("12345")
@@ -893,15 +893,15 @@
         unlink(TESTFN)
         unlink(TESTFN2)
 
-    def makeTestArchive(self, f, compression):
+    def make_test_archive(self, f, compression):
         # Create the ZIP archive
         zipfp = zipfile.ZipFile(f, "w", compression)
         zipfp.write(TESTFN, "another"+os.extsep+"name")
         zipfp.write(TESTFN, TESTFN)
         zipfp.close()
 
-    def zipTest(self, f, compression):
-        self.makeTestArchive(f, compression)
+    def zip_test(self, f, compression):
+        self.make_test_archive(f, compression)
 
         # Read the ZIP archive
         zipfp = zipfile.ZipFile(f, "r", compression)
@@ -911,12 +911,12 @@
         self.assertEqual(zipfp.read("another"+os.extsep+"name"), self.data)
         zipfp.close()
 
-    def test_Stored(self):
+    def test_stored(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipTest(f, zipfile.ZIP_STORED)
+            self.zip_test(f, zipfile.ZIP_STORED)
 
-    def zipOpenTest(self, f, compression):
-        self.makeTestArchive(f, compression)
+    def zip_open_test(self, f, compression):
+        self.make_test_archive(f, compression)
 
         # Read the ZIP archive
         zipfp = zipfile.ZipFile(f, "r", compression)
@@ -945,12 +945,12 @@
         self.assertEqual(testdata1, self.data)
         zipfp.close()
 
-    def test_OpenStored(self):
+    def test_open_stored(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipOpenTest(f, zipfile.ZIP_STORED)
+            self.zip_open_test(f, zipfile.ZIP_STORED)
 
-    def zipRandomOpenTest(self, f, compression):
-        self.makeTestArchive(f, compression)
+    def zip_random_open_test(self, f, compression):
+        self.make_test_archive(f, compression)
 
         # Read the ZIP archive
         zipfp = zipfile.ZipFile(f, "r", compression)
@@ -967,9 +967,9 @@
         self.assertEqual(testdata, self.data)
         zipfp.close()
 
-    def test_RandomOpenStored(self):
+    def test_random_open_stored(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.zipRandomOpenTest(f, zipfile.ZIP_STORED)
+            self.zip_random_open_test(f, zipfile.ZIP_STORED)
 
 
 class TestsWithMultipleOpens(unittest.TestCase):
@@ -980,7 +980,7 @@
         zipfp.writestr('twos', '2'*FIXEDTEST_SIZE)
         zipfp.close()
 
-    def test_SameFile(self):
+    def test_same_file(self):
         # Verify that (when the ZipFile is in control of creating file objects)
         # multiple open() calls can be made without interfering with each other.
         zipf = zipfile.ZipFile(TESTFN2, mode="r")
@@ -993,7 +993,7 @@
         self.assertEqual(data1, data2)
         zipf.close()
 
-    def test_DifferentFile(self):
+    def test_different_file(self):
         # Verify that (when the ZipFile is in control of creating file objects)
         # multiple open() calls can be made without interfering with each other.
         zipf = zipfile.ZipFile(TESTFN2, mode="r")
@@ -1007,7 +1007,7 @@
         self.assertEqual(data2, '2'*FIXEDTEST_SIZE)
         zipf.close()
 
-    def test_Interleaved(self):
+    def test_interleaved(self):
         # Verify that (when the ZipFile is in control of creating file objects)
         # multiple open() calls can be made without interfering with each other.
         zipf = zipfile.ZipFile(TESTFN2, mode="r")
@@ -1029,19 +1029,19 @@
     def setUp(self):
         os.mkdir(TESTFN2)
 
-    def test_ExtractDir(self):
+    def test_extract_dir(self):
         zipf = zipfile.ZipFile(findfile("zipdir.zip"))
         zipf.extractall(TESTFN2)
         self.assertTrue(os.path.isdir(os.path.join(TESTFN2, "a")))
         self.assertTrue(os.path.isdir(os.path.join(TESTFN2, "a", "b")))
         self.assertTrue(os.path.exists(os.path.join(TESTFN2, "a", "b", "c")))
 
-    def test_Bug6050(self):
+    def test_bug_6050(self):
         # Extraction should succeed if directories already exist
         os.mkdir(os.path.join(TESTFN2, "a"))
-        self.test_ExtractDir()
+        self.test_extract_dir()
 
-    def test_StoreDir(self):
+    def test_store_dir(self):
         os.mkdir(os.path.join(TESTFN2, "x"))
         zipf = zipfile.ZipFile(TESTFN, "w")
         zipf.write(os.path.join(TESTFN2, "x"), "x")
@@ -1063,15 +1063,15 @@
             self.arcfiles[s] = '%s-%d' % (TESTFN, n)
             open(self.arcfiles[s], "wb").write(self.arcdata[s])
 
-    def makeTestArchive(self, f, compression):
+    def make_test_archive(self, f, compression):
         # Create the ZIP archive
         zipfp = zipfile.ZipFile(f, "w", compression)
         for fn in self.arcfiles.values():
             zipfp.write(fn, fn)
         zipfp.close()
 
-    def readTest(self, f, compression):
-        self.makeTestArchive(f, compression)
+    def read_test(self, f, compression):
+        self.make_test_archive(f, compression)
 
         # Read the ZIP archive
         zipfp = zipfile.ZipFile(f, "r")
@@ -1081,8 +1081,8 @@
 
         zipfp.close()
 
-    def readlineTest(self, f, compression):
-        self.makeTestArchive(f, compression)
+    def readline_test(self, f, compression):
+        self.make_test_archive(f, compression)
 
         # Read the ZIP archive
         zipfp = zipfile.ZipFile(f, "r")
@@ -1094,8 +1094,8 @@
 
         zipfp.close()
 
-    def readlinesTest(self, f, compression):
-        self.makeTestArchive(f, compression)
+    def readlines_test(self, f, compression):
+        self.make_test_archive(f, compression)
 
         # Read the ZIP archive
         zipfp = zipfile.ZipFile(f, "r")
@@ -1106,8 +1106,8 @@
 
         zipfp.close()
 
-    def iterlinesTest(self, f, compression):
-        self.makeTestArchive(f, compression)
+    def iterlines_test(self, f, compression):
+        self.make_test_archive(f, compression)
 
         # Read the ZIP archive
         zipfp = zipfile.ZipFile(f, "r")
@@ -1117,41 +1117,41 @@
 
         zipfp.close()
 
-    def test_ReadStored(self):
+    def test_read_stored(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.readTest(f, zipfile.ZIP_STORED)
+            self.read_test(f, zipfile.ZIP_STORED)
 
-    def test_ReadlineStored(self):
+    def test_readline_stored(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.readlineTest(f, zipfile.ZIP_STORED)
+            self.readline_test(f, zipfile.ZIP_STORED)
 
-    def test_ReadlinesStored(self):
+    def test_readlines_stored(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.readlinesTest(f, zipfile.ZIP_STORED)
+            self.readlines_test(f, zipfile.ZIP_STORED)
 
-    def test_IterlinesStored(self):
+    def test_iterlines_stored(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.iterlinesTest(f, zipfile.ZIP_STORED)
+            self.iterlines_test(f, zipfile.ZIP_STORED)
 
     @skipUnless(zlib, "requires zlib")
-    def test_ReadDeflated(self):
+    def test_read_deflated(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.readTest(f, zipfile.ZIP_DEFLATED)
+            self.read_test(f, zipfile.ZIP_DEFLATED)
 
     @skipUnless(zlib, "requires zlib")
-    def test_ReadlineDeflated(self):
+    def test_readline_deflated(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.readlineTest(f, zipfile.ZIP_DEFLATED)
+            self.readline_test(f, zipfile.ZIP_DEFLATED)
 
     @skipUnless(zlib, "requires zlib")
-    def test_ReadlinesDeflated(self):
+    def test_readlines_deflated(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.readlinesTest(f, zipfile.ZIP_DEFLATED)
+            self.readlines_test(f, zipfile.ZIP_DEFLATED)
 
     @skipUnless(zlib, "requires zlib")
-    def test_IterlinesDeflated(self):
+    def test_iterlines_deflated(self):
         for f in (TESTFN2, TemporaryFile(), StringIO()):
-            self.iterlinesTest(f, zipfile.ZIP_DEFLATED)
+            self.iterlines_test(f, zipfile.ZIP_DEFLATED)
 
     def tearDown(self):
         for sep, fn in self.arcfiles.items():


More information about the Python-checkins mailing list