Fix test_identical_args to ignore type annotations in signature comparison (#1421) (#1423)

requests 2.33.0 added type annotations to its public functions, causing
inspect.signature() comparisons to fail since nailgun's wrappers don't
replicate those annotations. Strip annotations from both sides before
comparing so the test validates parameter names/kinds/defaults only.


(cherry picked from commit cbfb67f)

Co-authored-by: Vladimir Sedmik <46570670+vsedmik@users.noreply.github.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: 3 people

tests/test_client.py

@@ -130,8 +130,15 @@ def test_identical_args(self):
         signature as ``requests.delete``.
 
         """
+
+        def _strip_annotations(sig):
+            params = [
+                p.replace(annotation=inspect.Parameter.empty) for p in sig.parameters.values()
+            ]
+            return sig.replace(parameters=params, return_annotation=inspect.Parameter.empty)
+
         for meth in ('delete', 'get', 'head', 'patch', 'post', 'put'):
             self.assertEqual(
-                inspect.signature(getattr(client, meth)),
-                inspect.signature(getattr(requests, meth)),
+                _strip_annotations(inspect.signature(getattr(client, meth))),
+                _strip_annotations(inspect.signature(getattr(requests, meth))),
             )