Enhance main app test: add group creation, renaming, and object removal

Author: PierreRaybaut

cdl/core/gui/panel/base.py

@@ -649,16 +649,30 @@ def new_group(self) -> None:
         if ok:
             self.add_group(group_name)
 
-    def rename_group(self) -> None:
-        """Rename a group"""
-        # Open a message box to enter the group name
-        group = self.objview.get_sel_groups()[0]
-        group_name, ok = QW.QInputDialog.getText(
-            self, _("Rename group"), _("Group name:"), QW.QLineEdit.Normal, group.title
-        )
-        if ok:
-            group.title = group_name
-            self.objview.update_item(group.uuid)
+    def rename_group(self, new_name: str | None = None) -> None:
+        """Rename a group
+
+        Args:
+            new_name: new group name. Defaults to None (ask user).
+        """
+        sel_groups = self.objview.get_sel_groups()
+        if not sel_groups or len(sel_groups) > 1:
+            # Won't happen in the application, but could happen in tests or using the
+            # API directly
+            raise ValueError("Select one group to rename")
+        group = sel_groups[0]
+        if new_name is None:
+            new_name, ok = QW.QInputDialog.getText(
+                self,
+                _("Rename group"),
+                _("Group name:"),
+                QW.QLineEdit.Normal,
+                group.title,
+            )
+            if not ok:
+                return
+        group.title = new_name
+        self.objview.update_item(group.uuid)
 
     @abc.abstractmethod
     def get_newparam_from_current(

cdl/tests/backbone/main_app_test.py

@@ -31,8 +31,17 @@ def test_main_app():
         except ValueError:
             pass
 
-        # Add signals to signal panel
         panel = win.signalpanel
+
+        # Create new groups
+        panel.add_group("Group 1")
+        panel.add_group("Group 2")
+        # Rename group
+        panel.objview.select_groups([2])
+        panel.rename_group("Group xxx")
+        panel.remove_object(force=True)
+
+        # Add signals to signal panel
         sig1 = create_paracetamol_signal(500)
         panel.add_object(sig1)
         panel.processor.compute_derivative()