inspect: Fix BoundArguments.apply_defaults to handle empty arguments

1st1 · 1st1 · commit f9e1f2bda930 · 2016-03-02T11:07:47.000-05:00
Patch by Frederick Wagner (issue #26347)
diff --git a/Lib/inspect.py b/Lib/inspect.py
@@ -2591,8 +2591,6 @@ def apply_defaults(self):
         empty dict.
         """
         arguments = self.arguments
-        if not arguments:
-            return
         new_arguments = []
         for name, param in self._signature.parameters.items():
             try:
diff --git a/Lib/test/test_inspect.py b/Lib/test/test_inspect.py
@@ -3324,6 +3324,13 @@ def foo(): pass
         ba.apply_defaults()
         self.assertEqual(list(ba.arguments.items()), [])
 
+        # Make sure a no-args binding still acquires proper defaults.
+        def foo(a='spam'): pass
+        sig = inspect.signature(foo)
+        ba = sig.bind()
+        ba.apply_defaults()
+        self.assertEqual(list(ba.arguments.items()), [('a', 'spam')])
+
 
 class TestSignaturePrivateHelpers(unittest.TestCase):
     def test_signature_get_bound_param(self):
