diff --git a/sonic-chassisd/scripts/chassisd b/sonic-chassisd/scripts/chassisd index f114af598..69e9f9d54 100755 --- a/sonic-chassisd/scripts/chassisd +++ b/sonic-chassisd/scripts/chassisd @@ -682,8 +682,8 @@ class SmartSwitchModuleUpdater(ModuleUpdater): key = module_info_dict[CHASSIS_MODULE_INFO_NAME_FIELD] if not key.startswith(ModuleBase.MODULE_TYPE_DPU): - self.log_error("Incorrect module-name {}. Should start with {} or {} or {} or {}".format(key, - ModuleBase.MODULE_TYPE_DPU)) + self.log_error("Incorrect module-name {}. Should start with {} ".format(key, + ModuleBase.MODULE_TYPE_DPU)) continue fvs = swsscommon.FieldValuePairs([(CHASSIS_MODULE_INFO_DESC_FIELD, module_info_dict[CHASSIS_MODULE_INFO_DESC_FIELD]), diff --git a/sonic-chassisd/tests/test_chassisd.py b/sonic-chassisd/tests/test_chassisd.py index 0be780cd3..aa78ffaa1 100644 --- a/sonic-chassisd/tests/test_chassisd.py +++ b/sonic-chassisd/tests/test_chassisd.py @@ -128,13 +128,13 @@ def test_smartswitch_moduleupdater_check_invalid_name(): slot, module.supervisor_slot) module_updater.module_db_update() fvs = module_updater.module_table.get(name) - # assert fvs == None + assert fvs == None admin_state = 0 config_updater.module_config_update(name, admin_state) # No change since invalid key - # assert module.get_admin_state() != admin_state + assert module.get_admin_state() != admin_state def test_moduleupdater_check_invalid_name(): chassis = MockChassis()