diff options
author | Sai Sindhur Malleni <smalleni@redhat.com> | 2016-08-09 10:26:45 -0400 |
---|---|---|
committer | Sai Sindhur Malleni <smalleni@redhat.com> | 2016-08-10 12:53:22 -0400 |
commit | d826cafd69c9679a29a88a97699756cd408e4ac7 (patch) | |
tree | 185b1a0ddc607a3a45b07354aab5950ea7d0ce1c /func | |
parent | 42d97ed0627db286c1b1de445ba45a4dc3d9079e (diff) |
Fix Argument Parsing
Not making the option -l and -f causes a traceback when checking if -l is a
valid directory and -f is a valid file as concatenation is attemtped between
string and None type.
Signed-off-by: Sai Sindhur Malleni <smalleni@redhat.com>
Change-Id: If0ae87cde0e43122b3318f9c3749c05639ac3f36
Diffstat (limited to 'func')
-rw-r--r-- | func/cli.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/func/cli.py b/func/cli.py index 5e8f02cf..4613b507 100644 --- a/func/cli.py +++ b/func/cli.py @@ -54,13 +54,15 @@ class cli: @staticmethod def _parse_args(args): parser = argparse.ArgumentParser() - parser.add_argument('-l ', '--lab', help='Name of Lab on which being tested, These can' + parser.add_argument('-l ', '--lab', required=True, help='Name of Lab ' + 'on which being tested, These can' 'be found in the test_cases/ directory. Please ' 'ensure that you have edited the respective files ' 'before using them. For testing other than through Jenkins' ' The user should list default after -l . all the fields in' ' the files are necessary and should be filled') - parser.add_argument('-f', '--file', help='File in test_list with the list of tests. there are three files' + parser.add_argument('-f', '--file', required=True, help='File in ' + 'test_list with the list of tests. there are three files' '\n compute ' '\n storage ' '\n network ' |