-
Notifications
You must be signed in to change notification settings - Fork 421
Remove #define private public from all tests #529
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
#define private public
from all tests
oh, I also added it to the Windows Issue. |
@elBoberido yes right, this is for the new contributors to get familiar. |
@dkroenke I had a look at it yesterday and at least one is not quite trivial to solve. Shall I tag the files from easy to difficult? |
@elBoberido Yes that definitely make sense |
@dkroenke done. I think mq_interface will be refactored in the near future and should maybe be done with that refactoring |
…di_service_registry.cpp Signed-off-by: Mathias Kraus <[email protected]>
…di_service_registry.cpp Signed-off-by: Mathias Kraus <[email protected]>
…bjectpool iox-#529 Remove define private public from test_objectpool
* Use real subscriber to get introspection data (slows down a test unfortunately) instead of internal class members
* Use real subscriber to get introspection data (slows down a test unfortunately) instead of internal class members
* Use real subscriber to get introspection data (slows down a test unfortunately) instead of internal class members
* Use real subscriber to get introspection data (slows down a test unfortunately) instead of internal class members
…mepoo iox-#529 Remove define private from mepoo test
Required information
In some of the iceoryx tests are defines used for making private=public which is against our principle of white-box testing.
Additionally this doesn't compile on windows.
Task
#define private public
and#define protected public
from all testfilesThe text was updated successfully, but these errors were encountered: