more pylint

Author: aw-was-here

tinytag/tests/test_all.py

@@ -14,7 +14,7 @@
 from platform import python_implementation, system
 from sys import stdout
 from unittest import skipIf, TestCase
-from unittest.mock import patch, Mock
+from unittest.mock import patch
 
 from tinytag import ParseError, TinyTagException, UnsupportedFormatError
 from tinytag import Images, OtherFields, TinyTag
@@ -1791,6 +1791,7 @@ def test_mp4_xmp_invalid_track(self) -> None:
 
     def test_mp4_parse_xmp_metadata_empty(self) -> None:
         # Test _parse_xmp_metadata with empty/None input
+        # pylint: disable=protected-access
         result = _MP4._parse_xmp_metadata(b'')
         self.assertEqual(result, {})
         result = _MP4._parse_xmp_metadata(None)
@@ -1799,15 +1800,16 @@ def test_mp4_parse_xmp_metadata_empty(self) -> None:
     def test_mp4_parse_xmp_metadata_exception(self) -> None:
         # Test _parse_xmp_metadata with data that causes exception
         # Passing wrong type (str instead of bytes) triggers exception
+        # pylint: disable=protected-access
         result = _MP4._parse_xmp_metadata('not bytes')  # type: ignore
         # Should return empty dict on exception
         self.assertEqual(result, {})
 
-    def test_mp4_xmp_set_field_exception(self) -> None:
+    @staticmethod
+    def test_mp4_xmp_set_field_exception() -> None:
         # Test exception handler in _apply_xmp_metadata_fallback
         # when _set_field raises an exception
-        tag = TinyTag.get(
-            os.path.join(SAMPLE_FOLDER, 'mp4_xmp_fallback.m4a'))
+        # pylint: disable=protected-access
         # Create a new MP4 instance to test
         sample_path = os.path.join(SAMPLE_FOLDER, 'mp4_xmp_fallback.m4a')
         with open(sample_path, 'rb') as f: