Move test for copy.replace(structseq) to test_structseq.py

XuehaiPan · XuehaiPan · commit 43324a4bcbcc · 2023-10-03T00:47:26.000+08:00
diff --git a/Lib/test/test_copy.py b/Lib/test/test_copy.py
@@ -4,8 +4,6 @@
 import copyreg
 import weakref
 import abc
-import os
-import time
 from operator import le, lt, ge, gt, eq, ne, attrgetter
 
 import unittest
@@ -947,28 +945,6 @@ def test_namedtuple(self):
         with self.assertRaisesRegex(ValueError, 'unexpected field name'):
             copy.replace(p, x=1, error=2)
 
-    def test_structseq(self):
-        t = time.gmtime(0)
-        self.assertEqual(copy.replace(t), (1970, 1, 1, 0, 0, 0, 3, 1, 0))
-        self.assertEqual(copy.replace(t, tm_year=2000),
-                         (2000, 1, 1, 0, 0, 0, 3, 1, 0))
-        self.assertEqual(copy.replace(t, tm_mon=2),
-                         (1970, 2, 1, 0, 0, 0, 3, 1, 0))
-        self.assertEqual(copy.replace(t, tm_year=2000, tm_mon=2),
-                         (2000, 2, 1, 0, 0, 0, 3, 1, 0))
-        with self.assertRaisesRegex(ValueError, 'unexpected field name'):
-            copy.replace(t, tm_year=2000, error=2)
-
-        assert os.stat_result.n_unnamed_fields > 0
-        r = os.stat_result(range(os.stat_result.n_sequence_fields), {'st_atime_ns': -1})
-        self.assertHasAttr(r, 'st_atime_ns')
-        self.assertEqual(r.st_atime_ns, -1)
-        with self.assertRaisesRegex(AttributeError, 'readonly attribute'):
-            r.st_atime_ns = -2
-        r2 = copy.replace(r, st_atime_ns=-3)
-        self.assertHasAttr(r2, 'st_atime_ns')
-        self.assertEqual(r2.st_atime_ns, -3)
-
     def test_dataclass(self):
         from dataclasses import dataclass
         @dataclass
diff --git a/Lib/test/test_structseq.py b/Lib/test/test_structseq.py
@@ -1,3 +1,4 @@
+import copy
 import os
 import time
 import unittest
@@ -133,6 +134,31 @@ def test_match_args_with_unnamed_fields(self):
         self.assertEqual(os.stat_result.n_unnamed_fields, 3)
         self.assertEqual(os.stat_result.__match_args__, expected_args)
 
+    def test_copy_replace_all_fields_visible(self):
+        t = time.gmtime(0)
+        assert t.n_unnamed_fields == 0 and t.n_sequence_fields == t.n_fields
+
+        self.assertEqual(copy.replace(t), (1970, 1, 1, 0, 0, 0, 3, 1, 0))
+        self.assertEqual(copy.replace(t, tm_year=2000),
+                         (2000, 1, 1, 0, 0, 0, 3, 1, 0))
+        self.assertEqual(copy.replace(t, tm_mon=2),
+                         (1970, 2, 1, 0, 0, 0, 3, 1, 0))
+        self.assertEqual(copy.replace(t, tm_year=2000, tm_mon=2),
+                         (2000, 2, 1, 0, 0, 0, 3, 1, 0))
+        with self.assertRaisesRegex(ValueError, 'unexpected field name'):
+            copy.replace(t, tm_year=2000, error=2)
+
+    def test_copy_replace_has_invisible_fields(self):
+        assert os.stat_result.n_sequence_fields + os.stat_result.n_unnamed_fields < os.stat_result.n_fields
+        r = os.stat_result(range(os.stat_result.n_sequence_fields), {'st_atime_ns': -1})
+        self.assertHasAttr(r, 'st_atime_ns')
+        self.assertEqual(r.st_atime_ns, -1)
+        with self.assertRaisesRegex(AttributeError, 'readonly attribute'):
+            r.st_atime_ns = -2
+        r2 = copy.replace(r, st_atime_ns=-3)
+        self.assertHasAttr(r2, 'st_atime_ns')
+        self.assertEqual(r2.st_atime_ns, -3)
+
 
 if __name__ == "__main__":
     unittest.main()
