Fix test_assign_vm to be discovered by nose testrunner

the nosetests runner will not discover tests unless the tests start with
the test_ prefix. When wrapping by the log_test_exceptions method the
name of the method changes and is not test_ anymore causing the runner
to skip the test. Included the __doc__ string as well so the runner can
report results correctly.

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
This commit is contained in:
Prasanna Santhanam 2013-06-30 12:50:32 +05:30
parent e14f355a0b
commit 2a51c3e2c9

View File

@ -40,13 +40,14 @@ from marvin.integration.lib.common import (get_domain,
list_virtual_machines) list_virtual_machines)
def log_test_exceptions(func): def log_test_exceptions(func):
def _log_test_exceptions(self, *args, **kwargs): def test_wrap_exception_log(self, *args, **kwargs):
try: try:
func(self, *args, **kwargs) func(self, *args, **kwargs)
except Exception as e: except Exception as e:
self.debug('Test %s Failed due to Exception=%s' % (func, e)) self.debug('Test %s Failed due to Exception=%s' % (func, e))
raise e raise e
return _log_test_exceptions test_wrap_exception_log.__doc__ = func.__doc__
return test_wrap_exception_log
class Services: class Services:
"""Test service data for:Change the ownershop of """Test service data for:Change the ownershop of
@ -229,6 +230,7 @@ class TestVMOwnership(cloudstackTestCase):
except Exception as e: except Exception as e:
self.debug("Warning! Exception in tearDown: %s" % e) self.debug("Warning! Exception in tearDown: %s" % e)
@attr(tags = ["advanced"]) @attr(tags = ["advanced"])
@log_test_exceptions @log_test_exceptions
def test_01_move_across_different_domains(self): def test_01_move_across_different_domains(self):