[Piglit] [patch v2 2/2] tests/all.py: Add assertions for bad characters in test names
Dylan Baker
baker.dylan.c at gmail.com
Thu Aug 28 23:18:50 PDT 2014
'/' is not allowed in test names since it is used as the separator for
groups.
Signed-off-by: Dylan Baker <dylanx.c.baker at intel.com>
---
tests/all.py | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/all.py b/tests/all.py
index 05a2c8e..2ff1a0e 100644
--- a/tests/all.py
+++ b/tests/all.py
@@ -37,6 +37,7 @@ def add_single_param_test_set(group, name, *params):
group[name + '-' + param] = plain_test(name + ' ' + param)
def add_plain_test(group, args):
+ assert '/' not in args, args
group[args] = plain_test(args)
def concurrent_test(args):
@@ -45,6 +46,7 @@ def concurrent_test(args):
return test
def add_concurrent_test(group, args):
+ assert '/' not in args, args
group[args] = concurrent_test(args)
# Generate all possible subsets of the given set, including the empty set.
--
2.1.0
More information about the Piglit
mailing list