samschott / maestral

Open-source Dropbox client for macOS and Linux
https://maestral.app
MIT License
3.12k stars 64 forks source link

[sync] don't patch file events to track moves #885

Closed samschott closed 1 year ago

samschott commented 1 year ago

Use a dictionary to track linked events instead. This is safer since it does not meddle with classes that we don't own.

codecov[bot] commented 1 year ago

Codecov Report

Merging #885 (af977cd) into main (b752d10) will increase coverage by 0.00%. The diff coverage is 91.30%.

@@           Coverage Diff           @@
##             main     #885   +/-   ##
=======================================
  Coverage   78.59%   78.60%           
=======================================
  Files          42       42           
  Lines        7546     7507   -39     
=======================================
- Hits         5931     5901   -30     
+ Misses       1615     1606    -9     
Flag Coverage Δ
pytest 78.60% <91.30%> (+<0.01%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/maestral/utils/integration.py 69.23% <50.00%> (ø)
src/maestral/sync.py 84.51% <100.00%> (-0.18%) :arrow_down:
src/maestral/utils/path.py 91.51% <100.00%> (+2.92%) :arrow_up:

... and 1 file with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more