fix(sdk): Client::sync_once defaults to reuse previous token

[dragonfly1033]

Fixes #1835.

Introduces a new SyncToken enum for the SyncSettings::token field.
The enum has 3 variants: ReusePrevious (default), NoToken, Specific(String).

Some tests were changed to use the old default (NoToken).

Signed-off-by: Shrey Patel [email protected]

[codspeed-hq]

CodSpeed Performance Report

Merging #5522 will not alter performance

_{Comparing dragonfly1033/sync-once (f69b2cb) with main (1ffc014)}

Summary

✅ 31 untouched benchmarks

[codecov]

Codecov Report

❌ Patch coverage is 58.33333% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.57%. Comparing base (6c944a9) to head (f69b2cb).
⚠️ Report is 38 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
crates/matrix-sdk/src/config/sync.rs	54.54%	5 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5522   +/-   ##
=======================================
  Coverage   88.57%   88.57%           
=======================================
  Files         339      339           
  Lines       93627    93635    +8     
  Branches    93627    93635    +8     
=======================================
+ Hits        82927    82937   +10     
+ Misses       6565     6562    -3     
- Partials     4135     4136    +1     

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

[bnjbvr]

Looks great, thanks for fixing this! Can you add an entry in the matrix_sdk's CHANGELOG.md, please? (in the refactor section, with an annotation that it's a breaking change)

[bnjbvr]

Just missing a CHANGELOG.md entry, and we'll be good to go 🥳

[bnjbvr]

Thank you 🥳