fix: race condition in image uploader#779

Merged

ptpt merged 16 commits intomainfrom

fix-race-condition-image-uploader

Aug 27, 2025

Commits on Aug 27, 2025

fix: race condition in image uploader and add tests
ptpt
committed
Add tests for ImageSequenceUploader. Follow the following principles:
ptpt
committed
I saw that you use multithreads in test_image_sequence_uploader_multithreading_with_cache_enabled. No need to do that because `sequence_uploader.upload_images` is using inside multithreading. We ca…
ptpt
committed
Using cached_time to determine if it's cached is probably not reliable. Try mock some internal implementation, e.g. if single_image_uploader.cache.get() is called and filehandle if filehandle is ca…
ptpt
committed
increase num_images
ptpt
committed
Ok, it looks like it is hard to tests with the current implementation. Let's make some changes in uploader.py
ptpt
committed
Ok now let's improve tests. As we exposed both ImageSequenceUploader.cache and SingleImageUploader.cache, which means you can update them after construction, and run assertions based on the cache i…
ptpt
committed
I don't see you are calling `sequence_uploader.upload_images` in test_image_sequence_uploader_cache_hits_second_run
ptpt
committed
simplify tests
ptpt
committed
enable upload_cache_path
ptpt
committed
refactor
ptpt
committed
update tests
ptpt
committed
rename
ptpt
committed
tests
ptpt
committed
Assert that results from the uploader.upload_images() in TestImageSequenceUploader has not errors
ptpt
committed
cache path
ptpt
committed