fix(opta): Fix Opta F24 timestamp parsing

Opta does not zero-pad milliseconds. Therefore, they were incorrectly parsed
by Python's default "%f" format code.

See also PySport#267

kloppy/infra/serializers/event/opta/deserializer.py

@@ -246,6 +246,11 @@
 
 
 def _parse_f24_datetime(dt_str: str) -> float:
+    def zero_pad_milliseconds(timestamp):
+        parts = timestamp.split(".")
+        return ".".join(parts[:-1] + ["{:03d}".format(int(parts[-1]))])
+
+    dt_str = zero_pad_milliseconds(dt_str)
     return (
         datetime.strptime(dt_str, "%Y-%m-%dT%H:%M:%S.%f")
         .replace(tzinfo=pytz.utc)

kloppy/tests/test_opta.py

@@ -56,7 +56,6 @@ def dataset(base_dir) -> EventDataset:
     return dataset
 
 
-@pytest.mark.xfail
 def test_parse_f24_datetime():
     """Test if the F24 datetime is correctly parsed"""
     # timestamps have millisecond precision