Better test for Issue #15402: Add a __sizeof__ method to struct.Struct

diff --git a/Doc/ACKS.txt b/Doc/ACKS.txt
index 7507c6c6b76c448a297a0d5af947372d39c68d23..d32fcd927d0704352343b3d4c172ad528c25356a 100644 (file)
--- a/Doc/ACKS.txt
+++ b/Doc/ACKS.txt
@@ -197,6 +197,7 @@ docs@python.org), and we'll be glad to correct the problem.
    * Anthony Starks
    * Greg Stein
    * Peter Stoehr
+   * Serhiy Storchaka
    * Mark Summerfield
    * Reuben Sumner
    * Kalle Svensson

diff --git a/Lib/test/test_struct.py b/Lib/test/test_struct.py
index caada9cadba7c257fb1f73ce04cbd7472c34a680..f810ffe755f4d1eb193181c828ccbfa8c9eaf7d3 100644 (file)
--- a/Lib/test/test_struct.py
+++ b/Lib/test/test_struct.py
@@ -547,12 +547,12 @@ class StructTest(unittest.TestCase):
     def test_sizeof(self):
         self.assertGreater(sys.getsizeof(struct.Struct('BHILfdspP')),
                            sys.getsizeof(struct.Struct('B')))
-        self.assertGreaterEqual(sys.getsizeof(struct.Struct('123B')),
+        self.assertGreater(sys.getsizeof(struct.Struct('123B')),
                                 sys.getsizeof(struct.Struct('B')))
-        self.assertGreaterEqual(sys.getsizeof(struct.Struct('B' * 123)),
+        self.assertGreater(sys.getsizeof(struct.Struct('B' * 1234)),
+                                sys.getsizeof(struct.Struct('123B')))
+        self.assertGreater(sys.getsizeof(struct.Struct('1234B')),
                                 sys.getsizeof(struct.Struct('123B')))
-        self.assertGreaterEqual(sys.getsizeof(struct.Struct('123xB')),
-                                sys.getsizeof(struct.Struct('B')))
 
 def test_main():
     run_unittest(StructTest)