From 34f86489a837fc40700bd4de94f7e6239e3d2f96 Mon Sep 17 00:00:00 2001 From: Peter Hutterer Date: Fri, 15 Mar 2024 10:11:54 +1000 Subject: [PATCH] Revert "test: allow for substring matching in the various --filter- arguments" Substring matching means it's impossible to select filters that are substrings of others, making it too painful to test one specific filter. This reverts commit a524cf97619f97c742f982065b6aa7698ca4779f. Part-of: --- test/litest.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/test/litest.c b/test/litest.c index ebe0a93b..d7a23031 100644 --- a/test/litest.c +++ b/test/litest.c @@ -383,7 +383,6 @@ litest_add_tcase_no_device(struct suite *suite, const char *test_name = funcname; if (filter_device && - strstr(test_name, filter_device) == NULL && fnmatch(filter_device, test_name, 0) != 0) return; @@ -409,7 +408,6 @@ litest_add_tcase_deviceless(struct suite *suite, const char *test_name = funcname; if (filter_device && - strstr(test_name, filter_device) == NULL && fnmatch(filter_device, test_name, 0) != 0) return; @@ -475,7 +473,6 @@ litest_add_tcase(const char *filename, litest_assert(excluded >= LITEST_DEVICELESS); if (filter_test && - strstr(funcname, filter_test) == NULL && fnmatch(filter_test, funcname, 0) != 0) return; @@ -502,7 +499,6 @@ litest_add_tcase(const char *filename, continue; if (filter_device && - strstr(dev->shortname, filter_device) == NULL && fnmatch(filter_device, dev->shortname, 0) != 0) continue; if ((dev->features & required) != required || @@ -524,7 +520,6 @@ litest_add_tcase(const char *filename, continue; if (filter_device && - strstr(dev->shortname, filter_device) == NULL && fnmatch(filter_device, dev->shortname, 0) != 0) continue; @@ -629,18 +624,17 @@ _litest_add_ranged_for_device(const char *filename, litest_assert(type < LITEST_NO_DEVICE); if (filter_test && - strstr(funcname, filter_test) == NULL && fnmatch(filter_test, funcname, 0) != 0) return; create_suite_name(filename, suite_name); + if (filter_group && fnmatch(filter_group, suite_name, 0) != 0) return; s = get_suite(suite_name); list_for_each(dev, &devices, node) { if (filter_device && - strstr(dev->shortname, filter_device) == NULL && fnmatch(filter_device, dev->shortname, 0) != 0) { device_filtered = true; continue;