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

diff --git a/Doc/ACKS.txt b/Doc/ACKS.txt
index 03e1a6badc008804660a33c7aa0756dd057b3d28..1ff849812be698be12f934b029c5d3c0b01ad201 100644 (file)
--- a/Doc/ACKS.txt
+++ b/Doc/ACKS.txt
@@ -204,6 +204,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 bd899502e953fa724209888f2016837629d7081b..be32d8843e9e414db9cf64b9a65a3442f9cc90c5 100644 (file)
--- a/Lib/test/test_struct.py
+++ b/Lib/test/test_struct.py
@@ -559,12 +559,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)