Replies: 1 comment
-
@elfenpiff I might a bit late with my answer but I think this is a great idea and we should move this back #543 |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Brief feature description
The issue #542 showed that we - despite a good test coverage and a abstraction - we still have resource leaks. Therefore, we require a testing platform/framework similar to the Windows, MacOS platform files in iceoryx utils.
It should be realized in the following way:
iox_
prefix. Like it is already done with the semaphore where we use onlyiox_sem_open
in the code.iox_sem_open("mySemaphore")
is called it will track that there exist amySemaphore
in the system and wheniox_sem_close(mySemaphoreHandle)
is called it will release it.iceoryx_utils/platform/testing
platform. When we never call the posix calls directly and forward the arguments to the actual posix calls we have gained unit tests:@dkroenke @elBoberido what do you think?
Beta Was this translation helpful? Give feedback.
All reactions