I have a Python module that uses the argparse library. How do I write tests for that section of the code base?
11 Answers
You should refactor your code and move the parsing to a function:
def parse_args(args):
parser = argparse.ArgumentParser(...)
parser.add_argument...
# ...Create your parser as you like...
return parser.parse_args(args)
Then in your main
function you should just call it with:
parser = parse_args(sys.argv[1:])
(where the first element of sys.argv
that represents the script name is removed to not send it as an additional switch during CLI operation.)
In your tests, you can then call the parser function with whatever list of arguments you want to test it with:
def test_parser(self):
parser = parse_args(['-l', '-m'])
self.assertTrue(parser.long)
# ...and so on.
This way you'll never have to execute the code of your application just to test the parser.
If you need to change and/or add options to your parser later in your application, then create a factory method:
def create_parser():
parser = argparse.ArgumentParser(...)
parser.add_argument...
# ...Create your parser as you like...
return parser
You can later manipulate it if you want, and a test could look like:
class ParserTest(unittest.TestCase):
def setUp(self):
self.parser = create_parser()
def test_something(self):
parsed = self.parser.parse_args(['--something', 'test'])
self.assertEqual(parsed.something, 'test')
-
5Thanks for your answer. How do we test for errors when a certain argument is not passed? Commented Feb 5, 2015 at 2:49
-
4@PratikKhadloya If the argument is required and it's not passed, argparse will raise an exception. Commented Feb 5, 2015 at 11:26
-
3@PratikKhadloya Yes, the message is unfortunately not really helpful :( It's just
2
...argparse
is not very test friendly since it prints directly tosys.stderr
... Commented Feb 6, 2015 at 14:02 -
5I don't think it's necessary to call the
ArgumentParser.parse_args
-method calls with parameterargs=sys.argv[1:]
. It already calls theArgumentParser.parse_known_args
-method. With argumentargs==None
it will obtain them withargs = _sys.argv[1:]
where_sys
is an alias forsys
. (It might be an update since the answer was posted.) Commented Dec 14, 2017 at 1:20 -
6In response to @thomas-fauskanger above:
parse_args(args)
allows you to pass in the args from the test - which is the intent here. Of and by itselfparse_args()
will work without thesys.argv[1:]
being passed in frommain()
. This is super helpful by the way. Commented Mar 24, 2021 at 21:41
"argparse portion" is a bit vague so this answer focuses on one part: the parse_args
method. This is the method that interacts with your command line and gets all the passed values. Basically, you can mock what parse_args
returns so that it doesn't need to actually get values from the command line. The mock
package can be installed via pip for python versions 2.6-3.2. It's part of the standard library as unittest.mock
from version 3.3 onwards.
import argparse
try:
from unittest import mock # python 3.3+
except ImportError:
import mock # python 2.6-3.2
@mock.patch('argparse.ArgumentParser.parse_args',
return_value=argparse.Namespace(kwarg1=value, kwarg2=value))
def test_command(mock_args):
pass
You have to include all your command method's args in Namespace
even if they're not passed. Give those args a value of None
. (see the docs) This style is useful for quickly doing testing for cases where different values are passed for each method argument. If you opt to mock Namespace
itself for total argparse non-reliance in your tests, make sure it behaves similarly to the actual Namespace
class.
Below is an example using the first snippet from the argparse library.
# test_mock_argparse.py
import argparse
try:
from unittest import mock # python 3.3+
except ImportError:
import mock # python 2.6-3.2
def main():
parser = argparse.ArgumentParser(description='Process some integers.')
parser.add_argument('integers', metavar='N', type=int, nargs='+',
help='an integer for the accumulator')
parser.add_argument('--sum', dest='accumulate', action='store_const',
const=sum, default=max,
help='sum the integers (default: find the max)')
args = parser.parse_args()
print(args) # NOTE: this is how you would check what the kwargs are if you're unsure
return args.accumulate(args.integers)
@mock.patch('argparse.ArgumentParser.parse_args',
return_value=argparse.Namespace(accumulate=sum, integers=[1,2,3]))
def test_command(mock_args):
res = main()
assert res == 6, "1 + 2 + 3 = 6"
if __name__ == "__main__":
print(main())
-
1But now your unittest code also depends on
argparse
and itsNamespace
class. You should mockNamespace
.– imrekCommented May 13, 2017 at 17:03 -
1@DrunkenMaster apologies for the snarky tone. I updated my answer with explanation and possible uses. I'm learning here as well so if you would, can you (or someone else) provide cases where mocking the return value is beneficial? (or at least cases where not mocking the return value is detrimental)– munsuCommented May 15, 2017 at 3:11
-
1
from unittest import mock
is now the correct import method - well at least for python3 Commented Nov 9, 2018 at 17:11 -
1@MichaelHall thanks. I updated the snippet and added contextual info.– munsuCommented Nov 11, 2018 at 23:49
-
2The use of the
Namespace
class here is exactly what I was looking for. Despite the test still relying onargparse
, it does not rely on the particular implementation ofargparse
by the code under test, which is important for my unit tests. In addition, it's easy to usepytest
'sparametrize()
method to quickly test various argument combinations with a templated mock that includesreturn_value=argparse.Namespace(accumulate=accumulate, integers=integers)
.– acetoneCommented Aug 28, 2019 at 21:14
Make your main()
function take argv
as an argument rather than letting it read from sys.argv
as it will by default:
# mymodule.py
import argparse
import sys
def main(args):
parser = argparse.ArgumentParser()
parser.add_argument('-a')
process(**vars(parser.parse_args(args)))
return 0
def process(a=None):
pass
if __name__ == "__main__":
sys.exit(main(sys.argv[1:]))
Then you can test normally.
import mock
from mymodule import main
@mock.patch('mymodule.process')
def test_main(process):
main([])
process.assert_call_once_with(a=None)
@mock.patch('foo.process')
def test_main_a(process):
main(['-a', '1'])
process.assert_call_once_with(a='1')
I did not want to modify the original serving script so I just mocked out the sys.argv
part in argparse.
from unittest.mock import patch
with patch('argparse._sys.argv', ['python', 'serve.py']):
... # your test code here
This breaks if argparse implementation changes but enough for a quick test script. Sensibility is much more important than specificity in test scripts anyways.
-
1I love this solution, but when I execute my script,
'python'
isn't part ofsys.argv
. The first argument is only the script name. I think the mocked return value in your answer should be only['serve.py']
. Commented May 8, 2023 at 17:39
parse_args
throws a SystemExit
and prints to stderr, you can catch both of these:
import contextlib
import io
import sys
@contextlib.contextmanager
def captured_output():
new_out, new_err = io.StringIO(), io.StringIO()
old_out, old_err = sys.stdout, sys.stderr
try:
sys.stdout, sys.stderr = new_out, new_err
yield sys.stdout, sys.stderr
finally:
sys.stdout, sys.stderr = old_out, old_err
def validate_args(args):
with captured_output() as (out, err):
try:
parser.parse_args(args)
return True
except SystemExit as e:
return False
You inspect stderr (using err.seek(0); err.read()
but generally that granularity isn't required.
Now you can use assertTrue
or whichever testing you like:
assertTrue(validate_args(["-l", "-m"]))
Alternatively you might like to catch and rethrow a different error (instead of SystemExit
):
def validate_args(args):
with captured_output() as (out, err):
try:
return parser.parse_args(args)
except SystemExit as e:
err.seek(0)
raise argparse.ArgumentError(err.read())
-
1Can't believe this answer, which IMHO addresses the real problem with testing
argparse
, is buried so far down. Thanks for these details!– LiedmanCommented May 29, 2023 at 11:24
- Populate your arg list by using
sys.argv.append()
and then callparse()
, check the results and repeat. - Call from a batch/bash file with your flags and a dump args flag.
- Put all your argument parsing in a separate file and in the
if __name__ == "__main__":
call parse and dump/evaluate the results then test this from a batch/bash file.
A simple way of testing a parser is:
parser = ...
parser.add_argument('-a',type=int)
...
argv = '-a 1 foo'.split() # or ['-a','1','foo']
args = parser.parse_args(argv)
assert(args.a == 1)
...
Another way is to modify sys.argv
, and call args = parser.parse_args()
There are lots of examples of testing argparse
in lib/test/test_argparse.py
-
This should be the accepted answer: The simplest way to test different values given to each argument. Commented Jun 15, 2021 at 20:07
When passing results from argparse.ArgumentParser.parse_args
to a function, I sometimes use a namedtuple
to mock arguments for testing.
import unittest
from collections import namedtuple
from my_module import main
class TestMyModule(TestCase):
args_tuple = namedtuple('args', 'arg1 arg2 arg3 arg4')
def test_arg1(self):
args = TestMyModule.args_tuple("age > 85", None, None, None)
res = main(args)
assert res == ["55289-0524", "00591-3496"], 'arg1 failed'
def test_arg2(self):
args = TestMyModule.args_tuple(None, [42, 69], None, None)
res = main(args)
assert res == [], 'arg2 failed'
if __name__ == '__main__':
unittest.main()
For testing CLI (command line interface), and not command output I did something like this
import pytest
from argparse import ArgumentParser, _StoreAction
ap = ArgumentParser(prog="cli")
ap.add_argument("cmd", choices=("spam", "ham"))
ap.add_argument("-a", "--arg", type=str, nargs="?", default=None, const=None)
...
def test_parser():
assert isinstance(ap, ArgumentParser)
assert isinstance(ap, list)
args = {_.dest: _ for _ in ap._actions if isinstance(_, _StoreAction)}
assert args.keys() == {"cmd", "arg"}
assert args["cmd"] == ("spam", "ham")
assert args["arg"].type == str
assert args["arg"].nargs == "?"
...
Minmal complete example from my blog post (https://hughesadam87.medium.com/dead-simple-pytest-and-argparse-d1dbb6affbc3)
# coolapp.py
import argparse as ap
import sys
def _parse(args) -> ap.Namespace:
parser = ap.ArgumentParser()
parser.add_argument("myfile")
parsed = parser.parse_args(args)
start(parsed.myfile
def start(myfile) -> None:
print(f'my file: {myfile}')
if __name__ == "__main__":
_parse(sys.argv[1:])
Some tests
#test_coolapp.py
from coolapp import start, _parse
import sys
def test_coolapp():
""" Direct import of start """
start("myfile.txt")
def test_coolapp_sysargs():
""" Called through __main__ (eg. python coolapp.py myfile.txt) """
_parse(['myfile.txt'])
def test_coolapp_no_args(capsys):
""" ie. python coolapp.py """
with pytest.raises(SystemExit):
_parse([])
captured = capsys.readouterr()
assert "the following arguments are required: myfile" in captured.err
def test_coolapp_extra_args(capsys):
""" ie. python coolapp.py arg1 arg2 """
with pytest.raises(SystemExit):
_parse(['arg1', 'arg2'])
captured = capsys.readouterr()
assert "unrecognized arguments: arg2" in captured.err
In addition to many good answers...
In my case I should provide my parameters to function where parameters are parsed, e.g.:
# main.py
import argparse
def get_myparam():
parser = argparse.ArgumentParser()
parser.add_argument('--myparam', type=str, default='myvalue')
args = parser.parse_args()
return args.myparam
if __name__ == '__main__':
print(f'main: myparam={get_myparam()}')
Example of output:
$ python main.py
main: myparam=myvalue
$ python main.py --myparam newvalue
main: myparam=newvalue
pytest
test example:
# test_main.py
import argparse
import main
def test_mock_params(mocker):
mocker.patch('argparse.ArgumentParser.parse_args',
return_value=argparse.Namespace(myparam='mocked',))
assert main.get_myparam() == 'mocked'
For mocker
you need to install pytest-mock
:
$ pip install pytest-mock
foo()
is called". Mocking ofsys.argv
is the answer if that's the case. Take a look at the cli-test-helpers Python package. See also stackoverflow.com/a/58594599/202834