fix: use os.utime in mtime test for Windows compatibility
This commit is contained in:
+2
-3
@@ -1,7 +1,6 @@
|
|||||||
import os
|
import os
|
||||||
import shutil
|
import shutil
|
||||||
import tempfile
|
import tempfile
|
||||||
import time
|
|
||||||
from pathlib import Path
|
from pathlib import Path
|
||||||
|
|
||||||
import chromadb
|
import chromadb
|
||||||
@@ -240,10 +239,10 @@ def test_file_already_mined_check_mtime():
|
|||||||
# Already mined (mtime matches)
|
# Already mined (mtime matches)
|
||||||
assert file_already_mined(col, test_file, check_mtime=True) is True
|
assert file_already_mined(col, test_file, check_mtime=True) is True
|
||||||
|
|
||||||
# Modify file so mtime changes
|
# Modify file and force a different mtime (Windows has low mtime resolution)
|
||||||
time.sleep(0.1)
|
|
||||||
with open(test_file, "w") as f:
|
with open(test_file, "w") as f:
|
||||||
f.write("modified content")
|
f.write("modified content")
|
||||||
|
os.utime(test_file, (mtime + 10, mtime + 10))
|
||||||
|
|
||||||
# Still mined without mtime check
|
# Still mined without mtime check
|
||||||
assert file_already_mined(col, test_file) is True
|
assert file_already_mined(col, test_file) is True
|
||||||
|
|||||||
Reference in New Issue
Block a user