diff options
author | Peter Hutterer <peter.hutterer@who-t.net> | 2021-02-05 14:51:02 +1000 |
---|---|---|
committer | Peter Hutterer <peter.hutterer@who-t.net> | 2021-02-12 15:24:15 +1000 |
commit | d3115f48757c29150cbdd7a82d1e071b592bee7a (patch) | |
tree | 234680cc9f44ae53f566cc3812e44b5f24cfc19c /test/test-pad.c | |
parent | e6ed506df335e8eb0d79d22d22d0734a2b93fe43 (diff) |
test: drop the custom group names
The group names are forced by check (they are called suite names there) but
for our test suite they provide very little benefit. Much easier to just
use the filename a test is in as group name.
This removes the pure substring match for --filter-group, it's now fnmatch
only. group names are short enough that the typing isn't an issue and we don't
want to run tests twice (e.g. 'pad' is also in 'touchpad').
This patch caused #574 until it got fixed in d838e3a3a4c
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Diffstat (limited to 'test/test-pad.c')
-rw-r--r-- | test/test-pad.c | 52 |
1 files changed, 26 insertions, 26 deletions
diff --git a/test/test-pad.c b/test/test-pad.c index 54b2c4a3..cfa4c76e 100644 --- a/test/test-pad.c +++ b/test/test-pad.c @@ -984,38 +984,38 @@ END_TEST TEST_COLLECTION(tablet_pad) { - litest_add("pad:cap", pad_cap, LITEST_TABLET_PAD, LITEST_ANY); - litest_add("pad:cap", pad_no_cap, LITEST_ANY, LITEST_TABLET_PAD); + litest_add(pad_cap, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_no_cap, LITEST_ANY, LITEST_TABLET_PAD); - litest_add("pad:time", pad_time, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_time, LITEST_TABLET_PAD, LITEST_ANY); - litest_add("pad:button", pad_num_buttons, LITEST_TABLET_PAD, LITEST_ANY); - litest_add("pad:button", pad_num_buttons_libwacom, LITEST_TABLET_PAD, LITEST_ANY); - litest_add("pad:button", pad_button_intuos, LITEST_TABLET_PAD, LITEST_ANY); - litest_add("pad:button", pad_button_bamboo, LITEST_TABLET_PAD, LITEST_ANY); - litest_add("pad:button", pad_button_libwacom, LITEST_TABLET_PAD, LITEST_ANY); - litest_add("pad:button", pad_button_mode_groups, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_num_buttons, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_num_buttons_libwacom, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_button_intuos, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_button_bamboo, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_button_libwacom, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_button_mode_groups, LITEST_TABLET_PAD, LITEST_ANY); - litest_add("pad:ring", pad_has_ring, LITEST_RING, LITEST_ANY); - litest_add("pad:ring", pad_ring, LITEST_RING, LITEST_ANY); - litest_add("pad:ring", pad_ring_finger_up, LITEST_RING, LITEST_ANY); + litest_add(pad_has_ring, LITEST_RING, LITEST_ANY); + litest_add(pad_ring, LITEST_RING, LITEST_ANY); + litest_add(pad_ring_finger_up, LITEST_RING, LITEST_ANY); - litest_add("pad:strip", pad_has_strip, LITEST_STRIP, LITEST_ANY); - litest_add("pad:strip", pad_strip, LITEST_STRIP, LITEST_ANY); - litest_add("pad:strip", pad_strip_finger_up, LITEST_STRIP, LITEST_ANY); + litest_add(pad_has_strip, LITEST_STRIP, LITEST_ANY); + litest_add(pad_strip, LITEST_STRIP, LITEST_ANY); + litest_add(pad_strip_finger_up, LITEST_STRIP, LITEST_ANY); - litest_add_for_device("pad:left_handed", pad_left_handed_default, LITEST_WACOM_INTUOS5_PAD); - litest_add_for_device("pad:left_handed", pad_no_left_handed, LITEST_WACOM_INTUOS3_PAD); - litest_add_for_device("pad:left_handed", pad_left_handed_ring, LITEST_WACOM_INTUOS5_PAD); + litest_add_for_device(pad_left_handed_default, LITEST_WACOM_INTUOS5_PAD); + litest_add_for_device(pad_no_left_handed, LITEST_WACOM_INTUOS3_PAD); + litest_add_for_device(pad_left_handed_ring, LITEST_WACOM_INTUOS5_PAD); /* None of the current strip tablets are left-handed */ - litest_add("pad:modes", pad_mode_groups, LITEST_TABLET_PAD, LITEST_ANY); - litest_add("pad:modes", pad_mode_groups_userdata, LITEST_TABLET_PAD, LITEST_ANY); - litest_add("pad:modes", pad_mode_groups_ref, LITEST_TABLET_PAD, LITEST_ANY); - litest_add("pad:modes", pad_mode_group_mode, LITEST_TABLET_PAD, LITEST_ANY); - litest_add("pad:modes", pad_mode_group_has, LITEST_TABLET_PAD, LITEST_ANY); - litest_add("pad:modes", pad_mode_group_has_invalid, LITEST_TABLET_PAD, LITEST_ANY); - litest_add("pad:modes", pad_mode_group_has_no_toggle, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_mode_groups, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_mode_groups_userdata, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_mode_groups_ref, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_mode_group_mode, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_mode_group_has, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_mode_group_has_invalid, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_mode_group_has_no_toggle, LITEST_TABLET_PAD, LITEST_ANY); - litest_add("pad:keys", pad_keys, LITEST_TABLET_PAD, LITEST_ANY); + litest_add(pad_keys, LITEST_TABLET_PAD, LITEST_ANY); } |