search

leka / LekaOS

LekaOS is Leka's firmware based on Mbed OS
https://leka.io
Apache License 2.0
14 stars 7 forks source link

Functions not called in constructor #555

Open YannLocatelli opened 2 years ago

YannLocatelli commented 2 years ago

Determine why functions are not called in constructor.

Happens in #508. Despite modification and successful tests, BLE does not work by initialise it via constructor. Please follow the discussion here: https://github.com/leka/LekaOS/pull/508#discussion_r820524422

YannLocatelli commented 2 years ago

Note concernant les unit tests:

Quand on met des appels de fonctions dans le constructeur, ils arrivent lorsque l'objet est instancié. Dans le cas des tests, c'est lorsque RobotControllerTest est "instancié" Donc l'instance à tester est appelé avant qu'on ait le temps d'indiquer les EXPECT_CALL. Un moyen de contourner ça est d'utiliser un pointeur pour instancier uniquement après les EXPECT_CALL, d'où la présence du constructorExpectedCalls avant le pointeur

Dans le contexte de la PR mentionné ci-dessus