Change how default cache works for EventsClient

- Previous implementation assumed that all events were going to be utilised, meaning default cache had no TTL even if events hadn't been started/added.
- Default cache for `EventsClient` is now the same as `Client`, with relevant cache instances only being cleared once an event has been startd.

Author: mathsman5133

coc/client.py

@@ -1333,38 +1333,6 @@ def dispatch(self, event_name: str, *args, **kwargs):
         for event in self.extra_events.get(event_name, []):
             asyncio.ensure_future(self._run_event(event_name, event, *args, **kwargs), loop=self.loop)
 
-    def default_cache(self, client):
-        """The default cache for :class:`EventsClient`.
-
-        Because the whole 'events' notion hinges on comparing old values in cache to new values,
-        **all cache objects have no expiry** and 1024 max size.
-
-        .. note::
-
-            If you choose to alter the cache to include an expiry, you will miss events due to integration of
-            cache and events. The same applies to max size, although potentially missing events to trade for
-            memory control may be important for some.
-
-        """
-        cache_search_clans._is_clan = True
-        cache_war_logs._is_clan = True
-
-        cache_current_wars._is_war = True
-        cache_clan_wars._is_war = True
-        cache_league_groups._is_war = True
-        cache_league_wars._is_war = True
-        cache_war_clans._is_war = True
-        cache_war_players._is_war = True
-
-        cache_search_players._is_player = True
-
-        cache_locations._is_static = True
-        cache_leagues._is_static = True
-        cache_seasons._is_static = True
-
-        for cache in self._cache_lookup.values():
-            cache.clear(1024, None)
-
     def event(self, fctn, name=None):
         """A decorator or regular function that registers an event.
 
@@ -1614,17 +1582,19 @@ def start_updates(self, event_type='all'):
 
         """
         lookup = {
-            'clan': self._clan_update_event,
-            'player': self._player_update_event,
-            'war': self._war_update_event
+            'clan': [self._clan_update_event, [cache_current_wars]],
+            'player': [self._player_update_event, [cache_search_players]],
+            'war': [self._war_update_event, [cache_current_wars, cache_clan_wars, cache_league_wars]]
         }
         if event_type == 'all':
             events = lookup.values()
         else:
             events = [lookup[event_type]]
 
         for e in events:
-            e.set()
+            e[0].set()
+            for c in e[1]:
+                c.clear(1024, None)
 
     def stop_updates(self, event_type='all'):
         """Stops an, or all, events.
@@ -1649,17 +1619,19 @@ def stop_updates(self, event_type='all'):
         """
 
         lookup = {
-            'clan': self._clan_update_event,
-            'player': self._player_update_event,
-            'war': self._war_update_event
+            'clan': [self._clan_update_event, [cache_current_wars]],
+            'player': [self._player_update_event, [cache_search_players]],
+            'war': [self._war_update_event, [cache_current_wars, cache_clan_wars, cache_league_wars]]
         }
         if event_type == 'all':
             events = lookup.values()
         else:
             events = [lookup[event_type]]
 
         for e in events:
-            e.clear()
+            e[0].clear()
+            for c in e[1]:
+                c.clear(1024, None)
 
     def _dispatch_batch_updates(self, key_name):
         keys = cache_events.cache.keys()