Add type annotations to controllers/yleareena.py

mypy.ini

@@ -22,7 +22,6 @@ warn_unreachable = true
 exclude = (?x)(
     ^pychromecast/generated/ # The protobuf autogenerated files are not annotated
     | ^pychromecast/controllers/plex\.py$
-    | ^pychromecast/controllers/yleareena\.py$
     | ^pychromecast/controllers/youtube\.py$
     | ^pychromecast/quick_play\.py$
   )

pychromecast/controllers/yleareena.py

@@ -2,6 +2,9 @@
 Controller to interface with the Yle Areena app namespace.
 """
 
+from typing import Any
+
+from . import CallbackType
 from .media import BaseMediaPlayer, STREAM_TYPE_BUFFERED, TYPE_LOAD, MESSAGE_TYPE
 from ..config import APP_YLEAREENA
 from ..error import PyChromecastError
@@ -11,19 +14,20 @@
 class YleAreenaController(BaseMediaPlayer):
     """Controller to interact with Yle Areena app namespace."""
 
-    def __init__(self):
+    def __init__(self) -> None:
         super().__init__(supporting_app_id=APP_YLEAREENA)
 
     def play_areena_media(  # pylint: disable=too-many-locals
         self,
-        kaltura_id,
-        audio_language="",
-        text_language="off",
-        current_time=0,
-        autoplay=True,
-        stream_type=STREAM_TYPE_BUFFERED,
-        callback_function=None,
-    ):
+        kaltura_id: str,
+        *,
+        audio_language: str = "",
+        text_language: str = "off",
+        current_time: float = 0,
+        autoplay: bool = True,
+        stream_type: str = STREAM_TYPE_BUFFERED,
+        callback_function: CallbackType | None = None,
+    ) -> None:
         """
         Play media with the entry id "kaltura_id".
         This value can be found by loading a page on Areena, e.g. https://areena.yle.fi/1-50097921
@@ -54,15 +58,23 @@ def play_areena_media(  # pylint: disable=too-many-locals
         self.send_message(msg, inc_session_id=True, callback_function=callback_function)
 
     # pylint: disable-next=arguments-differ
-    def quick_play(self, media_id=None, audio_lang="", text_lang="off", **kwargs):
+    def quick_play(
+        self,
+        media_id: str | None = None,
+        audio_lang: str = "",
+        text_lang: str = "off",
+        **kwargs: Any,
+    ) -> None:
         """Quick Play"""
+        if media_id is None:
+            raise ValueError("media_id must be specified")
         response_handler = WaitResponse(10)
         self.play_areena_media(
             media_id,
             audio_language=audio_lang,
             text_language=text_lang,
             **kwargs,
-            callback_function=response_handler.callback
+            callback_function=response_handler.callback,
         )
         request_completed = response_handler.wait_response()