Issue #9347: Fix formatting for tuples in argparse type= error messages.
This commit is contained in:
parent
43bf045be0
commit
7cb20a8605
|
@ -1287,13 +1287,13 @@ class _ActionsContainer(object):
|
|||
# create the action object, and add it to the parser
|
||||
action_class = self._pop_action_class(kwargs)
|
||||
if not _callable(action_class):
|
||||
raise ValueError('unknown action "%s"' % action_class)
|
||||
raise ValueError('unknown action "%s"' % (action_class,))
|
||||
action = action_class(**kwargs)
|
||||
|
||||
# raise an error if the action type is not callable
|
||||
type_func = self._registry_get('type', action.type, action.type)
|
||||
if not _callable(type_func):
|
||||
raise ValueError('%r is not callable' % type_func)
|
||||
raise ValueError('%r is not callable' % (type_func,))
|
||||
|
||||
# raise an error if the metavar does not match the type
|
||||
if hasattr(self, "_get_formatter"):
|
||||
|
|
|
@ -4051,10 +4051,12 @@ class TestInvalidArgumentConstructors(TestCase):
|
|||
|
||||
def test_invalid_type(self):
|
||||
self.assertValueError('--foo', type='int')
|
||||
self.assertValueError('--foo', type=(int, float))
|
||||
|
||||
def test_invalid_action(self):
|
||||
self.assertValueError('-x', action='foo')
|
||||
self.assertValueError('foo', action='baz')
|
||||
self.assertValueError('--foo', action=('store', 'append'))
|
||||
parser = argparse.ArgumentParser()
|
||||
try:
|
||||
parser.add_argument("--foo", action="store-true")
|
||||
|
|
Loading…
Reference in New Issue