Use explicit string annotation in cast expressions

Updated all instances of `cast` to use string annotations, ensuring consistency and aligning with newer type hinting practices. This enhances readability and avoids potential issues with forward references in type checking.

Author: GitBib

pymkv/MKVFile.py

@@ -722,7 +722,7 @@ def split_size(
             Raised if if `size` is not a bitmath object or an integer.
         """
         if getattr(size, "__module__", None) == bitmath.__name__:
-            size = cast(bitmath.Bitmath, size).bytes
+            size = cast("bitmath.Bitmath", size).bytes
         elif not isinstance(size, int):
             msg = "size is not a bitmath object or integer"
             raise TypeError(msg)
@@ -1187,7 +1187,7 @@ def _flatten(item: T | Iterable[T | Iterable[T]]) -> Iterable[T]:
                 for subitem in item:
                     yield from _flatten(subitem)
             else:
-                yield cast(T, item)
+                yield cast("T", item)
 
         return list(_flatten(item))
 

pymkv/Timestamp.py

@@ -250,7 +250,7 @@ def hh(self) -> int:
         Returns:
             int: The hours value.
         """
-        return cast(int, self._hh)
+        return cast("int", self._hh)
 
     @hh.setter
     def hh(self, value: int) -> None:
@@ -270,7 +270,7 @@ def mm(self) -> int:
         Returns:
             int: The minutes value.
         """
-        return cast(int, self._mm)
+        return cast("int", self._mm)
 
     @mm.setter
     def mm(self, value: int) -> None:
@@ -290,7 +290,7 @@ def ss(self) -> int:
         Returns:
             int: The seconds value.
         """
-        return cast(int, self._ss)
+        return cast("int", self._ss)
 
     @ss.setter
     def ss(self, value: int) -> None:
@@ -310,7 +310,7 @@ def nn(self) -> int:
         Returns:
             int: The nanoseconds value.
         """
-        return cast(int, self._nn)
+        return cast("int", self._nn)
 
     @nn.setter
     def nn(self, value: int) -> None:
@@ -417,7 +417,7 @@ def splitting_timestamp(self, timestamp: str) -> None:
         assert timestamp_match is not None
         timestamp_groups = timestamp_match.groups()
 
-        timestamp_lst = [cast(str, timestamp_groups[i]) for i in (1, 2, 3, 4)]
+        timestamp_lst = [cast("str", timestamp_groups[i]) for i in (1, 2, 3, 4)]
         timestamp_clean = []
         for ts in timestamp_lst:
             if ts is None:

pymkv/utils.py

@@ -97,6 +97,6 @@ def wrapper(
                 setattr(self, info_attr, fetch_func(*func_params, **fetch_kwargs))
             return method(self, *args, **kwargs)
 
-        return cast(Callable[..., T], wrapper)
+        return cast("Callable[..., T]", wrapper)
 
     return decorator

tests/test_actions_track.py

@@ -182,7 +182,7 @@ def test_move_track_backward_and_mux(
 
 def test_get_track(get_path_test_file: Path) -> None:
     mkv = MKVFile(get_path_test_file)
-    track = cast(MKVTrack, mkv.get_track(1))
+    track = cast("MKVTrack", mkv.get_track(1))
 
     assert track.track_type == "audio"
 
@@ -204,7 +204,7 @@ def test_added_lang_in_track_and_mux_file(
 ) -> None:
     mkv = MKVFile(get_path_test_file)
     output_file = get_base_path / "file-test.mkv"
-    track = cast(MKVTrack, mkv.get_track(1))
+    track = cast("MKVTrack", mkv.get_track(1))
     track.language_ietf = "TEST"
     with pytest.raises(ValueError):  # noqa: PT011
         mkv.mux(output_file)

tests/test_iso639_2.py

@@ -12,7 +12,7 @@ def test_is_iso639_2_false() -> None:
 
 
 def test_is_iso639_2_non_str_input() -> None:
-    assert is_iso639_2(cast(str, 123)) is False
+    assert is_iso639_2(cast("str", 123)) is False
 
 
 def test_is_iso639_2_empty_string() -> None:

tests/test_timestamp.py

@@ -76,7 +76,7 @@ def test_getitem() -> None:
 
 def test_invalid_input() -> None:
     with pytest.raises(TypeError):
-        Timestamp(cast(str, []))  # Invalid type
+        Timestamp(cast("str", []))  # Invalid type
 
     with pytest.raises(ValueError):  # noqa: PT011
         Timestamp("invalid_timestamp")
@@ -90,4 +90,4 @@ def test_ts_property() -> None:
     assert ts.ts == "02:30:00"
 
     with pytest.raises(TypeError):
-        ts.ts = cast(str, [])
+        ts.ts = cast("str", [])

tests/test_utils_prepare_mkvtoolnix_path.py

@@ -54,4 +54,4 @@ def test_prepare_mkvmerge_path_with_pathlike() -> None:
 
 def test_prepare_mkvmerge_path_with_invalid_argument() -> None:
     with pytest.raises(ValueError):  # noqa: PT011
-        utils.prepare_mkvtoolnix_path(cast(str, 123))
+        utils.prepare_mkvtoolnix_path(cast("str", 123))