openstack/rally
Code Issues Proposed changes

Merge "Make default do_use=True in start method"

Browse Source
This commit is contained in:
Jenkins 2015-09-22 09:42:33 +00:00 committed by Gerrit Code Review
commit 7637e5f646
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
rally/cli/commands/verify.py
View File

@ -59,7 +59,7 @@ class VerifyCommands(object):
required=False, action="store_true")
@envutils.with_default_deployment(cli_arg_name="deployment")
def start(self, set_name="", deployment=None, regex=None,
tempest_config=None, do_use=False,
tempest_config=None, do_use=True,
system_wide_install=False):
"""Start set of tests.
@ -67,6 +67,7 @@ class VerifyCommands(object):
:param deployment: UUID or name of a deployment
:param regex: Regular expression of test
:param tempest_config: User specified Tempest config file location
:param do_use: Use new task as default for future operations
:param system_wide_install: Use virtualenv else run tests in
local environment
"""

6
tests/unit/cli/commands/test_verify.py
View File

@ -55,7 +55,7 @@ class VerifyCommandsTestCase(test.TestCase):
mock_clients().nova().flavors.list.return_value = [
self.flavor1, self.flavor2]
self.verify.start(deployment=deployment_id)
self.verify.start(deployment=deployment_id, do_use=False)
default_set_name = "full"
default_regex = None
@ -73,7 +73,7 @@ class VerifyCommandsTestCase(test.TestCase):
self.flavor1, self.flavor2]
tempest_config = tempfile.NamedTemporaryFile()
self.verify.start(deployment=deployment_id,
tempest_config=tempest_config.name)
tempest_config=tempest_config.name, do_use=False)
default_set_name = "full"
default_regex = None
@ -88,7 +88,7 @@ class VerifyCommandsTestCase(test.TestCase):
wrong_set_name = "unexpected_value"
self.verify.start(deployment_id, wrong_set_name)
self.verify.start(deployment_id, wrong_set_name, do_use=False)
self.assertNotIn(wrong_set_name, consts.TempestTestsSets,
consts.TempestTestsAPI)