Merge "Make plugin deallocation check optional"

This commit is contained in:
Jenkins 2014-05-10 07:15:52 +00:00 committed by Gerrit Code Review
commit 221f323ef3

View File

@ -69,8 +69,17 @@ class BaseTestCase(testtools.TestCase):
#TODO(marun) Fix plugins that do not properly initialize notifiers
agentschedulers_db.AgentSchedulerDbMixin.agent_notifiers = {}
# Perform a check for deallocation only if explicitly
# configured to do so since calling gc.collect() after every
# test increases test suite execution time by ~50%.
check_plugin_deallocation = (
os.environ.get('OS_CHECK_PLUGIN_DEALLOCATION') in TRUE_STRING)
if check_plugin_deallocation:
plugin = weakref.ref(nm._instance.plugin)
nm.clear_instance()
if check_plugin_deallocation:
gc.collect()
#TODO(marun) Ensure that mocks are deallocated?