test: abort on libinput bugs again

This was always intended but a bug prevented the actual abort.
strstr returns NULL when we cannot find the substring so we always
triggered the first noop condition on bugs.

Fixes: bd7b91065b13 ("evdev: warn if our event processing lags by 10ms or more")
Part-of: <https://gitlab.freedesktop.org/libinput/libinput/-/merge_requests/1048>

diff --git a/test/litest.c b/test/litest.c
index 787e84ae5421dc74d2b6a18928f33cce02314fae..c2b21bdba2f7737d6feef8d013332687139baf9e 100644 (file)
--- a/test/litest.c
+++ b/test/litest.c
@@ -733,9 +733,9 @@ litest_log_handler(struct libinput *libinput,
                 * short, don't abort if during a valgrind run we get a
                 * negative offset */
                if ((RUNNING_ON_VALGRIND && in_debugger) ||
-                   !strstr(format, "scheduled expiry is in the past")) {
+                   strstr(format, "scheduled expiry is in the past")) {
                        /* noop */
-               } else if (!strstr(format, "event processing lagging behind")) {
+               } else if (strstr(format, "event processing lagging behind")) {
                        /* noop */
                } else {
                        litest_abort_msg("libinput bug triggered, aborting.\n");