Skip to content

fix(redis): Use wait strategy instead of deprecated decorator#914

Open
mschmitzer wants to merge 2 commits intotestcontainers:mainfrom
solute:redis-wait-strategy
Open

fix(redis): Use wait strategy instead of deprecated decorator#914
mschmitzer wants to merge 2 commits intotestcontainers:mainfrom
solute:redis-wait-strategy

Conversation

@mschmitzer
Copy link
Copy Markdown
Contributor

Another part of fixing #874 (cf. #899).

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.52%. Comparing base (f608df9) to head (a9d5bc9).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #914      +/-   ##
==========================================
- Coverage   79.78%   79.52%   -0.26%     
==========================================
  Files          14       14              
  Lines        1182     1182              
  Branches      184      184              
==========================================
- Hits          943      940       -3     
- Misses        197      201       +4     
+ Partials       42       41       -1     

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

@alesdakshanin
Copy link
Copy Markdown

Hey there! Is this PR going to get merged? Maybe any help needed to get things moving?

@alexanderankin
Copy link
Copy Markdown
Member

please allow maintainer push to your branch or replace the content of your branch with this branch contents

https://github.com/testcontainers/testcontainers-python/pull/new/redis-wait-strategy-rebase - that should resolve the issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔧 fix 👀 intend to merge this or some version of it 🛠️ needs more work Need to invest more time, can be a rebase or code updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants