kontent-ai / management-sdk-net

Kontent.ai Management .NET SDK
https://www.nuget.org/packages/Kontent.Ai.Management
MIT License
7 stars 30 forks source link

fix retry policy #213

Closed gormal closed 1 year ago

gormal commented 1 year ago

Motivation

Fixes #211 Improve overall default retry policy. It only worked for responses from MAPI, but it did not take into account that an exception can be thrown.

Checklist

How to test

Unit tests added.

codecov-commenter commented 1 year ago

Codecov Report

Merging #213 (6daea80) into master (76b66e4) will decrease coverage by 0.07%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #213      +/-   ##
==========================================
- Coverage   81.06%   80.99%   -0.07%     
==========================================
  Files         231      231              
  Lines        2640     2631       -9     
  Branches      297      296       -1     
==========================================
- Hits         2140     2131       -9     
  Misses        386      386              
  Partials      114      114              
Impacted Files Coverage Δ
...nt.Ai.Management/Exceptions/ManagementException.cs 100.00% <100.00%> (ø)
...agement/Modules/HttpClient/ManagementHttpClient.cs 77.55% <100.00%> (-0.45%) :arrow_down:
...esiliencePolicy/DefaultResiliencePolicyProvider.cs 100.00% <100.00%> (ø)

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