Fix regression when changing the FC connection type

[amilcarlucas]

No description provided.

[Copilot]

Pull Request Overview

This PR refactors the update_protocol_combobox_entries method to optimize validation logic and improve code clarity. The method now validates the current protocol selection before updating the combobox instead of updating it twice (once with the current selection and once with None if invalid).

Key changes:

• Refactored validation logic to check protocol validity before updating the combobox
• Replaced .format(**locals()) with explicit named parameters for better code clarity and safety
• Updated test assertions to reflect the single combobox update call instead of two

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
ardupilot_methodic_configurator/frontend_tkinter_component_editor.py	Refactored update_protocol_combobox_entries to validate selection first and update combobox once; improved error message formatting
tests/test_frontend_tkinter_component_editor.py	Updated test assertions to expect single set_entries_tuple call instead of two; updated comments to reflect new behavior

Comments suppressed due to low confidence (1)

tests/test_frontend_tkinter_component_editor.py:37

• Call to ComponentEditorWindow.init with too few arguments; should be no fewer than 2.

        editor = ComponentEditorWindow()  # pylint: disable=no-value-for-parameter

[github-actions]

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines	Covered	Coverage	Threshold	Status
8897	7514	84%	80%	🟢

New Files

No new covered files...

Modified Files

File	Coverage	Status
ardupilot_methodic_configurator/frontend_tkinter_component_editor.py	89%	🟢
TOTAL	89%	🟢

updated for commit: 8098a2d by action🐍