From 1d5c1fedb45a50fcff7b05bba7da35c5735d9fd1 Mon Sep 17 00:00:00 2001 From: Peter Hutterer Date: Mon, 1 Aug 2016 14:38:34 +1000 Subject: [PATCH] test: drop the sysname comparison in the device add/remove test Running tests in parallel virtually guarantees a different device is added in between. What we're testing here is that the device comes back and the original ref doesn't send events, so a false test failure would still indicate a bug anyway. Signed-off-by: Peter Hutterer --- test/device.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/test/device.c b/test/device.c index 01f794f2..82ef03be 100644 --- a/test/device.c +++ b/test/device.c @@ -296,7 +296,7 @@ START_TEST(device_reenable_syspath_changed) { struct libinput *li; struct litest_device *litest_device; - struct libinput_device *device1, *device2; + struct libinput_device *device1; enum libinput_config_status status; struct libinput_event *event; @@ -315,12 +315,6 @@ START_TEST(device_reenable_syspath_changed) litest_drain_events(li); litest_device = litest_add_device(li, LITEST_MOUSE); - device2 = litest_device->libinput_device; - /* Note: if the sysname isn't the same, some other device got added - * or removed while this test was running. This is unlikely and - * would result in a false positive, so let's fail the test here */ - ck_assert_str_eq(libinput_device_get_sysname(device1), - libinput_device_get_sysname(device2)); status = libinput_device_config_send_events_set_mode(device1, LIBINPUT_CONFIG_SEND_EVENTS_ENABLED);