Closed jadecastro closed 3 years ago
@stonier I think gmock
was a stray include (carried over from SimpleCar's test). It can be removed since we're not actually mocking anything here.
In addition to the inline changes, I've reworked the agent version of UnicycleCar
, UnicycleCarAgent
, so that it doesn't rely any longer on ignition messaging. The benefit for this PR is that all the ignition-related translators for the AngularRateAccelerationCommand
aren't needed anymore, which sharply reduces the infrastructure added in this PR and the one in delphyne_gui (https://github.com/ToyotaResearchInstitute/delphyne-gui/pull/322).
@jadecastro I rebased and then removed the unused gmock header in 1a3ccb0 since it failed to compile after the rebase (there was a change to use system gtest in #717 yesterday. Was there any reason you were including gmock?