Skip to content

Commit c43ad07

Browse files
author
Andrei Neagu
committed
use in memory redis
1 parent b079ac5 commit c43ad07

File tree

2 files changed

+10
-13
lines changed

2 files changed

+10
-13
lines changed

packages/service-library/tests/deferred_tasks/test__base_deferred_handler.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@
3434

3535
pytest_simcore_core_services_selection = [
3636
"rabbit",
37-
"redis",
3837
]
3938

4039

@@ -51,10 +50,10 @@ class MockKeys(StrAutoEnum):
5150

5251
@pytest.fixture
5352
async def redis_client_sdk(
54-
redis_service: RedisSettings,
53+
use_in_memory_redis: RedisSettings,
5554
) -> AsyncIterable[RedisClientSDK]:
5655
sdk = RedisClientSDK(
57-
redis_service.build_redis_dsn(RedisDatabase.DEFERRED_TASKS),
56+
use_in_memory_redis.build_redis_dsn(RedisDatabase.DEFERRED_TASKS),
5857
decode_responses=False,
5958
client_name="pytest",
6059
)

packages/service-library/tests/deferred_tasks/test_deferred_tasks.py

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@
3232

3333
pytest_simcore_core_services_selection = [
3434
"rabbit",
35-
"redis",
3635
]
3736
pytest_simcore_ops_services_selection = [
3837
"redis-commander",
@@ -136,12 +135,12 @@ def __init__(
136135
self,
137136
remote_process: _RemoteProcess,
138137
rabbit_service: RabbitSettings,
139-
redis_service: RedisSettings,
138+
redis_settings: RedisSettings,
140139
max_workers: int,
141140
) -> None:
142141
self.remote_process = remote_process
143142
self.rabbit_service = rabbit_service
144-
self.redis_service = redis_service
143+
self.redis_settings = redis_settings
145144
self.max_workers = max_workers
146145

147146
async def __aenter__(self) -> "_RemoteProcessLifecycleManager":
@@ -168,7 +167,7 @@ async def start(self) -> None:
168167
),
169168
"redis": json_dumps(
170169
model_dump_with_secrets(
171-
self.redis_service,
170+
self.redis_settings,
172171
show_secrets=True,
173172
**_get_serialization_options(),
174173
)
@@ -256,7 +255,7 @@ async def _sleep_in_interval(lower: NonNegativeFloat, upper: NonNegativeFloat) -
256255
async def test_workflow_with_outages_in_process_running_deferred_manager(
257256
get_remote_process: Callable[[], Awaitable[_RemoteProcess]],
258257
rabbit_service: RabbitSettings,
259-
redis_service: RedisSettings,
258+
use_in_memory_redis: RedisSettings,
260259
remote_processes: int,
261260
max_workers: int,
262261
deferred_tasks_to_start: NonNegativeInt,
@@ -269,7 +268,7 @@ async def test_workflow_with_outages_in_process_running_deferred_manager(
269268
_RemoteProcessLifecycleManager(
270269
await get_remote_process(),
271270
rabbit_service,
272-
redis_service,
271+
use_in_memory_redis,
273272
max_workers,
274273
)
275274
)
@@ -333,8 +332,7 @@ async def rabbit_client(
333332

334333

335334
class ClientWithPingProtocol(Protocol):
336-
async def ping(self) -> bool:
337-
...
335+
async def ping(self) -> bool: ...
338336

339337

340338
class ServiceManager:
@@ -404,7 +402,7 @@ async def test_workflow_with_third_party_services_outages(
404402
rabbit_client: RabbitMQClient,
405403
get_remote_process: Callable[[], Awaitable[_RemoteProcess]],
406404
rabbit_service: RabbitSettings,
407-
redis_service: RedisSettings,
405+
use_in_memory_redis: RedisSettings,
408406
max_workers: int,
409407
deferred_tasks_to_start: int,
410408
service: str,
@@ -416,7 +414,7 @@ async def test_workflow_with_third_party_services_outages(
416414
async with _RemoteProcessLifecycleManager(
417415
await get_remote_process(),
418416
rabbit_service,
419-
redis_service,
417+
use_in_memory_redis,
420418
max_workers,
421419
) as manager:
422420
# start all in parallel

0 commit comments

Comments
 (0)