Clean up some PluginManager method names and references

Author: strohganoff

streamdeck/manager.py

@@ -73,7 +73,7 @@ def __init__(
         self._event_listener_manager = EventListenerManager()
         self._event_adapter = EventAdapter()
 
-    def _ensure_action_has_valid_events(self, action: SupportsEventHandlers) -> None:
+    def _ensure_catalog_has_valid_events(self, action: SupportsEventHandlers) -> None:
         """Ensure that the event handler catalog's registered events are valid.
 
         Args:
@@ -85,14 +85,14 @@ def _ensure_action_has_valid_events(self, action: SupportsEventHandlers) -> None
                 logger.error(msg)
                 raise KeyError(msg)
 
-    def register_handler(self, action: ActionBase) -> None:
+    def register_action(self, action: ActionBase) -> None:
         """Register an action with the PluginManager, and configure its logger.
 
         Args:
             action (Action): The action to register.
         """
         # First, validate that the action's registered events are valid.
-        self._ensure_action_has_valid_events(action)
+        self._ensure_catalog_has_valid_events(action)
 
         # Next, configure a logger for the action, giving it the last part of its uuid as name (if it has one).
         action_component_name = action.uuid.split(".")[-1] if isinstance(action, Action) else "global"

tests/plugin_manager/test_command_sender_binding.py

@@ -108,7 +108,7 @@ def test_run_manager_events_handled_with_correct_params(
     mock_event_handler_without_cmd_sender: Mock = create_autospec(create_event_handler(include_command_sender_param=False), spec_set=True)
     action.on("keyDown")(mock_event_handler_without_cmd_sender)
 
-    plugin_manager.register_handler(action)
+    plugin_manager.register_action(action)
 
     # Run the PluginManager and let it process the fake event messages generated by the mocked EventListenerManager.
     # Since the EventListenerManager.event_stream() method is mocked to return a finite list of fake event messages, it will stop after yielding all of them rather than running indefinitely.

tests/plugin_manager/test_plugin_manager.py

@@ -46,7 +46,7 @@ def test_plugin_manager_register_action(plugin_manager: PluginManager) -> None:
     assert len(plugin_manager._handlers_registry._plugin_actions) == 0
 
     action = Action("my-fake-action-uuid")
-    plugin_manager.register_handler(action)
+    plugin_manager.register_action(action)
 
     assert len(plugin_manager._handlers_registry._plugin_actions) == 1
     assert plugin_manager._handlers_registry._plugin_actions[0] == action