Common: Rename subscribe_all() -> subscribe_all_events()

monkey/common/event_queue/i_event_queue.py

@@ -10,7 +10,7 @@ class IEventQueue(ABC):
     """
 
     @abstractmethod
-    def subscribe_all(self, subscriber: Callable[[AbstractEvent], None]):
+    def subscribe_all_events(self, subscriber: Callable[[AbstractEvent], None]):
         """
         Subscribes a subscriber to all events
 

monkey/common/event_queue/pypubsub_event_queue.py

@@ -1,8 +1,9 @@
 from typing import Callable
 
-from common.events import AbstractEvent
 from pubsub.core import Publisher
 
+from common.events import AbstractEvent
+
 from .i_event_queue import IEventQueue
 
 INTERNAL_ALL_EVENT_TYPES_TOPIC = "internal_all_event_types"
@@ -12,7 +13,7 @@ class PyPubSubEventQueue(IEventQueue):
     def __init__(self, pypubsub_publisher: Publisher):
         self._pypubsub_publisher = pypubsub_publisher
 
-    def subscribe_all(self, subscriber: Callable[[AbstractEvent], None]):
+    def subscribe_all_events(self, subscriber: Callable[[AbstractEvent], None]):
         self._pypubsub_publisher.subscribe(
             listener=subscriber, topicName=INTERNAL_ALL_EVENT_TYPES_TOPIC
         )

monkey/tests/unit_tests/common/event_queue/test_pypubsub_event_queue.py

@@ -67,7 +67,7 @@ def test_subscribe_all():
     def subscriber(event, topic=pub.AUTO_TOPIC):
         subscriber_calls.append(topic.getName())
 
-    pypubsub_event_queue.subscribe_all(subscriber)
+    pypubsub_event_queue.subscribe_all_events(subscriber)
     pypubsub_event_queue.publish(EventType)
 
     assert len(subscriber_calls) == 1

vulture_allowlist.py

@@ -270,4 +270,4 @@ subscribe_type  # common\event_queue\pypubsub_event_queue.py
 subscribe_tag  # common\event_queue\pypubsub_event_queue.py
 publish  # common\event_queue\pypubsub_event_queue.py
 PyPubSubEventQueue  # common\event_queue\pypubsub_event_queue.py
-subscribe_all_event_types  # common\event_queue\pypubsub_event_queue.py
+subscribe_all_events  # common\event_queue\pypubsub_event_queue.py