From 845ba8d6e8049905fa2e660b00b86dd60c578c28 Mon Sep 17 00:00:00 2001 From: Daniel-dev22 <47092714+Daniel-dev22@users.noreply.github.com> Date: Sat, 20 Jun 2026 17:05:23 -0400 Subject: [PATCH] test: drop test_timestamps_are_memory_bounded It only asserted that deque(maxlen=) caps length, which is stdlib behavior rather than something this change needs to verify. Co-Authored-By: Claude Opus 4.8 (1M context) --- frigate/test/test_builtin.py | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/frigate/test/test_builtin.py b/frigate/test/test_builtin.py index 4c9db6140d..7cf47de5c6 100644 --- a/frigate/test/test_builtin.py +++ b/frigate/test/test_builtin.py @@ -36,18 +36,6 @@ class TestEventsPerSecond(unittest.TestCase): clock[0] += 100.0 self.assertEqual(eps.eps(), 0.0) - def test_timestamps_are_memory_bounded(self) -> None: - # last_n_seconds large enough that nothing expires by time, so the - # maxlen cap is what bounds retention - eps = EventsPerSecond(max_events=5, last_n_seconds=10_000) - clock = [0.0] - with patch("frigate.util.builtin.time.monotonic", side_effect=lambda: clock[0]): - eps.start() - for _ in range(1000): - clock[0] += 0.001 - eps.update() - self.assertLessEqual(len(eps._timestamps), 5) - if __name__ == "__main__": unittest.main()