Fix tests

src/can_explorer/views.py

@@ -39,6 +39,10 @@ def _add_row(self) -> None:
         self._row_values.append(row)
 
     def _sync_rows(self) -> None:
+        """
+        Pair the current row_keys with the row_values.
+        Automatically hide's any rows not being used.
+        """
         while len(self._row_keys) > len(self._row_values):
             self._add_row()
 

tests/test_controller.py

@@ -42,3 +42,10 @@ def test_controller_cannot_start_without_stopping(app, tag, controller):
     assert controller.is_active()
     with pytest.raises(RuntimeError):
         controller.start()
+
+
+def test_controller_must_be_inactive_to_apply_settings(app, tag, controller):
+    controller.start()
+    assert controller.is_active()
+    with pytest.raises(RuntimeError):
+        controller.settings_apply_button_callback()

tests/test_gui.py

@@ -81,7 +81,7 @@ def virtual_gui(request, process):
         save_screenshot(request)
 
 
-def test_gui_launch_basic(app, controller, tag):
+def test_gui_launch(app, controller, tag):
     assert dpg.is_dearpygui_running()
 
     # Check if the main window was created
@@ -103,21 +103,6 @@ def test_gui_launch_basic(app, controller, tag):
     assert dpg.get_item_label(tag.main_button) == "Start"
 
 
-def test_gui_must_be_inactive_to_apply_settings(app, tag, controller):
-    pytest.skip("Todo")
-    # app.run()
-
-    # dpg.set_value(tag.settings_interface, "virtual")
-    # controller.settings_apply_button_callback()
-
-    # controller.start_stop_button_callback()
-    # assert controller.is_active()
-
-    # dpg.set_value(tag.settings_interface, None)
-    # with pytest.raises(RuntimeError):
-    #     controller.settings_apply_button_callback()
-
-
 def test_gui_visualizes_traffic(virtual_gui):
     pyautogui.click(pyautogui.locate(Gui.Button.START))
     sleep(1)