From c0ddedce540496b826d614bf18929eacd4575007 Mon Sep 17 00:00:00 2001 From: thePR0M3TH3AN <53631862+PR0M3TH3AN@users.noreply.github.com> Date: Sun, 29 Jun 2025 18:42:51 -0400 Subject: [PATCH] Relax lock timing assertion --- src/tests/test_file_lock.py | 5 ++++- src/utils/file_lock.py | 1 + 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/src/tests/test_file_lock.py b/src/tests/test_file_lock.py index 6f797af..4b542e0 100644 --- a/src/tests/test_file_lock.py +++ b/src/tests/test_file_lock.py @@ -2,6 +2,8 @@ import multiprocessing as mp import time from pathlib import Path +import pytest + from utils.file_lock import exclusive_lock @@ -34,4 +36,5 @@ def test_exclusive_lock_blocks_until_released(tmp_path: Path): p1.join() p2.join() - assert wait_time.value >= 0.9 + # CI runners can be jittery; allow some slack around the 1s lock hold time + assert wait_time.value == pytest.approx(1.0, rel=0.2) diff --git a/src/utils/file_lock.py b/src/utils/file_lock.py index e838f33..9c71b62 100644 --- a/src/utils/file_lock.py +++ b/src/utils/file_lock.py @@ -1,4 +1,5 @@ """File-based locking utilities using portalocker for cross-platform support.""" + from contextlib import contextmanager from typing import Generator, Optional from pathlib import Path