Skip to content

Add redis sentinel integration tests#755

Merged
drivebyer merged 2 commits intoOT-CONTAINER-KIT:masterfrom
MathieuCesbron:add_redis_sentinel_integration_tests
Jan 21, 2024
Merged

Add redis sentinel integration tests#755
drivebyer merged 2 commits intoOT-CONTAINER-KIT:masterfrom
MathieuCesbron:add_redis_sentinel_integration_tests

Conversation

@MathieuCesbron
Copy link
Copy Markdown
Contributor

Description

Add redis sentinel integration tests (last one).

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • Tests have been added/modified and all tests pass.
  • Functionality/bugs have been confirmed to be unchanged or fixed.
  • I have performed a self-review of my own code.
  • Documentation has been updated or added where necessary.

Signed-off-by: Mathieu Cesbron <mathieu.cesbron@protonmail.com>
Signed-off-by: Mathieu Cesbron <mathieu.cesbron@protonmail.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (f9938bc) 32.11% compared to head (7dda6ee) 32.79%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #755      +/-   ##
==========================================
+ Coverage   32.11%   32.79%   +0.68%     
==========================================
  Files          19       19              
  Lines        3232     3232              
==========================================
+ Hits         1038     1060      +22     
+ Misses       2140     2113      -27     
- Partials       54       59       +5     

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

@MathieuCesbron MathieuCesbron marked this pull request as ready for review January 20, 2024 13:54
@drivebyer drivebyer enabled auto-merge (squash) January 21, 2024 02:52
@drivebyer drivebyer merged commit 2e1ba97 into OT-CONTAINER-KIT:master Jan 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants