Skip to content

Commit 4aaeb54

Browse files
committed
fix bugs: address bugs caused by outdated test code
1 parent 85611c8 commit 4aaeb54

File tree

3 files changed

+10
-5
lines changed

3 files changed

+10
-5
lines changed

src/memos/mem_scheduler/base_scheduler.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,6 @@ def __init__(self, config: BaseSchedulerConfig):
137137
self.dispatcher = SchedulerDispatcher(
138138
config=self.config,
139139
memos_message_queue=self.memos_message_queue,
140-
use_redis_queue=self.use_redis_queue,
141140
max_workers=self.thread_pool_max_workers,
142141
enable_parallel_dispatch=self.enable_parallel_dispatch,
143142
status_tracker=self.status_tracker,

src/memos/mem_scheduler/task_schedule_modules/dispatcher.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,6 @@ def __init__(
4343
self,
4444
max_workers: int = 30,
4545
memos_message_queue: ScheduleTaskQueue | None = None,
46-
use_redis_queue: bool | None = None,
4746
enable_parallel_dispatch: bool = True,
4847
config=None,
4948
status_tracker: TaskStatusTracker | None = None,
@@ -56,8 +55,12 @@ def __init__(
5655
# Main dispatcher thread pool
5756
self.max_workers = max_workers
5857

59-
self.memos_message_queue = memos_message_queue.memos_message_queue
60-
self.use_redis_queue = use_redis_queue
58+
# Accept either a ScheduleTaskQueue wrapper or a concrete queue instance
59+
self.memos_message_queue = (
60+
memos_message_queue.memos_message_queue
61+
if hasattr(memos_message_queue, "memos_message_queue")
62+
else memos_message_queue
63+
)
6164

6265
# Get multi-task timeout from config
6366
self.multi_task_running_timeout = (

tests/mem_scheduler/test_dispatcher.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,10 @@ def test_dispatch_serial(self):
157157
"""Test dispatching messages in serial mode."""
158158
# Create a new dispatcher with parallel dispatch disabled
159159
serial_dispatcher = SchedulerDispatcher(
160-
max_workers=2, enable_parallel_dispatch=False, metrics=MagicMock()
160+
max_workers=2,
161+
memos_message_queue=self.dispatcher.memos_message_queue,
162+
enable_parallel_dispatch=False,
163+
metrics=MagicMock(),
161164
)
162165

163166
# Create fresh mock handlers for this test

0 commit comments

Comments
 (0)