Wrong usage of mock.called_once

[StephanHasler]

In several test files the called_once method of mock is used without the required parentheses ().
In this way the called_once statements have no effect, the tests seem to succeed, but actually nothing is tested.

See for example:

ROS-TCP-Endpoint/test/test_server.py

Line 75 in 58cd28f

mock_import_module.assert_called_once

In

ROS-TCP-Endpoint/test/test_server.py

Line 76 in 58cd28f

mock_sys_modules.assert_called_once

there is a further bug, besides the missing parentheses.
It should not be checked that the mock is called_once but its getitem method.

[hyounesy]

[AIRO-1689]

[github-actions]

This issue has been marked stale because it has been open for 14 days with no activity. Please remove the stale label or comment on this issue, or the issue will be automatically closed in the next 14 days.