https://github.com/python/cpython/commit/2a40dda89de418cd563c3fc2144f3831b6feece7
commit: 2a40dda89de418cd563c3fc2144f3831b6feece7
branch: 3.13
author: Miss Islington (bot) <[email protected]>
committer: serhiy-storchaka <[email protected]>
date: 2024-10-07T21:40:47Z
summary:

[3.13] gh-95468: Add more tests for "--" (double dash) in test_argparse 
(GH-124274) (GH-125068)

(cherry picked from commit baa3550bc3a119f41cc4eaed5373f9d695208e8e)

Co-authored-by: Serhiy Storchaka <[email protected]>

files:
M Lib/test/test_argparse.py

diff --git a/Lib/test/test_argparse.py b/Lib/test/test_argparse.py
index fe71e8a2a95154..39023a8789267f 100644
--- a/Lib/test/test_argparse.py
+++ b/Lib/test/test_argparse.py
@@ -5993,7 +5993,9 @@ def test_zero_or_more_optional(self):
         args = parser.parse_args([])
         self.assertEqual(NS(x=[]), args)
 
-    def test_double_dash(self):
+
+class TestDoubleDash(TestCase):
+    def test_single_argument_option(self):
         parser = argparse.ArgumentParser(exit_on_error=False)
         parser.add_argument('-f', '--foo')
         parser.add_argument('bar', nargs='*')
@@ -6017,6 +6019,7 @@ def test_double_dash(self):
         args = parser.parse_args(['a', '--', 'b', '--', 'c', '--foo', 'd'])
         self.assertEqual(NS(foo=None, bar=['a', 'b', '--', 'c', '--foo', 
'd']), args)
 
+    def test_multiple_argument_option(self):
         parser = argparse.ArgumentParser(exit_on_error=False)
         parser.add_argument('-f', '--foo', nargs='*')
         parser.add_argument('bar', nargs='*')
@@ -6039,6 +6042,7 @@ def test_double_dash(self):
         self.assertEqual(NS(foo=['c'], bar=['a', 'b']), args)
         self.assertEqual(argv, ['--', 'd'])
 
+    def test_multiple_double_dashes(self):
         parser = argparse.ArgumentParser(exit_on_error=False)
         parser.add_argument('foo')
         parser.add_argument('bar', nargs='*')
@@ -6054,9 +6058,10 @@ def test_double_dash(self):
         args = parser.parse_args(['--', '--', 'a', '--', 'b', 'c'])
         self.assertEqual(NS(foo='--', bar=['a', '--', 'b', 'c']), args)
 
+    def test_remainder(self):
         parser = argparse.ArgumentParser(exit_on_error=False)
         parser.add_argument('foo')
-        parser.add_argument('bar', nargs=argparse.REMAINDER)
+        parser.add_argument('bar', nargs='...')
 
         args = parser.parse_args(['--', 'a', 'b', 'c'])
         self.assertEqual(NS(foo='a', bar=['b', 'c']), args)
@@ -6067,6 +6072,40 @@ def test_double_dash(self):
         args = parser.parse_args(['a', '--', 'b', '--', 'c'])
         self.assertEqual(NS(foo='a', bar=['b', '--', 'c']), args)
 
+        parser = argparse.ArgumentParser(exit_on_error=False)
+        parser.add_argument('--foo')
+        parser.add_argument('bar', nargs='...')
+        args = parser.parse_args(['--foo', 'a', '--', 'b', '--', 'c'])
+        self.assertEqual(NS(foo='a', bar=['--', 'b', '--', 'c']), args)
+
+    def test_subparser(self):
+        parser = argparse.ArgumentParser(exit_on_error=False)
+        parser.add_argument('foo')
+        subparsers = parser.add_subparsers()
+        parser1 = subparsers.add_parser('run')
+        parser1.add_argument('-f')
+        parser1.add_argument('bar', nargs='*')
+
+        args = parser.parse_args(['x', 'run', 'a', 'b', '-f', 'c'])
+        self.assertEqual(NS(foo='x', f='c', bar=['a', 'b']), args)
+        args = parser.parse_args(['x', 'run', 'a', 'b', '--', '-f', 'c'])
+        self.assertEqual(NS(foo='x', f=None, bar=['a', 'b', '-f', 'c']), args)
+        args = parser.parse_args(['x', 'run', 'a', '--', 'b', '-f', 'c'])
+        self.assertEqual(NS(foo='x', f=None, bar=['a', 'b', '-f', 'c']), args)
+        args = parser.parse_args(['x', 'run', '--', 'a', 'b', '-f', 'c'])
+        self.assertEqual(NS(foo='x', f=None, bar=['a', 'b', '-f', 'c']), args)
+        args = parser.parse_args(['x', '--', 'run', 'a', 'b', '-f', 'c'])
+        self.assertEqual(NS(foo='x', f='c', bar=['a', 'b']), args)
+        args = parser.parse_args(['--', 'x', 'run', 'a', 'b', '-f', 'c'])
+        self.assertEqual(NS(foo='x', f='c', bar=['a', 'b']), args)
+        args = parser.parse_args(['x', 'run', '--', 'a', '--', 'b'])
+        self.assertEqual(NS(foo='x', f=None, bar=['a', '--', 'b']), args)
+        args = parser.parse_args(['x', '--', 'run', '--', 'a', '--', 'b'])
+        self.assertEqual(NS(foo='x', f=None, bar=['a', '--', 'b']), args)
+        self.assertRaisesRegex(argparse.ArgumentError,
+            "invalid choice: '--'",
+            parser.parse_args, ['--', 'x', '--', 'run', 'a', 'b'])
+
 
 # ===========================
 # parse_intermixed_args tests

_______________________________________________
Python-checkins mailing list -- [email protected]
To unsubscribe send an email to [email protected]
https://mail.python.org/mailman3/lists/python-checkins.python.org/
Member address: [email protected]

Reply via email to