eclipse-uprotocol / up-cpp

uProtocol Language Specific Library for C++
Apache License 2.0
16 stars 25 forks source link

Refactor UTransportMock to remove lifecycle testing, and so make suitable for inclusion in other tests. #177

Closed debruce closed 3 months ago

debruce commented 3 months ago

init/deinit tesing has been removed, and so the UTransportMock no longer needs to contain a static member for checking for deinitialization. Furthermore, the memory leak detector was removed, since this is supposed to be cover elsewhere by valgrind.

gregmedd commented 3 months ago

Could the mock info object just be a regular member in UTransportMock? Does it need to be a singleton or static at all?