This will make it easier to distinguish tests written for Vulkan. It makes a bit of an inconsistency because the GL and CL tests don’t have any prefix. Ideally maybe we would add a prefix for those too, but changing the test names at this late stage would probably cause a lot of hassle.
Adding the prefix will avoid problems if we eventually decide to merge the Vulkan profile into the “quick” profile because otherwise we risk having name collisions when the same thing is tested on both Vulkan and GL. --- tests/vulkan.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/vulkan.py b/tests/vulkan.py index eed199c55..ae01677a9 100644 --- a/tests/vulkan.py +++ b/tests/vulkan.py @@ -21,7 +21,8 @@ profile = TestProfile() for basedir in [TESTS_DIR, GENERATED_TESTS_DIR]: _basedir = os.path.join(basedir, 'vulkan') for dirpath, _, filenames in os.walk(_basedir): - groupname = grouptools.from_path(os.path.relpath(dirpath, _basedir)) + groupname = ('vulkan' + grouptools.SEPARATOR + + grouptools.from_path(os.path.relpath(dirpath, _basedir))) dirname = os.path.relpath(dirpath, os.path.join(basedir, '..')) for filename in filenames: testname, ext = os.path.splitext(filename) -- 2.17.1 _______________________________________________ Piglit mailing list Piglit@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/piglit