Remove comment between xfail and test function. (#952)

In python 3.13, when a comment is placed between `xfail` and a type checked test function, it causes `inspect.get_source` to return the decorator, comment, function signature, but no body.

Author: dnwpark

gel/_internal/_testbase/_models.py

@@ -89,21 +89,6 @@
 TNAME_PY = "tname__"
 
 
-MYPY_INI = """\
-[mypy]
-strict = True
-ignore_errors = False
-follow_imports = normal
-show_error_codes = True
-local_partial_types = True
-
-# This is very intentional as it allows us to type check things
-# that must be flagged as an error by the type checker.
-# Don't "type: ignore" unless it's part of the test.
-warn_unused_ignores = True
-"""
-
-
 def _get_test_class_dir(cls: type[unittest.TestCase]) -> pathlib.Path:
     test_file = sys.modules[cls.__module__].__file__
     if test_file is None:

tests/test_model_generator.py

@@ -3982,8 +3982,9 @@ def test_modelgen_save_reload_links_01(self):
         assert fresh.user is not None
         self.assertEqual(fresh.user.id, alice.id)
 
-    @tb.xfail
-    # See XXX comments
+    @tb.xfail('''
+        See XXX comments
+    ''')
     def test_modelgen_save_reload_links_02(self):
         from models.orm import default
 
@@ -5190,16 +5191,17 @@ def test_modelgen_linkprops_05(self):
         self.assertEqual(gs0_link.__linkprops__.is_tall_enough, True)
         self.assertEqual(gs1_link.__linkprops__.is_tall_enough, False)
 
-    @tb.xfail
-    # victor: merge doesn't happen
-    # yury: this is probably OK -- we decided that the Gel server should
-    #       "merge things". When a ProxyModel is assigned to a single link
-    #       or added to a link set, it should just replace the existing one
-    #       there.  Whatever __linkprops__ are coming in after the assignment
-    #       will be sent to Gel in save() / sync() -- they both are only saving
-    #       the __gel_changed_fields__.
-    #       I'm keeping the test here, but I think we'll remove it later
-    #       once we discuss this one more time.
+    @tb.xfail('''
+        victor: merge doesn't happen
+        yury: this is probably OK -- we decided that the Gel server should
+              "merge things". When a ProxyModel is assigned to a single link
+              or added to a link set, it should just replace the existing one
+              there.  Whatever __linkprops__ are coming in after the assignment
+              will be sent to Gel in save() / sync() -- they both are only
+              saving the __gel_changed_fields__.
+              I'm keeping the test here, but I think we'll remove it later
+              once we discuss this one more time.
+    ''')
     def test_modelgen_linkprops_06(self):
         from models.orm import default
 
@@ -5244,9 +5246,10 @@ def test_modelgen_linkprops_07(self):
         self.assertEqual(member.__linkprops__.rank, None)
         self.assertEqual(member.__linkprops__.role, "sorceress")
 
-    @tb.xfail
-    # victor: merge doesn't happen
-    # yury: read the comment in test_modelgen_linkprops_06
+    @tb.xfail('''
+        victor: merge doesn't happen
+        yury: read the comment in test_modelgen_linkprops_06
+    ''')
     def test_modelgen_linkprops_08(self):
         from models.orm import default