Skip to content

Commit f556aa9

Browse files
author
Benjamin Tissoires
committed
selftests/hid: fix ruff linter complains
rename ambiguous variables l, r, and m, and ignore the return values of uhdev.get_evdev() and uhdev.get_slot() Acked-by: Jiri Kosina <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Benjamin Tissoires <[email protected]>
1 parent ed5bc56 commit f556aa9

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

tools/testing/selftests/hid/tests/test_mouse.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -52,13 +52,13 @@ def create_report(self, x, y, buttons=None, wheels=None, reportID=None):
5252
:param reportID: the numeric report ID for this report, if needed
5353
"""
5454
if buttons is not None:
55-
l, r, m = buttons
56-
if l is not None:
57-
self.left = l
58-
if r is not None:
59-
self.right = r
60-
if m is not None:
61-
self.middle = m
55+
left, right, middle = buttons
56+
if left is not None:
57+
self.left = left
58+
if right is not None:
59+
self.right = right
60+
if middle is not None:
61+
self.middle = middle
6262
left = self.left
6363
right = self.right
6464
middle = self.middle

tools/testing/selftests/hid/tests/test_wacom_generic.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -909,14 +909,14 @@ def test_confidence_false(self):
909909
Ensure that the confidence bit being set to false should not result in a touch event.
910910
"""
911911
uhdev = self.uhdev
912-
evdev = uhdev.get_evdev()
912+
_evdev = uhdev.get_evdev()
913913

914914
t0 = test_multitouch.Touch(1, 50, 100)
915915
t0.confidence = False
916916
r = uhdev.event([t0])
917917
events = uhdev.next_sync_events()
918918
self.debug_reports(r, uhdev, events)
919919

920-
slot = self.get_slot(uhdev, t0, 0)
920+
_slot = self.get_slot(uhdev, t0, 0)
921921

922-
assert not events
922+
assert not events

0 commit comments

Comments
 (0)