Skip to content

Conversation

@tony
Copy link
Member

@tony tony commented Dec 30, 2025

Summary

  • Add async tests for AsyncFileLock and async_atomic_init
  • Add pytest-asyncio dependency and configuration

Depends on #501.

Test plan

  • 8 async tests for AsyncFileLock (context manager, acquire/release, reentrant, timeout, etc.)
  • 4 async tests for async_atomic_init (first init, already done, sync fn, concurrent)

tony added 3 commits December 30, 2025 09:22
why: Required for async test support.
what:
- Add pytest-asyncio to dev and testing dependency groups
why: Test async file locking functionality.
what:
- Add TestAsyncFileLock with 8 async tests
- Add TestAsyncAtomicInit with 4 async tests
- Tests cover context manager, acquire/release, timeout, concurrent access
@codecov
Copy link

codecov bot commented Dec 30, 2025

Codecov Report

❌ Patch coverage is 99.20635% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 57.23%. Comparing base (bbb80be) to head (77787d2).

Files with missing lines Patch % Lines
tests/_internal/test_file_lock_async.py 99.20% 1 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           pr/file-lock     #505      +/-   ##
================================================
+ Coverage         55.56%   57.23%   +1.66%     
================================================
  Files                40       41       +1     
  Lines              6117     6243     +126     
  Branches           1095     1095              
================================================
+ Hits               3399     3573     +174     
+ Misses             2202     2151      -51     
- Partials            516      519       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants