Skip to content

Commit fd8168a

Browse files
authored
fix: remove agent from swarm test to get more consistency out of it (#1946)
1 parent 80fdd94 commit fd8168a

File tree

1 file changed

+13
-15
lines changed

1 file changed

+13
-15
lines changed

tests_integ/test_multiagent_swarm.py

Lines changed: 13 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -88,11 +88,11 @@ def __init__(self):
8888
self.should_exit = True
8989

9090
def register_hooks(self, registry):
91-
registry.add_callback(BeforeNodeCallEvent, self.exit_before_analyst)
91+
registry.add_callback(BeforeNodeCallEvent, self.exit_before_writer)
9292

93-
def exit_before_analyst(self, event):
94-
if event.node_id == "analyst" and self.should_exit:
95-
raise SystemExit("Controlled exit before analyst")
93+
def exit_before_writer(self, event):
94+
if event.node_id == "writer" and self.should_exit:
95+
raise SystemExit("Controlled exit before writer")
9696

9797
return ExitHook()
9898

@@ -365,32 +365,30 @@ def test_swarm_resume_from_executing_state(tmpdir, exit_hook, verify_hook):
365365
# First execution - exit before second node
366366
session_manager = FileSessionManager(session_id=session_id, storage_dir=tmpdir)
367367
researcher = Agent(name="researcher", system_prompt="you are a researcher.")
368-
analyst = Agent(name="analyst", system_prompt="you are an analyst.")
369368
writer = Agent(name="writer", system_prompt="you are a writer.")
370369

371-
swarm = Swarm([researcher, analyst, writer], session_manager=session_manager, hooks=[exit_hook])
370+
swarm = Swarm([researcher, writer], session_manager=session_manager, hooks=[exit_hook])
372371

373372
try:
374-
swarm("write AI trends and calculate growth in 100 words")
373+
swarm("write AI trends in 100 words")
375374
except SystemExit as e:
376-
assert "Controlled exit before analyst" in str(e)
375+
assert "Controlled exit before writer" in str(e)
377376

378377
# Verify state was persisted with EXECUTING status and next node
379378
persisted_state = session_manager.read_multi_agent(session_id, swarm.id)
380379
assert persisted_state["status"] == "executing"
381380
assert len(persisted_state["node_history"]) == 1
382381
assert persisted_state["node_history"][0] == "researcher"
383-
assert persisted_state["next_nodes_to_execute"] == ["analyst"]
382+
assert persisted_state["next_nodes_to_execute"] == ["writer"]
384383

385384
exit_hook.should_exit = False
386385
researcher2 = Agent(name="researcher", system_prompt="you are a researcher.")
387-
analyst2 = Agent(name="analyst", system_prompt="you are an analyst.")
388386
writer2 = Agent(name="writer", system_prompt="you are a writer.")
389-
new_swarm = Swarm([researcher2, analyst2, writer2], session_manager=session_manager, hooks=[verify_hook])
390-
result = new_swarm("write AI trends and calculate growth in 100 words")
387+
new_swarm = Swarm([researcher2, writer2], session_manager=session_manager, hooks=[verify_hook])
388+
result = new_swarm("write AI trends in 100 words")
391389

392-
# Verify swarm behavior - should resume from analyst, not restart
390+
# Verify swarm behavior - should resume from writer, not restart
393391
assert result.status.value == "completed"
394-
assert verify_hook.first_node == "analyst"
392+
assert verify_hook.first_node == "writer"
395393
node_ids = [n.node_id for n in result.node_history]
396-
assert "analyst" in node_ids
394+
assert "writer" in node_ids

0 commit comments

Comments
 (0)