[libcamera-devel,v4,02/16] py: unittests: Fix test_select()
diff mbox series

Message ID 20220530142722.57618-3-tomi.valkeinen@ideasonboard.com
State Accepted
Headers show
Series
  • More misc Python patches
Related show

Commit Message

Tomi Valkeinen May 30, 2022, 2:27 p.m. UTC 
The test_select() currently uses self.assertTrue(len(ready_reqs) > 0) to
see that cm.get_ready_requests() returns something. This is not always
the case, as there may be two eventfd events queued, and the first call
to cm.get_ready_requests() returns all the requests, and thus the second
call returns none.

Remove the self.assertTrue(len(ready_reqs) > 0) assert.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
 test/py/unittests.py | 2 --
 1 file changed, 2 deletions(-)

Patch
diff mbox series 

diff --git a/test/py/unittests.py b/test/py/unittests.py
index 45b35223..33b35a0a 100755
--- a/test/py/unittests.py
+++ b/test/py/unittests.py
@@ -287,8 +287,6 @@  class SimpleCaptureMethods(CameraTesterBase):
 
                 ready_reqs = cm.get_ready_requests()
 
-                self.assertTrue(len(ready_reqs) > 0)
-
                 reqs += ready_reqs
 
                 if len(reqs) == num_bufs: