Removed unused ParamSpec and added a consistency check in the tests.

Author: semohr

beets/logging.py

@@ -60,9 +60,6 @@
     T = TypeVar("T")
 
 
-P = ParamSpec("P")
-
-
 def _logsafe(val: T) -> str | T:
     """Coerce `bytes` to `str` to avoid crashes solely due to logging.
 

test/test_logging.py

@@ -62,6 +62,7 @@ def test_str_format_logging(
         with caplog.at_level(level, logger="test_logger"):
             logger.log(level, msg, *args, **kwargs)
 
+        assert caplog.records, "No log records were captured"
         assert str(caplog.records[0].msg) == expected