From 463031e38c44997aa00d8db07cfed09849dbae78 Mon Sep 17 00:00:00 2001 From: mikecarper Date: Wed, 28 Oct 2026 09:15:27 -0700 Subject: [PATCH] Update preference snapshot contract for logging policy --- test/test_companion_settings_persistence_contract.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/test/test_companion_settings_persistence_contract.py b/test/test_companion_settings_persistence_contract.py index 0702ae214..4776117f6 100644 --- a/test/test_companion_settings_persistence_contract.py +++ b/test/test_companion_settings_persistence_contract.py @@ -196,13 +196,13 @@ class CompanionSettingsPersistenceContractTests(unittest.TestCase): "bool copyPersistedValuesFrom(const CompanionNodePrefs& source)", ) self.assertIn("reinterpret_cast(&airtime_factor)", copy) - # Both common traffic preferences follow the optional RAK policy. - # The final value may never include either runtime adapter. - self.assertIn("const auto& last_value = tx_ssh;", copy) + # The always-log value follows both common traffic preferences. Include + # that final scalar while keeping both runtime adapters outside the copy. + self.assertIn("const auto& last_value = usb_logging_always;", copy) self.assertIn("reinterpret_cast(&last_value)", copy) - self.assertIn("&values.tx_ssh", copy) - self.assertIn("+ sizeof(tx_ssh)", copy) - self.assertIn("end - begin == size - sizeof(values.tx_ssh)", copy) + self.assertIn("&values.usb_logging_always", copy) + self.assertIn("+ sizeof(usb_logging_always)", copy) + self.assertIn("end - begin == size - sizeof(values.usb_logging_always)", copy) self.assertIn("reinterpret_cast(&values.radio)", copy) self.assertIn("reinterpret_cast(&values.custom)", copy) self.assertIn("size <= radio_begin - begin", copy)