]> granicus.if.org Git - python/commitdiff
Issue #13862: Relax zlib version test to avoid spurious failures.
authorNadeem Vawda <nadeem.vawda@gmail.com>
Wed, 25 Jan 2012 21:16:50 +0000 (23:16 +0200)
committerNadeem Vawda <nadeem.vawda@gmail.com>
Wed, 25 Jan 2012 21:16:50 +0000 (23:16 +0200)
Lib/test/test_zlib.py
Misc/NEWS

index 51a9d4bda4b22321221fcd48ad463ae323448aaf..851fbf7cf71ff08c126b55f882f8a1d7c57d1858 100644 (file)
@@ -16,12 +16,12 @@ except ImportError:
 class VersionTestCase(unittest.TestCase):
 
     def test_library_version(self):
-        # On the build system, ZLIB_RUNTIME_VERSION should match ZLIB_VERSION.
-        # ZLIB_RUNTIME_VERSION is the actual library version while ZLIB_VERSION
-        # is the version from the header file. On the build system, the headers
-        # should match with the library exactly. At runtime, only the first
-        # digit is required to match.
-        self.assertEqual(zlib.ZLIB_RUNTIME_VERSION, zlib.ZLIB_VERSION)
+        # Test that the major version of the actual library in use matches the
+        # major version that we were compiled against. We can't guarantee that
+        # the minor versions will match (even on the machine on which the module
+        # was compiled), and the API is stable between minor versions, so
+        # testing only the major verions avoids spurious failures.
+        self.assertEqual(zlib.ZLIB_RUNTIME_VERSION[0], zlib.ZLIB_VERSION[0])
 
 
 class ChecksumTestCase(unittest.TestCase):
index f6b1f0db3a90c9b52d8c9f0dcb68ea3ee01144fd..2dab0a1c368ec3f2bc7dc8fe883c46db6489b1a9 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -461,6 +461,9 @@ Core and Builtins
 Library
 -------
 
+- Issue #13862: Fix spurious failure in test_zlib due to runtime/compile time
+  minor versions not matching.
+
 - Issue #12804: Fix test_socket and test_urllib2net failures when running tests
   on a system without internet access.