[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