[Python-Dev] [Python-checkins] cpython (2.7): Issue #11277: Remove useless test from test_zlib. (original) (raw)

Jim Jewett jimjjewett at gmail.com
Mon May 9 14:53:52 CEST 2011

Can you clarify (preferably in the commit message as well) exactly why these largefile tests are useless? For example, is there another test that covers this already?

-jJ

On 5/7/11, nadeem.vawda <python-checkins at python.org> wrote:

http://hg.python.org/cpython/rev/201dcfc56e86 changeset: 69886:201dcfc56e86 branch: 2.7 parent: 69881:a0147a1f1776 user: Nadeem Vawda <nadeem.vawda at gmail.com> date: Sat May 07 11:28:03 2011 +0200 summary: Issue #11277: Remove useless test from testzlib.

files: Lib/test/testzlib.py | 42 ------------------------------- 1 files changed, 0 insertions(+), 42 deletions(-)

diff --git a/Lib/test/testzlib.py b/Lib/test/testzlib.py --- a/Lib/test/testzlib.py +++ b/Lib/test/testzlib.py @@ -72,47 +72,6 @@ zlib.crc32('spam', (2**31))) -# Issue #11277 - check that inputs of 2 GB (or 1 GB on 32 bits system) are -# handled correctly. Be aware of issues #1202. We cannot test a buffer of 4 GB -# or more (#8650, #8651 and #10276), because the zlib stores the buffer size -# into an int. -class ChecksumBigBufferTestCase(unittest.TestCase): - if sys.maxsize > 4G: - # (64 bits system) crc32() and adler32() stores the buffer size into an - # int, the maximum filesize is INTMAX (0x7FFFFFFF) - filesize = 0x7FFFFFFF - else: - # (32 bits system) On a 32 bits OS, a process cannot usually address - # more than 2 GB, so test only 1 GB - filesize = 1G - - @unittest.skipUnless(mmap, "mmap() is not available.") - def testbigbuffer(self): - if sys.platform[:3] == 'win' or sys.platform == 'darwin': - requires('largefile', - 'test requires %s bytes and a long time to run' % - str(self.filesize)) - try: - with open(TESTFN, "wb+") as f: - f.seek(self.filesize-4) - f.write("asdf") - f.flush() - m = mmap.mmap(f.fileno(), 0, access=mmap.ACCESSREAD) - try: - if sys.maxsize > 4G: - self.assertEqual(zlib.crc32(m), 0x709418e7) - self.assertEqual(zlib.adler32(m), -2072837729) - else: - self.assertEqual(zlib.crc32(m), 722071057) - self.assertEqual(zlib.adler32(m), -1002962529) - finally: - m.close() - except (IOError, OverflowError): - raise unittest.SkipTest("filesystem doesn't have largefile support") - finally: - unlink(TESTFN) - - class ExceptionTestCase(unittest.TestCase): # make sure we generate some expected errors def testbadlevel(self): @@ -595,7 +554,6 @@ def testmain(): rununittest( ChecksumTestCase, - ChecksumBigBufferTestCase, ExceptionTestCase, CompressTestCase, CompressObjectTestCase -- Repository URL: http://hg.python.org/cpython

More information about the Python-Dev mailing list