Mark tests correctly

ShaharNaveh · ShaharNaveh · commit 26cdbfe048df · 2025-07-25T15:59:14.000+02:00
diff --git a/Lib/test/test_pickle.py b/Lib/test/test_pickle.py
@@ -465,6 +465,8 @@ def persistent_load(pid):
                 return pid
         check(PersUnpickler)
 
+    # TODO: RUSTPYTHON
+    @unittest.expectedFailure
     def test_pickler_super(self):
         class PersPickler(self.pickler):
             def persistent_id(subself, obj):
@@ -494,6 +496,8 @@ def persistent_load(subself, pid):
             self.assertEqual(unpickler.load(), 'abc')
             self.assertEqual(called, ['abc'])
 
+    # TODO: RUSTPYTHON
+    @unittest.expectedFailure
     def test_pickler_instance_attribute(self):
         def persistent_id(obj):
             called.append(obj)
@@ -528,6 +532,8 @@ def persistent_load(pid):
             del unpickler.persistent_load
             self.assertEqual(unpickler.persistent_load, old_persistent_load)
 
+    # TODO: RUSTPYTHON
+    @unittest.expectedFailure
     def test_pickler_super_instance_attribute(self):
         class PersPickler(self.pickler):
             def persistent_id(subself, obj):
@@ -571,22 +577,6 @@ def _persistent_load(subself, pid):
             del unpickler.persistent_load
             self.assertEqual(unpickler.persistent_load, old_persistent_load)
 
-    # TODO: RUSTPYTHON
-    @unittest.expectedFailure
-    def test_pickler_instance_attribute(self): # TODO(RUSTPYTHON): Remove this test when it passes
-        return super().test_pickler_instance_attribute()
-
-    # TODO: RUSTPYTHON
-    @unittest.expectedFailure
-    def test_pickler_super(self): # TODO(RUSTPYTHON): Remove this test when it passes
-        return super().test_pickler_super()
-
-    # TODO: RUSTPYTHON
-    @unittest.expectedFailure
-    def test_pickler_super_instance_attribute(self): # TODO(RUSTPYTHON): Remove this test when it passes
-        return super().test_pickler_super_instance_attribute()
-
-
 class PyPicklerUnpicklerObjectTests(AbstractPicklerUnpicklerObjectTests, unittest.TestCase):
 
     pickler_class = pickle._Pickler
@@ -923,6 +913,8 @@ def test_reverse_name_mapping(self):
                 module, name = mapping(module, name)
                 self.assertEqual((module, name), (module3, name3))
 
+    # TODO: RUSTPYTHON
+    @unittest.expectedFailure
     def test_exceptions(self):
         self.assertEqual(mapping('exceptions', 'StandardError'),
                          ('builtins', 'Exception'))
@@ -972,12 +964,6 @@ def test_multiprocessing_exceptions(self):
                 self.assertEqual(mapping('multiprocessing', name),
                                  ('multiprocessing.context', name))
 
-    # TODO: RUSTPYTHON
-    @unittest.expectedFailure
-    def test_exceptions(self): # TODO(RUSTPYTHON): Remove this test when it passes
-        return super().test_exceptions()
-
-
 def load_tests(loader, tests, pattern):
     tests.addTest(doctest.DocTestSuite(pickle))
     return tests
