Review all ignoreLogMessage usage in test/Vehicle/VehicleLinkManagerTest.cc.
Under the new strict log checking (PR #14508), several tests rely on ignoreLogMessage to suppress warnings/debug messages that leak through during comm-loss and link-switch scenarios (e.g. MavCommandQueue "Giving up sending command…", RequestMetaDataTypeStateMachine "failed to load metadata", and the Switching communication to secondary link app message).
These ignores were added to get the test green, but some of them may be masking real timing/behavioral issues rather than benign expected output. Audit each ignoreLogMessage call to confirm:
- The message is genuinely expected for the scenario being tested, not a symptom of a bug.
- The pattern/category/level are as narrow as possible (no over-broad suppression).
- Any timing-dependent warning that only sometimes appears is intentionally optional.
Review all
ignoreLogMessageusage intest/Vehicle/VehicleLinkManagerTest.cc.Under the new strict log checking (PR #14508), several tests rely on
ignoreLogMessageto suppress warnings/debug messages that leak through during comm-loss and link-switch scenarios (e.g.MavCommandQueue"Giving up sending command…",RequestMetaDataTypeStateMachine"failed to load metadata", and theSwitching communication to secondary linkapp message).These ignores were added to get the test green, but some of them may be masking real timing/behavioral issues rather than benign expected output. Audit each
ignoreLogMessagecall to confirm: