Skip to content

Commit 962f277

Browse files
committed
fix integration test for episodic memory
1 parent 98e0e87 commit 962f277

File tree

1 file changed

+24
-16
lines changed

1 file changed

+24
-16
lines changed

tests/test_integration/test_memory_reasoning.py

Lines changed: 24 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -280,12 +280,14 @@ def test_plan_records_to_memory(self, monkeypatch):
280280
plan = reasoning.plan(obs=obs)
281281

282282
assert isinstance(plan, Plan)
283-
assert memory.step_content["Observation"]["content"] == str(obs)
284-
assert memory.step_content["Plan"]["content"] == plan_content
285-
assert memory.step_content["Plan-Execution"]["content"] == str(plan)
286-
assert memory.step_content["Observation"]["importance"] == 3
287-
assert memory.step_content["Plan"]["importance"] == 3
288-
assert memory.step_content["Plan-Execution"]["importance"] == 3
283+
entries = list(memory.memory_entries)
284+
assert len(entries) == 3
285+
assert entries[0].content["Observation"]["content"] == str(obs)
286+
assert entries[1].content["Plan"]["content"] == plan_content
287+
assert entries[2].content["Plan-Execution"]["content"] == str(plan)
288+
assert entries[0].content["Observation"]["importance"] == 3
289+
assert entries[1].content["Plan"]["importance"] == 3
290+
assert entries[2].content["Plan-Execution"]["importance"] == 3
289291
assert memory.grade_event_importance.call_count == 3
290292

291293
def test_async_plan_works(self, monkeypatch):
@@ -301,12 +303,14 @@ def test_async_plan_works(self, monkeypatch):
301303
plan = asyncio.run(reasoning.aplan(obs=obs))
302304

303305
assert isinstance(plan, Plan)
304-
assert memory.step_content["Observation"]["content"] == str(obs)
305-
assert memory.step_content["Plan"]["content"] == plan_content
306-
assert memory.step_content["Plan-Execution"]["content"] == str(plan)
307-
assert memory.step_content["Observation"]["importance"] == 3
308-
assert memory.step_content["Plan"]["importance"] == 3
309-
assert memory.step_content["Plan-Execution"]["importance"] == 3
306+
entries = list(memory.memory_entries)
307+
assert len(entries) == 3
308+
assert entries[0].content["Observation"]["content"] == str(obs)
309+
assert entries[1].content["Plan"]["content"] == plan_content
310+
assert entries[2].content["Plan-Execution"]["content"] == str(plan)
311+
assert entries[0].content["Observation"]["importance"] == 3
312+
assert entries[1].content["Plan"]["importance"] == 3
313+
assert entries[2].content["Plan-Execution"]["importance"] == 3
310314
assert memory.agrade_event_importance.await_count == 3
311315

312316

@@ -677,8 +681,10 @@ def test_plan_records_to_memory(self, monkeypatch):
677681

678682
plan = reasoning.plan()
679683
assert isinstance(plan, Plan)
680-
assert memory.step_content["plan"]["content"] == plan_content
681-
assert memory.step_content["plan"]["importance"] == 3
684+
entries = list(memory.memory_entries)
685+
assert len(entries) == 1
686+
assert entries[0].content["plan"]["content"] == plan_content
687+
assert entries[0].content["plan"]["importance"] == 3
682688
assert memory.grade_event_importance.call_count == 1
683689
reasoning.execute_tool_call.assert_called_once_with(
684690
plan_content, selected_tools=None, ttl=1
@@ -699,8 +705,10 @@ def test_async_plan_works(self, monkeypatch):
699705

700706
plan = asyncio.run(reasoning.aplan())
701707
assert isinstance(plan, Plan)
702-
assert memory.step_content["plan"]["content"] == plan_content
703-
assert memory.step_content["plan"]["importance"] == 3
708+
entries = list(memory.memory_entries)
709+
assert len(entries) == 1
710+
assert entries[0].content["plan"]["content"] == plan_content
711+
assert entries[0].content["plan"]["importance"] == 3
704712
assert memory.grade_event_importance.call_count == 1
705713
reasoning.aexecute_tool_call.assert_awaited_once_with(
706714
plan_content, selected_tools=None, ttl=1

0 commit comments

Comments
 (0)