diff options
author | Jenkins <jenkins@review.openstack.org> | 2013-01-15 20:41:40 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2013-01-15 20:41:40 +0000 |
commit | 3eb1c8736186012f62b008b8e823942f345f70aa (patch) | |
tree | 97f7ee1798fd828c53c6ee0b80e707ca5cb60980 | |
parent | f1bf204b0bffbb407bf38c8545929b49a3055958 (diff) | |
parent | 580c259b39fa5e639d3fbe00dc24da09c11d282a (diff) | |
download | oslo-3eb1c8736186012f62b008b8e823942f345f70aa.tar.gz oslo-3eb1c8736186012f62b008b8e823942f345f70aa.tar.xz oslo-3eb1c8736186012f62b008b8e823942f345f70aa.zip |
Merge "Make tox run doctests"
-rw-r--r-- | openstack/common/cfg.py | 14 | ||||
-rw-r--r-- | openstack/common/cliutils.py | 4 | ||||
-rw-r--r-- | tox.ini | 2 |
3 files changed, 11 insertions, 9 deletions
diff --git a/openstack/common/cfg.py b/openstack/common/cfg.py index a6500eb..8c965c1 100644 --- a/openstack/common/cfg.py +++ b/openstack/common/cfg.py @@ -236,10 +236,11 @@ in order to support a common usage pattern in OpenStack:: Positional command line arguments are supported via a 'positional' Opt constructor argument:: - >>> CONF.register_cli_opt(MultiStrOpt('bar', positional=True)) + >>> conf = ConfigOpts() + >>> conf.register_cli_opt(MultiStrOpt('bar', positional=True)) True - >>> CONF(['a', 'b']) - >>> CONF.bar + >>> conf(['a', 'b']) + >>> conf.bar ['a', 'b'] It is also possible to use argparse "sub-parsers" to parse additional @@ -249,10 +250,11 @@ command line arguments using the SubCommandOpt class: ... list_action = subparsers.add_parser('list') ... list_action.add_argument('id') ... - >>> CONF.register_cli_opt(SubCommandOpt('action', handler=add_parsers)) + >>> conf = ConfigOpts() + >>> conf.register_cli_opt(SubCommandOpt('action', handler=add_parsers)) True - >>> CONF(['list', '10']) - >>> CONF.action.name, CONF.action.id + >>> conf(args=['list', '10']) + >>> conf.action.name, conf.action.id ('list', '10') """ diff --git a/openstack/common/cliutils.py b/openstack/common/cliutils.py index 89a6f62..411bd58 100644 --- a/openstack/common/cliutils.py +++ b/openstack/common/cliutils.py @@ -36,11 +36,11 @@ def validate_args(fn, *args, **kwargs): >>> validate_args(lambda a: None) Traceback (most recent call last): ... - MissingArgs: An argument is missing: a + MissingArgs: An argument is missing >>> validate_args(lambda a, b, c, d: None, 0, c=1) Traceback (most recent call last): ... - MissingArgs: 2 arguments are missing: b, d + MissingArgs: 2 arguments are missing :param fn: the function to check :param arg: the positional arguments supplied @@ -11,7 +11,7 @@ setenv = VIRTUAL_ENV={envdir} NOSE_OPENSTACK_STDOUT=1 deps = -r{toxinidir}/tools/pip-requires -r{toxinidir}/tools/test-requires -commands = nosetests {posargs} +commands = nosetests --with-doctest --exclude-dir=tests/testmods {posargs} [testenv:pep8] deps = pep8==1.3.3 |