diff options
author | Steven Bethard <steven.bethard@gmail.com> | 2011-04-04 02:10:40 +0200 |
---|---|---|
committer | Steven Bethard <steven.bethard@gmail.com> | 2011-04-04 02:10:40 +0200 |
commit | 7c8ea37d9bc66db3e6ea0b6a90c3f8b3deafaac1 (patch) | |
tree | 2527a6043a85daff4ec4cec4e41c65547c00b73a | |
parent | fc2e3763c81ab6a2b25308046bc42bab12afeb69 (diff) | |
parent | 7cb20a8605d79faae64cccf8bbe8418b492f0e30 (diff) | |
download | cpython-git-7c8ea37d9bc66db3e6ea0b6a90c3f8b3deafaac1.tar.gz |
Issue #9347: Fix formatting for tuples in argparse type= error messages.
-rw-r--r-- | Lib/argparse.py | 4 | ||||
-rw-r--r-- | Lib/test/test_argparse.py | 2 | ||||
-rw-r--r-- | Misc/NEWS | 2 |
3 files changed, 6 insertions, 2 deletions
diff --git a/Lib/argparse.py b/Lib/argparse.py index 79f1fe5a6b..0658472acf 100644 --- a/Lib/argparse.py +++ b/Lib/argparse.py @@ -1312,13 +1312,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"): diff --git a/Lib/test/test_argparse.py b/Lib/test/test_argparse.py index 6ca1019c9a..2836e7ec94 100644 --- a/Lib/test/test_argparse.py +++ b/Lib/test/test_argparse.py @@ -4082,10 +4082,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") @@ -349,6 +349,8 @@ Library - Issue #9026: Fix order of argparse sub-commands in help messages. +- Issue #9347: Fix formatting for tuples in argparse type= error messages. + Build ----- |