Skip to content

refactor(tests): migrate legacy redis_client imports to autobot_shared (#2047) #2171

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mrveiss merged 1 commit into from
Mar 24, 2026
Merged

refactor(tests): migrate legacy redis_client imports to autobot_shared (#2047) #2171

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion autobot-backend/async_baseline.performance_test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@
from constants.network_constants import NetworkConstants, ServiceURLs

# Import canonical Redis client pattern
from utils.redis_client import get_redis_client
from autobot_shared.redis_client import get_redis_client

# Configure logging
logging.basicConfig(
Expand Down
3 changes: 2 additions & 1 deletion autobot-backend/monitoring/redis_prometheus_metrics_test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,8 @@

import pytest
from monitoring.prometheus_metrics import get_metrics_manager
from utils.redis_client import RedisConnectionManager

from autobot_shared.redis_client import RedisConnectionManager


@pytest.fixture
Expand Down
3 changes: 2 additions & 1 deletion autobot-backend/services/redis_listeners.e2e_test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,8 @@

import redis
from config import config as global_config_manager
from utils.redis_client import get_redis_client

from autobot_shared.redis_client import get_redis_client


def test_worker_capabilities():
Expand Down
4 changes: 2 additions & 2 deletions autobot-backend/utils/redis_consolidation_test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
sys.path.insert(0, str(Path(__file__).parent.parent))


from utils.redis_client import (
from autobot_shared.redis_client import (
ConnectionState,
ManagerStats,
PoolStatistics,
Expand Down Expand Up @@ -255,7 +255,7 @@ def test_circuit_breaker_logic(self):

def test_module_imports():
"""Test all expected exports are available"""
from utils import redis_client
from autobot_shared import redis_client

# Check all main exports
assert hasattr(redis_client, "get_redis_client")
Expand Down
5 changes: 3 additions & 2 deletions autobot-backend/utils/redis_thread_safety_test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,8 @@
from unittest.mock import MagicMock, Mock, patch

import pytest
from utils.redis_client import RedisConnectionManager

from autobot_shared.redis_client import RedisConnectionManager


class TestPoolStatisticsThreadSafety:
Expand Down Expand Up @@ -275,7 +276,7 @@ async def test_cleanup_doesnt_affect_active_connections(self):

# Use freezegun or manual datetime mocking
# Mock datetime.now() to return consistent value
import utils.redis_client as redis_client_module
import autobot_shared.redis_client as redis_client_module

original_datetime = redis_client_module.datetime

Expand Down
Loading