[IMP] tests logging
* Make phantomjs logging clearer by logging within the test case's module, not odoo.tests.common. This way, a tour in e.g. point of sale is clearly logged as that, rather than be logged as odoo.tests.common.phantomjs and require hunting which module/test we happen to be in. * Clarify tagging causes for exclusion of selection of tests.
This commit is contained in:
committed by
Christophe Simonis
parent
80fd5abd90
commit
35eaa523bd
@@ -338,6 +338,8 @@ class HttpCase(TransactionCase):
|
||||
self.xmlrpc_common = xmlrpclib.ServerProxy(url_8 + 'common')
|
||||
self.xmlrpc_db = xmlrpclib.ServerProxy(url_8 + 'db')
|
||||
self.xmlrpc_object = xmlrpclib.ServerProxy(url_8 + 'object')
|
||||
cls = type(self)
|
||||
self._logger = logging.getLogger('%s.%s' % (cls.__module__, cls.__name__))
|
||||
|
||||
def setUp(self):
|
||||
super(HttpCase, self).setUp()
|
||||
@@ -395,7 +397,7 @@ class HttpCase(TransactionCase):
|
||||
Other lines are relayed to the test log.
|
||||
|
||||
"""
|
||||
logger = _logger.getChild('phantomjs')
|
||||
logger = self._logger.getChild('phantomjs')
|
||||
t0 = datetime.now()
|
||||
td = timedelta(seconds=timeout)
|
||||
buf = bytearray()
|
||||
@@ -1332,10 +1334,11 @@ class TagsSelector(object):
|
||||
tags = getattr(arg, 'test_tags', set())
|
||||
inter_no_test = self.exclude.intersection(tags)
|
||||
if inter_no_test:
|
||||
_logger.debug("Test '%s' not selected because of following tag(s): '%s'", arg, inter_no_test)
|
||||
_logger.debug("Test '%s' not selected because it is tagged with : %s (exclusions: %s)", arg, inter_no_test, self.exclude)
|
||||
return False
|
||||
inter_to_test = self.include.intersection(tags)
|
||||
if not inter_to_test:
|
||||
_logger.debug("Test '%s' not selected because it was not tagged with '%s'", arg, self.include)
|
||||
_logger.debug("Test '%s' not selected because it was not tagged with %s", arg, self.include)
|
||||
return False
|
||||
_logger.debug("Test '%s' selected: tagged with %s, exclusions: %s, inclusions: %s", arg, tags, self.exclude, self.include)
|
||||
return True
|
||||
|
||||
Reference in New Issue
Block a user