Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.48%. Comparing base (79ec40a) to head (e37740a).
Report is 1 commits behind head on main.

@@            Coverage Diff             @@
##             main    #8903      +/-   ##
==========================================
+ Coverage   81.47%   81.48%   +0.01%     
==========================================
  Files         150      150              
  Lines       18681    18686       +5     
  Branches     3192     3193       +1     
==========================================
+ Hits        15221    15227       +6     
  Misses       3168     3168              
+ Partials      292      291       -1     

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

thanks for the proposed fix with unit test. should we try to add extra integration tests for this as well? it's not mandatory though

You were reading my mind @auvipy .
Although IMHO I'd say it is mandatory.

But let's see first if the CI passed successfully with the current tests.

@Nusnus I've added the integration test, but now one of the checks is not successful, I don't think it has anything to do with me, it's getting connection refused from the Redis.

@Nusnus I've added the integration test, but now one of the checks is not successful, I don't think it has anything to do with me, it's getting connection refused from the Redis.

Indeed, unrelated.

Merged!

Good work, thank you.