diff options
author | Zbigniew J??drzejewski-Szmek <zbyszek@in.waw.pl> | 2012-09-02 14:59:19 +0200 |
---|---|---|
committer | Zbigniew J??drzejewski-Szmek <zbyszek@in.waw.pl> | 2012-09-02 14:59:19 +0200 |
commit | 6de0331acf4cf34df98f892f9073d292dc7d5140 (patch) | |
tree | b9ea4ad21587804fcbe16d603d80089df6454c00 | |
parent | 26bd64f87c11cdc7acf503c88353ce87f3c65683 (diff) | |
download | cpython-6de0331acf4cf34df98f892f9073d292dc7d5140.tar.gz |
Fix bug with argparse.Parser.parse_args(*args)
-rw-r--r-- | Lib/argparse.py | 5 | ||||
-rw-r--r-- | Lib/test/test_argparse.py | 18 |
2 files changed, 22 insertions, 1 deletions
diff --git a/Lib/argparse.py b/Lib/argparse.py index cc3e374a66..67bbef2046 100644 --- a/Lib/argparse.py +++ b/Lib/argparse.py @@ -1709,9 +1709,12 @@ class ArgumentParser(_AttributeHolder, _ActionsContainer): return args def parse_known_args(self, args=None, namespace=None): - # args default to the system args if args is None: + # args default to the system args args = _sys.argv[1:] + else: + # make sure that args are mutable + args = list(args) # default Namespace built from parser defaults if namespace is None: diff --git a/Lib/test/test_argparse.py b/Lib/test/test_argparse.py index fe930a3e8c..2e6584f5d9 100644 --- a/Lib/test/test_argparse.py +++ b/Lib/test/test_argparse.py @@ -4565,6 +4565,24 @@ class TestMessageContentError(TestCase): class TestParseKnownArgs(TestCase): + def test_arguments_tuple(self): + parser = argparse.ArgumentParser() + parser.parse_args(()) + + def test_arguments_list(self): + parser = argparse.ArgumentParser() + parser.parse_args([]) + + def test_arguments_tuple_positional(self): + parser = argparse.ArgumentParser() + parser.add_argument('x') + parser.parse_args(('x',)) + + def test_arguments_list_positional(self): + parser = argparse.ArgumentParser() + parser.add_argument('x') + parser.parse_args(['x']) + def test_optionals(self): parser = argparse.ArgumentParser() parser.add_argument('--foo') |