Skip to content

Remove Redis from legacy API test client - #3815

Merged
anth-volk merged 2 commits into
masterfrom
fix/2112-remove-redis-test-fixture
Sep 4, 2026
Merged

Remove Redis from legacy API test client#3815
anth-volk merged 2 commits into
masterfrom
fix/2112-remove-redis-test-fixture

Conversation

@anth-volk

Copy link
Copy Markdown
Collaborator

Fixes #2112

Summary

  • replace the legacy rest_client fixture with a Redis-independent api_client fixture
  • remove Redis subprocess startup, connection checking, and the fixed startup delay
  • update the six affected legacy route-test modules to use the renamed fixture

Testing

  • uv run --extra dev ruff check on all changed Python files
  • uv run --extra dev pytest on the six affected legacy test modules: 41 passed

@codecov

codecov Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.18%. Comparing base (5e2071a) to head (ad7d866).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #3815       +/-   ##
===========================================
+ Coverage   46.26%   87.18%   +40.92%     
===========================================
  Files         135      150       +15     
  Lines        7963     8935      +972     
  Branches     1326     1542      +216     
===========================================
+ Hits         3684     7790     +4106     
+ Misses       3990      703     -3287     
- Partials      289      442      +153     

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

@anth-volk
anth-volk marked this pull request as ready for review September 4, 2026 23:48
@anth-volk
anth-volk merged commit ee07259 into master Sep 4, 2026
14 checks passed
@anth-volk
anth-volk deleted the fix/2112-remove-redis-test-fixture branch September 4, 2026 23:48
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.

Remove Redis dependency from legacy route tests that do not need it

1 participant