[timestamps] Move method validate to constructor & update test

pyonfx/timestamps.py

@@ -316,7 +316,16 @@ def __init__(
                 )
             self.last_frame_time = last_frame_time
 
-            Timestamps.validate(timestamps)
+            # Validate the timestamps
+            if len(timestamps) <= 1:
+                raise ValueError("There must be at least 2 timestamps.")
+
+            if any(timestamps[i] > timestamps[i + 1] for i in range(len(timestamps) - 1)):
+                raise ValueError("Timestamps must be in non-decreasing order.")
+
+            if timestamps.count(timestamps[0]) == len(timestamps):
+                raise ValueError("Timestamps must not be all identical.")
+
             self.timestamps = timestamps
 
             if normalize:
@@ -516,22 +525,6 @@ def get_timestamps(packets) -> Tuple[Fraction, Fraction, List[int]]:
             last_frame_time=last_frame_time,
         )
 
-    @staticmethod
-    def validate(timestamps: List[int]) -> None:
-        """Verify that the provided timestamps are valid, raising ValueError in case they are not.
-
-        Parameters:
-            timestamps (list of int): A list of [timestamps](https://en.wikipedia.org/wiki/Timestamp) encoded as integers.
-        """
-        if len(timestamps) <= 1:
-            raise ValueError("There must be at least 2 timestamps.")
-
-        if any(timestamps[i] > timestamps[i + 1] for i in range(len(timestamps) - 1)):
-            raise ValueError("Timestamps must be in non-decreasing order.")
-
-        if timestamps.count(timestamps[0]) == len(timestamps):
-            raise ValueError("Timestamps must not be all identical.")
-
     @staticmethod
     def normalize(
         timestamps: List[int], last_frame_time: Fraction

tests/test_timestamps.py

@@ -285,15 +285,15 @@ def test_from_video_file_invalid_path():
 
 def test_validate():
     with pytest.raises(ValueError) as exc_info:
-        Timestamps.validate([0])
+        Timestamps(RoundingMethod.FLOOR, [0], last_frame_time=Fraction(0))
     assert str(exc_info.value) == "There must be at least 2 timestamps."
 
     with pytest.raises(ValueError) as exc_info:
-        Timestamps.validate([0, 42, 20])
+        Timestamps(RoundingMethod.FLOOR, [0, 42, 20], last_frame_time=Fraction(0))
     assert str(exc_info.value) == "Timestamps must be in non-decreasing order."
 
     with pytest.raises(ValueError) as exc_info:
-        Timestamps.validate([20, 20])
+        Timestamps(RoundingMethod.FLOOR, [20, 20], last_frame_time=Fraction(0))
     assert str(exc_info.value) == "Timestamps must not be all identical."