summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZbigniew J??drzejewski-Szmek <zbyszek@in.waw.pl>2012-09-02 14:59:19 +0200
committerZbigniew J??drzejewski-Szmek <zbyszek@in.waw.pl>2012-09-02 14:59:19 +0200
commit6de0331acf4cf34df98f892f9073d292dc7d5140 (patch)
treeb9ea4ad21587804fcbe16d603d80089df6454c00
parent26bd64f87c11cdc7acf503c88353ce87f3c65683 (diff)
downloadcpython-6de0331acf4cf34df98f892f9073d292dc7d5140.tar.gz
Fix bug with argparse.Parser.parse_args(*args)
-rw-r--r--Lib/argparse.py5
-rw-r--r--Lib/test/test_argparse.py18
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')