From 500ad0510f951950b4d2ace7fb9877c68ec2375b Mon Sep 17 00:00:00 2001 From: Ian Douglas Scott Date: Thu, 31 Jul 2025 19:33:26 -0700 Subject: [PATCH] test: Advertise scroll/button interfaces in `test_connect_receive_seat` This test was checking for the capabilities for `EI_SCROLL` and `EI_BUTTON`, but not advertising those interfaces initially. Presumably this was unintentional, given there's no comment that it's intentionally advertising some protocols but not others. Also, should the test have failed to find the capabilities when it didn't advertise the protocol to to start with? This change just fixes the test, anyway. This was causing the test to fail for `reis` CI in a proposed change in https://github.com/ids1024/reis/pull/11. Part-of: --- test/test_protocol.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/test/test_protocol.py b/test/test_protocol.py index 9f25eef..2e956b9 100644 --- a/test/test_protocol.py +++ b/test/test_protocol.py @@ -571,6 +571,8 @@ class TestEiProtocol: InterfaceName.EI_POINTER_ABSOLUTE, InterfaceName.EI_KEYBOARD, InterfaceName.EI_TOUCHSCREEN, + InterfaceName.EI_SCROLL, + InterfaceName.EI_BUTTON, ]: ei.send( setup.InterfaceVersion(interface, VERSION_V(1))