From 2dc7dd1257cdd7105948282bbceb6f23119ba468 Mon Sep 17 00:00:00 2001 From: Thibault Saunier Date: Sat, 28 Apr 2018 10:15:17 +0200 Subject: [PATCH] validate: launcher: Make sure testsuites are used/configured once only --- validate/launcher/baseclasses.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/validate/launcher/baseclasses.py b/validate/launcher/baseclasses.py index c7cf6edccc..7068716313 100644 --- a/validate/launcher/baseclasses.py +++ b/validate/launcher/baseclasses.py @@ -1424,7 +1424,7 @@ class _TestsLauncher(Loggable): return (None, exceptions) def _load_testsuites(self): - testsuites = [] + testsuites = set() for testsuite in self.options.testsuites: if os.path.exists(testsuite): testsuite = os.path.abspath(os.path.expanduser(testsuite)) @@ -1446,13 +1446,13 @@ class _TestsLauncher(Loggable): testsuite, loaded_module[1]), Colors.FAIL) continue - testsuites.append(module) + testsuites.add(module) if not hasattr(module, "TEST_MANAGER"): module.TEST_MANAGER = [tester.name for tester in self.testers] elif not isinstance(module.TEST_MANAGER, list): module.TEST_MANAGER = [module.TEST_MANAGER] - self.options.testsuites = testsuites + self.options.testsuites = list(testsuites) def _setup_testsuites(self): for testsuite in self.options.testsuites: