[WhitWaldo]

Description

Fixing inconsistent actor state bug since it marked changes clean before persistence succeeded. This means that retries after failed or cancelled saves silently skipped writes.

This issue does not impact Dapr.Actors.Next.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #1616

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

• Code compiles correctly
• Created/updated tests
• Extended the documentation

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.39%. Comparing base (008b782) to head (9540bc5).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1946      +/-   ##
==========================================
+ Coverage   68.33%   68.39%   +0.05%     
==========================================
  Files         376      376              
  Lines       10876    10877       +1     
  Branches     1274     1275       +1     
==========================================
+ Hits         7432     7439       +7     
+ Misses       3100     3097       -3     
+ Partials      344      341       -3     

Flag	Coverage Δ
net10.0	68.37% <100.00%> (+0.05%)	⬆️

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

☔ View full report in Codecov by Harness.
📢 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.