z4kn4fein commented 2 years ago

Describe the purpose of your pull request

When the cache write is failed, further write attempts were skipped because the latest ETag was cached in memory.

Related issues (only if applicable)

12

Requirement checklist (only if applicable)

[x] I have covered the applied changes with automated tests.
[x] I have executed the full automated test set against my changes.
[x] I have validated my changes against all supported platform versions.
[x] I have read and accepted the contribution agreement.

codecov[bot] commented 2 years ago

Codecov Report

Merging #13 (76e30a6) into master (64892bb) will decrease coverage by 0.09%. The diff coverage is n/a.

:exclamation: Current head 76e30a6 differs from pull request most recent head b9f7fd2. Consider uploading reports for the commit b9f7fd2 to get more accurate results

@@             Coverage Diff              @@
##             master      #13      +/-   ##
============================================
- Coverage     80.23%   80.14%   -0.10%     
  Complexity      285      285              
============================================
  Files            26       26              
  Lines           835      831       -4     
  Branches        107      106       -1     
============================================
- Hits            670      666       -4     
  Misses          122      122              
  Partials         43       43

Impacted Files	Coverage Δ
src/main/java/com/configcat/ConfigJsonCache.java	`84.61% <ø> (-1.10%)`	:arrow_down:
src/main/java/com/configcat/ConfigModels.java	`100.00% <ø> (ø)`

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 64892bb...b9f7fd2. Read the comment docs.