ardupilot-methodic-configurator 1.4.8__py3-none-any.whl → 1.5.0__py3-none-any.whl

ardupilot_methodic_configurator/frontend_tkinter_component_editor_base.py

@@ -19,11 +19,19 @@ from logging import getLevelName as logging_getLevelName
 from logging import info as logging_info
 from sys import exit as sys_exit
 from tkinter import messagebox, ttk
-from typing import Any, Union
+from typing import Optional, Union, cast
+from unittest.mock import patch
 
 from ardupilot_methodic_configurator import _, __version__
 from ardupilot_methodic_configurator.backend_filesystem import LocalFilesystem
 from ardupilot_methodic_configurator.common_arguments import add_common_arguments
+from ardupilot_methodic_configurator.data_model_vehicle_components import ComponentDataModel
+from ardupilot_methodic_configurator.data_model_vehicle_components_base import (
+    ComponentData,
+    ComponentPath,
+    ComponentValue,
+    ValidationRulePath,
+)
 from ardupilot_methodic_configurator.frontend_tkinter_base_window import BaseWindow
 from ardupilot_methodic_configurator.frontend_tkinter_component_template_manager import ComponentTemplateManager
 from ardupilot_methodic_configurator.frontend_tkinter_rich_text import RichText
@@ -55,7 +63,14 @@ def argument_parser() -> Namespace:
     # pylint: enable=duplicate-code
 
 
-class ComponentEditorWindowBase(BaseWindow):
+# Type aliases to improve code readability
+EntryWidget = Union[ttk.Entry, ttk.Combobox]
+
+WINDOW_WIDTH_PIX = 880
+VEICLE_IMAGE_WIDTH_PIX = 100
+
+
+class ComponentEditorWindowBase(BaseWindow):  # pylint: disable=too-many-instance-attributes
     """
     A class for editing JSON files in the ArduPilot methodic configurator.
 
@@ -64,26 +79,79 @@ class ComponentEditorWindowBase(BaseWindow):
     class, which provides basic window functionality.
     """
 
-    def __init__(self, version: str, local_filesystem: LocalFilesystem) -> None:
-        super().__init__()
-        self.local_filesystem = local_filesystem
-
-        self.root.title(_("Amilcar Lucas's - ArduPilot methodic configurator ") + version + _(" - Vehicle Component Editor"))
-        self.root.geometry("880x600")  # Set the window width
+    def __init__(
+        self,
+        version: str,
+        local_filesystem: LocalFilesystem,
+        data_model: Optional[ComponentDataModel] = None,
+        root_tk: Optional[tk.Tk] = None,
+    ) -> None:
+        """
+        Initialize the ComponentEditorWindowBase.
 
-        self.root.protocol("WM_DELETE_WINDOW", self.on_closing)
+        Args:
+            version: Application version string
+            local_filesystem: Filesystem interface for loading/saving data
+            data_model: Optional pre-configured data model (for testing)
+            root_tk: Optional parent Tk window (for testing)
 
-        self.data = local_filesystem.load_vehicle_components_json_data(local_filesystem.vehicle_dir)
-        if len(self.data) < 1:
+        """
+        super().__init__(root_tk)
+        self.local_filesystem = local_filesystem
+        self.version = version
+
+        # Initialize the data model - allow injection for testing
+        self.data_model = data_model or self._create_data_model()
+
+        # UI elements dictionary for easier access and testing
+        self.entry_widgets: dict[ComponentPath, EntryWidget] = {}
+        self.scroll_frame: ScrollFrame
+        self.save_button: ttk.Button
+        self.template_manager: ComponentTemplateManager
+
+        # Initialize UI if there's data to work with
+        if self._check_data():
+            self._initialize_ui()
+            self._finalize_initialization()
+
+    def _create_data_model(self) -> ComponentDataModel:
+        """Create the data model. Extracted for better testability."""
+        raw_data = self.local_filesystem.load_vehicle_components_json_data(self.local_filesystem.vehicle_dir)
+        component_datatypes = self.local_filesystem.get_all_value_datatypes()
+        return ComponentDataModel(raw_data, component_datatypes)
+
+    def _initialize_ui(self) -> None:
+        """Initialize the complete UI. Extracted for better testability."""
+        self._setup_window()
+        self._setup_styles()
+        self._create_intro_frame()
+        self._create_scroll_frame()
+        self._setup_template_manager()
+        self._create_save_frame()
+        self._check_show_usage_instructions()
+
+    def _finalize_initialization(self) -> None:
+        """Finalize initialization after UI setup. Extracted for better testability."""
+        self.data_model.post_init(self.local_filesystem.doc_dict)
+
+    def _check_data(self) -> bool:
+        """Check if we have data to work with and prepare for UI setup."""
+        if not self.data_model.is_valid_component_data() or not self.data_model.has_components():
             # Schedule the window to be destroyed after the mainloop has started
-            self.root.after(100, self.root.destroy)  # Adjust the delay as needed
-            return
-
-        self.entry_widgets: dict[tuple, Union[ttk.Entry, ttk.Combobox]] = {}
-
-        intro_frame = ttk.Frame(self.main_frame)
-        intro_frame.pack(side=tk.TOP, fill="x", expand=False)
+            self.root.after(100, self.root.destroy)
+            return False
+        return True
+
+    def _setup_window(self) -> None:
+        """Setup the main window properties."""
+        self.root.title(
+            _("Amilcar Lucas's - ArduPilot methodic configurator ") + self.version + _(" - Vehicle Component Editor")
+        )
+        self.root.geometry(f"{WINDOW_WIDTH_PIX}x600")
+        self.root.protocol("WM_DELETE_WINDOW", self.on_closing)
 
+    def _setup_styles(self) -> None:
+        """Configure the styles for UI elements."""
         style = ttk.Style()
         style.configure("bigger.TLabel", font=("TkDefaultFont", 13))
         style.configure("comb_input_invalid.TCombobox", fieldbackground="red")
@@ -93,29 +161,48 @@ class ComponentEditorWindowBase(BaseWindow):
         style.configure("Optional.TLabelframe", borderwidth=2)
         style.configure("Optional.TLabelframe.Label", foreground="gray")
 
+    def _create_intro_frame(self) -> None:
+        """Create the introduction frame with explanations and image."""
+        intro_frame = ttk.Frame(self.main_frame)
+        intro_frame.pack(side=tk.TOP, fill="x", expand=False)
+
+        self._add_explanation_text(intro_frame)
+        self._add_vehicle_image(intro_frame)
+
+    def _add_explanation_text(self, parent: ttk.Frame) -> None:
+        """Add explanation text to the parent frame."""
         explanation_text = _("Please configure properties of the vehicle components.\n")
         explanation_text += _("Labels for optional properties are displayed in gray text.\n")
         explanation_text += _("Scroll down to ensure that you do not overlook any properties.\n")
-        explanation_label = ttk.Label(intro_frame, text=explanation_text, wraplength=800, justify=tk.LEFT)
+
+        explanation_label = ttk.Label(
+            parent, text=explanation_text, wraplength=WINDOW_WIDTH_PIX - VEICLE_IMAGE_WIDTH_PIX, justify=tk.LEFT
+        )
         explanation_label.configure(style="bigger.TLabel")
         explanation_label.pack(side=tk.LEFT, padx=(10, 10), pady=(10, 0), anchor=tk.NW)
 
-        # Load the vehicle image and scale it down to image_height pixels in height
-        if local_filesystem.vehicle_image_exists():
-            image_label = self.put_image_in_label(intro_frame, local_filesystem.vehicle_image_filepath(), 100)
+    def _add_vehicle_image(self, parent: ttk.Frame) -> None:
+        """Add the vehicle image to the parent frame."""
+        if self.local_filesystem.vehicle_image_exists():
+            image_label = self.put_image_in_label(
+                parent, self.local_filesystem.vehicle_image_filepath(), VEICLE_IMAGE_WIDTH_PIX
+            )
             image_label.pack(side=tk.RIGHT, anchor=tk.NE, padx=(4, 4), pady=(4, 0))
             show_tooltip(image_label, _("Replace the vehicle.jpg file in the vehicle directory to change the vehicle image."))
         else:
-            image_label = ttk.Label(intro_frame, text=_("Add a 'vehicle.jpg' image file to the vehicle directory."))
+            image_label = ttk.Label(parent, text=_("Add a 'vehicle.jpg' image file to the vehicle directory."))
             image_label.pack(side=tk.RIGHT, anchor=tk.NE, padx=(4, 4), pady=(4, 0))
 
+    def _create_scroll_frame(self) -> None:
+        """Create the scrollable frame for component widgets."""
         self.scroll_frame = ScrollFrame(self.main_frame)
         self.scroll_frame.pack(side="top", fill="both", expand=True)
 
-        self.update_json_data()
-
+    def _create_save_frame(self) -> None:
+        """Create the frame with save button."""
         save_frame = ttk.Frame(self.main_frame)
         save_frame.pack(side=tk.TOP, fill="x", expand=False)
+
         self.save_button = ttk.Button(
             save_frame, text=_("Save data and start configuration"), command=self.validate_and_save_component_json
         )
@@ -124,23 +211,30 @@ class ComponentEditorWindowBase(BaseWindow):
             _("Save component data to the vehicle_components.json file\nand start parameter value configuration and tuning."),
         )
         self.save_button.pack(pady=7)
-        if UsagePopupWindow.should_display("component_editor"):
-            self.root.after(10, self.__display_component_editor_usage_instructions(self.root))  # type: ignore[arg-type]
+
+    def _setup_template_manager(self) -> None:
+        """Set up the component template manager."""
 
         def update_data_callback(comp_name: str, template_data: dict) -> None:
-            self.data["Components"][comp_name] = template_data
+            self.data_model.update_component(comp_name, template_data)
 
         self.template_manager = ComponentTemplateManager(
             self.root,
             self.entry_widgets,
             self.get_component_data_from_gui,
             update_data_callback,
-            self.derive_initial_template_name,
-            local_filesystem.save_component_to_system_templates,
+            self.data_model.derive_initial_template_name,
+            self.local_filesystem.save_component_to_system_templates,
         )
 
-    @staticmethod
-    def __display_component_editor_usage_instructions(parent: tk.Tk) -> None:
+    def _check_show_usage_instructions(self) -> None:
+        """Check if usage instructions should be displayed."""
+        if UsagePopupWindow.should_display("component_editor"):
+            # Cast to Tk since we know root is a Tk instance in this context
+            self.root.after(10, lambda: self._display_component_editor_usage_instructions(cast("tk.Tk", self.root)))
+
+    def _display_component_editor_usage_instructions(self, parent: tk.Tk) -> None:
+        """Display usage instructions for the component editor."""
         usage_popup_window = BaseWindow(parent)
         style = ttk.Style()
 
@@ -165,27 +259,30 @@ class ComponentEditorWindowBase(BaseWindow):
             instructions_text,
         )
 
-    def update_json_data(self) -> None:  # should be overwritten in child classes
-        if "Format version" not in self.data:
-            self.data["Format version"] = 1
-
-    def _set_component_value_and_update_ui(self, path: tuple, value: str) -> None:
-        data_path = self.data["Components"]
-        for key in path[:-1]:
-            data_path = data_path[key]
-        data_path[path[-1]] = value
-        entry = self.entry_widgets[path]
-        entry.delete(0, tk.END)
-        entry.insert(0, value)
-        entry.config(state="disabled")
+    def set_component_value_and_update_ui(self, path: ComponentPath, value: str) -> None:
+        """Set a component value and update the UI to reflect it."""
+        self.data_model.set_component_value(path, value)
+        if path in self.entry_widgets:
+            entry = self.entry_widgets[path]
+            entry.delete(0, tk.END)
+            entry.insert(0, value)
+            entry.config(state="disabled")
 
     def populate_frames(self) -> None:
         """Populates the ScrollFrame with widgets based on the JSON data."""
-        if "Components" in self.data:
-            for key, value in self.data["Components"].items():
-                self._add_widget(self.scroll_frame.view_port, key, value, [])
+        components = self.data_model.get_all_components()
+        for key, value in components.items():
+            self.add_widget(self.scroll_frame.view_port, key, value, [])
+
+    def add_widget(self, parent: tk.Widget, key: str, value: ComponentValue, path: list[str]) -> None:
+        """
+        Adds a widget to the parent widget with the given key and value.
+
+        Public version for better testability.
+        """
+        self._add_widget(parent, key, value, path)
 
-    def _add_widget(self, parent: tk.Widget, key: str, value: Union[dict, float], path: list) -> None:
+    def _add_widget(self, parent: tk.Widget, key: str, value: ComponentValue, path: list[str]) -> None:
         """
         Adds a widget to the parent widget with the given key and value.
 
@@ -197,156 +294,146 @@ class ComponentEditorWindowBase(BaseWindow):
 
         """
         if isinstance(value, dict):  # JSON non-leaf elements, add LabelFrame widget
-            self.__add_non_leaf_widget(parent, key, value, path)
+            self._add_non_leaf_widget(parent, key, value, path)
         else:  # JSON leaf elements, add Entry widget
-            self.__add_leaf_widget(parent, key, value, path)
-
-    def __add_non_leaf_widget(self, parent: tk.Widget, key: str, value: dict, path: list) -> None:
-        is_toplevel = parent == self.scroll_frame.view_port
-        pady = 5 if is_toplevel else 3
+            self._add_leaf_widget(parent, key, value, path)
 
+    def _prepare_non_leaf_widget_config(self, key: str, value: dict, path: list[str]) -> dict:
+        """Prepare configuration for non-leaf widget creation. Pure function for easy testing."""
+        is_toplevel = len(path) == 0  # More explicit than checking parent type
         current_path = (*path, key)
         description, is_optional = self.local_filesystem.get_component_property_description(current_path)
         description = _(description) if description else ""
 
-        if is_optional:
-            frame = ttk.LabelFrame(parent, text=_(key), style="Optional.TLabelframe")
+        # Enhance tooltip for optional fields
+        if is_optional and description:
+            description += _("\nThis is optional and can be left blank")
+
+        return {
+            "key": key,
+            "value": value,
+            "path": current_path,
+            "description": description,
+            "is_optional": is_optional,
+            "is_toplevel": is_toplevel,
+            "pady": 5 if is_toplevel else 3,
+        }
+
+    def _create_non_leaf_widget_ui(self, parent: tk.Widget, config: dict) -> ttk.LabelFrame:
+        """Create the UI elements for a non-leaf widget. Separated for better testability."""
+        if config["is_optional"]:
+            frame = ttk.LabelFrame(parent, text=_(config["key"]), style="Optional.TLabelframe")
         else:
-            frame = ttk.LabelFrame(parent, text=_(key))
+            frame = ttk.LabelFrame(parent, text=_(config["key"]))
 
         frame.pack(
-            fill=tk.X, side=tk.TOP if is_toplevel else tk.LEFT, pady=pady, padx=5, anchor=tk.NW if is_toplevel else tk.N
+            fill=tk.X,
+            side=tk.TOP if config["is_toplevel"] else tk.LEFT,
+            pady=config["pady"],
+            padx=5,
+            anchor=tk.NW if config["is_toplevel"] else tk.N,
         )
 
-        # Enhance tooltip for optional fields
-        if is_optional and description:
-            description += _("\nThis is optional and can be left blank")
-
         # Add tooltip based on schema description
-        if description:
-            show_tooltip(frame, description, position_below=False)
+        if config["description"]:
+            show_tooltip(frame, config["description"], position_below=False)
+
+        return frame
 
-        if is_toplevel and key in self.data.get("Components", {}):
+    def _add_non_leaf_widget(self, parent: tk.Widget, key: str, value: dict, path: list[str]) -> None:
+        """Add a non-leaf widget (frame containing other widgets) to the UI."""
+        # Extract business logic to separate method for better testability
+        widget_config = self._prepare_non_leaf_widget_config(key, value, path)
+        frame = self._create_non_leaf_widget_ui(parent, widget_config)
+
+        if widget_config["is_toplevel"] and key in self.data_model.get_all_components():
             self._add_template_controls(frame, key)
 
         for sub_key, sub_value in value.items():
             # recursively add child elements
             self._add_widget(frame, sub_key, sub_value, [*path, key])
 
-    def __add_leaf_widget(self, parent: tk.Widget, key: str, value: float, path: list) -> None:
-        entry_frame = ttk.Frame(parent)
-        entry_frame.pack(side=tk.TOP, fill=tk.X, pady=(0, 5))
+    def _add_leaf_widget(self, parent: tk.Widget, key: str, value: Union[str, float], path: list[str]) -> None:
+        """Add a leaf widget (containing input controls) to the UI."""
+        # Extract business logic to separate method for better testability
+        widget_config = self._prepare_leaf_widget_config(key, value, path)
+        self._create_leaf_widget_ui(parent, widget_config)
 
-        description, is_optional = self.local_filesystem.get_component_property_description((*path, key))
+    def _prepare_leaf_widget_config(self, key: str, value: Union[str, float], path: list[str]) -> dict:
+        """Prepare configuration for leaf widget creation. Pure function for easy testing."""
+        component_path = (*path, key)
+        description, is_optional = self.local_filesystem.get_component_property_description(component_path)
         description = _(description) if description else ""
 
-        label = ttk.Label(entry_frame, text=_(key), foreground="gray" if is_optional else "black")
+        # Enhance tooltip for optional fields
+        if is_optional and description:
+            description += _("\nThis is optional and can be left blank")
+
+        return {
+            "key": key,
+            "value": value,
+            "path": component_path,
+            "description": description,
+            "is_optional": is_optional,
+            "label_color": "gray" if is_optional else "black",
+        }
+
+    def _create_leaf_widget_ui(self, parent: tk.Widget, config: dict) -> None:
+        """Create the UI elements for a leaf widget. Separated for better testability."""
+        entry_frame = ttk.Frame(parent)
+        entry_frame.pack(side=tk.TOP, fill=tk.X, pady=(0, 5))
 
+        label = ttk.Label(entry_frame, text=_(config["key"]), foreground=config["label_color"])
         label.pack(side=tk.LEFT)
 
-        entry = self.add_entry_or_combobox(value, entry_frame, (*path, key), is_optional)
+        entry = self.add_entry_or_combobox(config["value"], entry_frame, config["path"], config["is_optional"])
         entry.pack(side=tk.LEFT, expand=True, fill=tk.X, padx=(0, 5))
 
-        # Store the entry widget in the entry_widgets dictionary for later retrieval
-        self.entry_widgets[(*path, key)] = entry
-
-        # Enhance tooltip for optional fields
-        if is_optional and description:
-            description += _("\nThis is optional and can be left blank")
+        # Store the entry widget in the entry_widgets Dictionary for later retrieval
+        self.entry_widgets[config["path"]] = entry
 
         # Add tooltip based on schema description
-        if description:
-            show_tooltip(label, description)
-            show_tooltip(entry, description)
+        if config["description"]:
+            show_tooltip(label, config["description"])
+            show_tooltip(entry, config["description"])
 
     def _add_template_controls(self, parent_frame: ttk.LabelFrame, component_name: str) -> None:
         """Add template controls for a component."""
         self.template_manager.add_template_controls(parent_frame, component_name)
 
-    def derive_initial_template_name(self, component_data: dict[str, Any]) -> str:  # pylint: disable=unused-argument # noqa: ARG002
-        """
-        Derive an initial template name from the component data.
+    def get_component_data_from_gui(self, component_name: str) -> ComponentData:
+        """Extract component data from GUI elements."""
+        # Get all entry widget values as a dictionary
+        entry_values = {path: entry.get() for path, entry in self.entry_widgets.items()}
 
-        This is a basic implementation that can be overridden in derived classes
-        to provide more specific template naming logic.
-        """
-        return ""
-
-    def get_component_data_from_gui(self, component_name: str) -> dict[str, Any]:
-        """Save the current component configuration as a template."""
-        # Get fresh component data from the GUI elements instead of stored data
-        component_data: dict[str, Any] = {}
-
-        # Find all entry widgets belonging to this component and extract their values
-        for path, entry in self.entry_widgets.items():
-            if len(path) >= 1 and path[0] == component_name:
-                # Get the current value from the entry widget
-                value: Union[str, int, float] = entry.get()
-
-                # Try to convert to appropriate type (int, float, or string)
-                if path[-1] != "Version":
-                    try:
-                        value = int(value)
-                    except ValueError:
-                        try:
-                            value = float(value)
-                        except ValueError:
-                            value = str(value).strip()
-
-                # Create nested structure by navigating through the path
-                current_level = component_data
-                for key in path[1:-1]:  # Skip component_name and the last key
-                    if key not in current_level:
-                        current_level[key] = {}
-                    current_level = current_level[key]
-
-                # Set the value at the final level
-                current_level[path[-1]] = value
-        return component_data
+        # Use the data model to extract and process the component data
+        return self.data_model.extract_component_data_from_entries(component_name, entry_values)
 
     def validate_and_save_component_json(self) -> None:
-        """Saves the edited JSON data back to the file."""
+        """Validate and save the component JSON data."""
+        if self._confirm_component_properties():
+            self.save_component_json()
+
+    def _confirm_component_properties(self) -> bool:
+        """Show confirmation dialog for component properties."""
         confirm_message = _(
             "ArduPilot Methodic Configurator only operates correctly if all component properties are correct."
             " ArduPilot parameter values depend on the components used and their connections.\n\n"
             " Have you used the scrollbar on the right side of the window and "
             "entered the correct values for all components?"
         )
-        user_confirmation = messagebox.askyesno(_("Confirm that all component properties are correct"), confirm_message)
-
-        if not user_confirmation:
-            # User chose 'No', so return and do not save data
-            return
-
-        self.save_component_json()
+        return messagebox.askyesno(_("Confirm that all component properties are correct"), confirm_message)
 
     def save_component_json(self) -> None:
-        # User confirmed, proceed with saving the data
-        for path, entry in self.entry_widgets.items():
-            value: Union[str, int, float] = entry.get()
-            # Navigate through the nested dictionaries using the elements of the path
-            current_data = self.data["Components"]
-            for key in path[:-1]:
-                current_data = current_data[key]
-
-            if path[-1] != "Version":
-                try:
-                    value = int(value)
-                except ValueError:
-                    try:
-                        value = float(value)
-                    except ValueError:
-                        value = str(value).strip()
-
-            # Update the value in the data dictionary
-            current_data[path[-1]] = value
-
-        # Save the updated data back to the JSON file
-        failed, msg = self.local_filesystem.save_vehicle_components_json_data(self.data, self.local_filesystem.vehicle_dir)
+        """Save component JSON data to file."""
+        # Use data model to save (which handles validation and data processing)
+        failed, msg = self.data_model.save_to_filesystem(self.local_filesystem)
+
         if failed:
             show_error_message(_("Error"), _("Failed to save data to file.") + "\n" + msg)
         else:
             logging_info(_("Vehicle component data saved successfully."))
+
         self.root.destroy()
 
     def on_closing(self) -> None:
@@ -365,17 +452,19 @@ class ComponentEditorWindowBase(BaseWindow):
     # This function will be overwritten in child classes
     def add_entry_or_combobox(
         self,
-        value: float,
+        value: Union[str, float],
         entry_frame: ttk.Frame,
-        _path: tuple[str, str, str],
+        _path: ValidationRulePath,
         is_optional: bool = False,  # pylint: disable=unused-argument # noqa: ARG002
-    ) -> Union[ttk.Entry, ttk.Combobox]:
+    ) -> EntryWidget:
+        """Create an entry widget for input values."""
         entry = ttk.Entry(entry_frame)
         entry.insert(0, str(value))
         return entry
 
     @staticmethod
     def add_argparse_arguments(parser: ArgumentParser) -> ArgumentParser:
+        """Add component editor specific arguments to the parser."""
         parser.add_argument(
             "--skip-component-editor",
             action="store_true",
@@ -386,6 +475,64 @@ class ComponentEditorWindowBase(BaseWindow):
         )
         return parser
 
+    @classmethod
+    def create_for_testing(
+        cls,
+        version: str = "test",
+        local_filesystem: Optional[LocalFilesystem] = None,
+        data_model: Optional[ComponentDataModel] = None,
+        root_tk: Optional[tk.Tk] = None,  # pylint: disable=unused-argument # noqa: ARG003
+    ) -> "ComponentEditorWindowBase":
+        """
+        Factory method for creating instances suitable for testing.
+
+        This method provides a convenient way to create instances with minimal
+        dependencies for testing purposes.
+
+        Args:
+            version: Application version string
+            local_filesystem: Optional filesystem interface
+            data_model: Optional pre-configured data model
+            root_tk: Optional parent Tk window
+
+        Returns:
+            ComponentEditorWindowBase instance configured for testing
+
+        """
+        # Import MagicMock at the method level to avoid import issues
+        from unittest.mock import MagicMock  # pylint: disable=import-outside-toplevel
+
+        if local_filesystem is None:
+            # Create a minimal mock filesystem for testing
+            local_filesystem = MagicMock(spec=LocalFilesystem)
+            local_filesystem.vehicle_dir = "test_vehicle"
+            local_filesystem.doc_dict = {}
+            local_filesystem.load_vehicle_components_json_data.return_value = {}
+            local_filesystem.get_all_value_datatypes.return_value = {}
+
+        # Patch the BaseWindow initialization to avoid Tkinter dependencies
+        with patch("ardupilot_methodic_configurator.frontend_tkinter_component_editor_base.BaseWindow.__init__"):
+            instance = cls.__new__(cls)
+
+            # Manually initialize the essential attributes without calling BaseWindow.__init__
+            instance.local_filesystem = local_filesystem
+            instance.version = version
+
+            # Create mock UI elements to avoid Tkinter dependencies
+            instance.root = MagicMock()
+            instance.main_frame = MagicMock()
+
+            # Initialize the data model - allow injection for testing
+            instance.data_model = data_model or instance._create_data_model()  # noqa: SLF001
+
+            # Initialize UI elements dictionary for easier access and testing
+            instance.entry_widgets = {}
+            instance.scroll_frame = MagicMock()
+            instance.save_button = MagicMock()
+            instance.template_manager = MagicMock()
+
+            return instance
+
 
 if __name__ == "__main__":
     args = argument_parser()
@@ -395,5 +542,12 @@ if __name__ == "__main__":
     filesystem = LocalFilesystem(
         args.vehicle_dir, args.vehicle_type, "", args.allow_editing_template_files, args.save_component_to_system_templates
     )
-    app = ComponentEditorWindowBase(__version__, filesystem)
-    app.root.mainloop()
+    component_editor_window = ComponentEditorWindowBase(__version__, filesystem)
+
+    component_editor_window.populate_frames()
+    if args.skip_component_editor:
+        component_editor_window.root.after(10, component_editor_window.root.destroy)
+
+    # component_editor_window.validate_data()
+
+    component_editor_window.root.mainloop()

ardupilot_methodic_configurator/frontend_tkinter_parameter_editor_table.py

@@ -296,7 +296,7 @@ class ParameterEditorTable(ScrollFrame):  # pylint: disable=too-many-ancestors
         else:
             new_value_entry.configure(style="TEntry")
 
-    def __create_new_value_entry(  # pylint: disable=too-many-arguments, too-many-positional-arguments, too-many-locals
+    def __create_new_value_entry(  # pylint: disable=too-many-arguments,too-many-positional-arguments,too-many-locals,too-many-statements # noqa: PLR0915
         self,
         param_name: str,
         param: Par,

ardupilot_methodic_configurator/frontend_tkinter_show.py

@@ -28,6 +28,16 @@ def show_error_message(title: str, message: str) -> None:
     root.destroy()
 
 
+def show_warning_message(title: str, message: str) -> None:
+    root = tk.Tk()
+    # Set the theme to 'alt'
+    style = ttk.Style()
+    style.theme_use("alt")
+    root.withdraw()  # Hide the main window
+    messagebox.showwarning(title, message)
+    root.destroy()
+
+
 def show_no_param_files_error(_dirname: str) -> None:
     error_message = _(
         "No intermediate parameter files found in the selected '{_dirname}' vehicle directory.\n"