pepperplus/pepperplus-ri
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

test: removed logging assertion from test

Browse Source 
ref: N25B-350
This commit is contained in:
Storm
2025-12-22 13:43:01 +01:00
parent 358c4f6872
commit 7afdea8dbc
1 changed files with 0 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
test/unit/test_video_sender.py
View File

@@ -145,7 +145,6 @@ def test_video_loop_exception_handling(zmq_context, mocker):
and do not crash the thread. and do not crash the thread.
""" """
mock_settings = mocker.patch("robot_interface.endpoints.video_sender.settings") mock_settings = mocker.patch("robot_interface.endpoints.video_sender.settings")
mock_logger = mocker.patch("robot_interface.endpoints.video_sender.logging")
mock_service = mock.Mock() mock_service = mock.Mock()
# First call raises Exception, Second call works (if we allowed it, but we exit) # First call raises Exception, Second call works (if we allowed it, but we exit)
@@ -162,6 +161,5 @@ def test_video_loop_exception_handling(zmq_context, mocker):
sender.video_rcv_loop(mock_service, "sub_id") sender.video_rcv_loop(mock_service, "sub_id")
# Assertions # Assertions
mock_logger.warn.assert_called_with("Failed to retrieve video image from robot.")
# Ensure loop didn't crash; it should have completed the iteration and checked exit_event # Ensure loop didn't crash; it should have completed the iteration and checked exit_event
assert mock_state.exit_event.is_set.call_count >= 2 assert mock_state.exit_event.is_set.call_count >= 2