9201ef759e
Harness Compat / harness compat (push) Failing after 0s
CI / test on 3.12 (standard) (push) Has been cancelled
CI / test on 3.13 (standard) (push) Has been cancelled
CI / test on 3.14 (standard) (push) Has been cancelled
CI / test on 3.10 (all-extras) (push) Has been cancelled
CI / test on 3.11 (all-extras) (push) Has been cancelled
CI / test on 3.12 (all-extras) (push) Has been cancelled
CI / test on 3.14 (pydantic-ai-slim) (push) Has been cancelled
CI / test on 3.10 (pydantic-evals) (push) Has been cancelled
CI / test on 3.11 (pydantic-evals) (push) Has been cancelled
CI / test on 3.12 (pydantic-evals) (push) Has been cancelled
CI / deploy-docs-preview (push) Has been cancelled
CI / build release artifacts (push) Has been cancelled
CI / publish to PyPI (push) Has been cancelled
CI / Send tweet (push) Has been cancelled
CI / lint (push) Has been cancelled
CI / mypy (push) Has been cancelled
CI / docs (push) Has been cancelled
CI / test on 3.10 (standard) (push) Has been cancelled
CI / test on 3.11 (standard) (push) Has been cancelled
CI / test on 3.13 (all-extras) (push) Has been cancelled
CI / test on 3.14 (all-extras) (push) Has been cancelled
CI / test on 3.10 (pydantic-ai-slim) (push) Has been cancelled
CI / test on 3.11 (pydantic-ai-slim) (push) Has been cancelled
CI / test on 3.12 (pydantic-ai-slim) (push) Has been cancelled
CI / test on 3.13 (pydantic-ai-slim) (push) Has been cancelled
CI / test on 3.13 (pydantic-evals) (push) Has been cancelled
CI / test on 3.14 (pydantic-evals) (push) Has been cancelled
CI / test on 3.10 (lowest-versions) (push) Has been cancelled
CI / test on 3.11 (lowest-versions) (push) Has been cancelled
CI / test on 3.12 (lowest-versions) (push) Has been cancelled
CI / test on 3.13 (lowest-versions) (push) Has been cancelled
CI / test on 3.14 (lowest-versions) (push) Has been cancelled
CI / test examples on 3.11 (push) Has been cancelled
CI / test examples on 3.12 (push) Has been cancelled
CI / test examples on 3.13 (push) Has been cancelled
CI / test examples on 3.14 (push) Has been cancelled
CI / coverage (push) Has been cancelled
CI / check (push) Has been cancelled
CI / deploy-docs (push) Has been cancelled
487 lines
16 KiB
Python
487 lines
16 KiB
Python
"""Tests for graph execution internals and edge cases."""
|
|
|
|
from __future__ import annotations
|
|
|
|
from dataclasses import dataclass, field
|
|
|
|
import pytest
|
|
from anyio import create_task_group
|
|
|
|
from pydantic_graph import GraphBuilder, StepContext
|
|
from pydantic_graph.graph_builder import GraphTask, _GraphIterator # pyright: ignore[reportPrivateUsage]
|
|
from pydantic_graph.id_types import NodeRunID, TaskID
|
|
from pydantic_graph.join import ReduceFirstValue, reduce_list_append, reduce_list_extend
|
|
|
|
pytestmark = pytest.mark.anyio
|
|
|
|
|
|
@dataclass
|
|
class ExecutionState:
|
|
log: list[str] = field(default_factory=list[str])
|
|
counter: int = 0
|
|
|
|
|
|
async def test_map_to_end_node_cancels_pending():
|
|
"""Test that mapping directly to end_node cancels pending tasks"""
|
|
import asyncio
|
|
|
|
g = GraphBuilder(state_type=ExecutionState, output_type=int)
|
|
|
|
@g.step
|
|
async def generate(ctx: StepContext[ExecutionState, None, None]) -> list[int]:
|
|
return [1, 2, 3, 4, 5]
|
|
|
|
@g.step
|
|
async def early_exit(ctx: StepContext[ExecutionState, None, int]) -> int:
|
|
# First item returns immediately
|
|
if ctx.inputs == 1:
|
|
return ctx.inputs
|
|
# Others would take longer
|
|
await asyncio.sleep(1)
|
|
return ctx.inputs # pragma: no cover
|
|
|
|
g.add(
|
|
g.edge_from(g.start_node).to(generate),
|
|
g.edge_from(generate).map().to(early_exit),
|
|
g.edge_from(early_exit).to(g.end_node),
|
|
)
|
|
|
|
graph = g.build()
|
|
result = await graph.run(state=ExecutionState())
|
|
# Should complete quickly with the first result
|
|
assert result in [1, 2, 3, 4, 5]
|
|
|
|
|
|
async def test_map_non_iterable_raises_error():
|
|
"""Test that mapping a non-iterable raises RuntimeError."""
|
|
g = GraphBuilder(state_type=ExecutionState, output_type=int)
|
|
|
|
@g.step
|
|
async def return_non_iterable(ctx: StepContext[ExecutionState, None, None]) -> int:
|
|
return 42 # Not iterable!
|
|
|
|
@g.step
|
|
async def process_item(ctx: StepContext[ExecutionState, None, int]) -> int:
|
|
return ctx.inputs # pragma: no cover
|
|
|
|
g.add(
|
|
g.edge_from(g.start_node).to(return_non_iterable),
|
|
g.edge_from(return_non_iterable).map().to(process_item), # type: ignore # purposely have a type error here
|
|
g.edge_from(process_item).to(g.end_node),
|
|
)
|
|
|
|
graph = g.build()
|
|
with pytest.raises(RuntimeError, match='Cannot map non-iterable value'):
|
|
await graph.run(state=ExecutionState())
|
|
|
|
|
|
async def test_broadcast_marker_handling():
|
|
"""Test that BroadcastMarker is handled in paths"""
|
|
g = GraphBuilder(state_type=ExecutionState, output_type=list[str])
|
|
|
|
@g.step
|
|
async def source(ctx: StepContext[ExecutionState, None, None]) -> str:
|
|
return 'data'
|
|
|
|
@g.step
|
|
async def branch_a(ctx: StepContext[ExecutionState, None, str]) -> str:
|
|
return f'{ctx.inputs}-A'
|
|
|
|
@g.step
|
|
async def branch_b(ctx: StepContext[ExecutionState, None, str]) -> str:
|
|
return f'{ctx.inputs}-B'
|
|
|
|
collect = g.join(reduce_list_append, initial_factory=list[str])
|
|
|
|
g.add(
|
|
g.edge_from(g.start_node).to(source),
|
|
# Use multiple .to() destinations to create broadcast
|
|
g.edge_from(source).to(branch_a, branch_b),
|
|
g.edge_from(branch_a, branch_b).to(collect),
|
|
g.edge_from(collect).to(g.end_node),
|
|
)
|
|
|
|
graph = g.build()
|
|
result = await graph.run(state=ExecutionState())
|
|
assert sorted(result) == ['data-A', 'data-B']
|
|
|
|
|
|
async def test_nested_joins_with_different_fork_stacks():
|
|
"""Test nested joins with different fork stack depths"""
|
|
g = GraphBuilder(state_type=ExecutionState, output_type=list[int])
|
|
|
|
@g.step
|
|
async def generate_outer(ctx: StepContext[ExecutionState, None, None]) -> list[int]:
|
|
return [1, 2]
|
|
|
|
@g.step
|
|
async def generate_inner(ctx: StepContext[ExecutionState, None, int]) -> list[int]:
|
|
return [ctx.inputs * 10, ctx.inputs * 20]
|
|
|
|
@g.step
|
|
async def process(ctx: StepContext[ExecutionState, None, int]) -> int:
|
|
return ctx.inputs
|
|
|
|
final_join = g.join(reduce_list_append, initial_factory=list[int])
|
|
|
|
g.add(
|
|
g.edge_from(g.start_node).to(generate_outer),
|
|
g.edge_from(generate_outer).map().to(generate_inner),
|
|
g.edge_from(generate_inner).map().to(process),
|
|
g.edge_from(process).to(final_join),
|
|
g.edge_from(final_join).to(g.end_node),
|
|
)
|
|
|
|
graph = g.build()
|
|
result = await graph.run(state=ExecutionState())
|
|
# Should have 4 total elements (2 outer * 2 inner each)
|
|
assert len(result) == 4
|
|
assert sorted(result) == [10, 20, 20, 40]
|
|
|
|
|
|
async def test_reduce_first_value_task_cancellation():
|
|
"""Test that ReduceFirstValue properly cancels sibling tasks"""
|
|
import asyncio
|
|
|
|
g = GraphBuilder(state_type=ExecutionState, output_type=str)
|
|
|
|
@g.step
|
|
async def generate(ctx: StepContext[ExecutionState, None, None]) -> list[int]:
|
|
return [1, 2, 3, 4, 5]
|
|
|
|
@g.step
|
|
async def slow_process(ctx: StepContext[ExecutionState, None, int]) -> str:
|
|
if ctx.inputs == 1:
|
|
# First one completes quickly
|
|
await asyncio.sleep(0.01)
|
|
else:
|
|
# Others take longer (should be cancelled)
|
|
await asyncio.sleep(10)
|
|
ctx.state.log.append(f'completed-{ctx.inputs}')
|
|
return f'result-{ctx.inputs}'
|
|
|
|
first_join = g.join(ReduceFirstValue[str](), initial='')
|
|
|
|
g.add(
|
|
g.edge_from(g.start_node).to(generate),
|
|
g.edge_from(generate).map().to(slow_process),
|
|
g.edge_from(slow_process).to(first_join),
|
|
g.edge_from(first_join).to(g.end_node),
|
|
)
|
|
|
|
graph = g.build()
|
|
state = ExecutionState()
|
|
result = await graph.run(state=state)
|
|
|
|
# Should get a result
|
|
assert result is not None and result.startswith('result-')
|
|
# Not all tasks should have completed due to cancellation
|
|
assert len(state.log) < 5
|
|
|
|
|
|
async def test_empty_map_handling():
|
|
"""Test handling of mapping an empty iterable.
|
|
|
|
Note: Empty maps with joins can be tricky and may need the downstream_join_id hint.
|
|
This test documents expected behavior.
|
|
"""
|
|
# Skipping this test as empty maps need special handling with downstream_join_id
|
|
# The actual line coverage is achieved through other tests
|
|
pass
|
|
|
|
|
|
async def test_complex_fork_stack_with_multiple_levels():
|
|
"""Test complex scenarios with multiple fork levels"""
|
|
g = GraphBuilder(state_type=ExecutionState, output_type=list[int])
|
|
|
|
@g.step
|
|
async def level1(ctx: StepContext[ExecutionState, None, None]) -> list[int]:
|
|
return [1, 2]
|
|
|
|
@g.step
|
|
async def level2(ctx: StepContext[ExecutionState, None, int]) -> list[int]:
|
|
return [ctx.inputs * 10, ctx.inputs * 10 + 1]
|
|
|
|
@g.step
|
|
async def level3(ctx: StepContext[ExecutionState, None, int]) -> int:
|
|
ctx.state.log.append(f'processing-{ctx.inputs}')
|
|
return ctx.inputs
|
|
|
|
collect = g.join(reduce_list_append, initial_factory=list[int])
|
|
|
|
g.add(
|
|
g.edge_from(g.start_node).to(level1),
|
|
g.edge_from(level1).map().to(level2),
|
|
g.edge_from(level2).map().to(level3),
|
|
g.edge_from(level3).to(collect),
|
|
g.edge_from(collect).to(g.end_node),
|
|
)
|
|
|
|
graph = g.build()
|
|
state = ExecutionState()
|
|
result = await graph.run(state=state)
|
|
|
|
# Should process 4 items total (2 from level1 * 2 from each level2)
|
|
assert len(result) == 4
|
|
assert sorted(result) == [10, 11, 20, 21]
|
|
assert len(state.log) == 4
|
|
|
|
|
|
async def test_broadcast_with_immediate_join():
|
|
"""Test broadcast that immediately joins."""
|
|
g = GraphBuilder(state_type=ExecutionState, output_type=list[int])
|
|
|
|
@g.step
|
|
async def source(ctx: StepContext[ExecutionState, None, None]) -> int:
|
|
return 10
|
|
|
|
@g.step
|
|
async def path_a(ctx: StepContext[ExecutionState, None, int]) -> int:
|
|
return ctx.inputs * 2
|
|
|
|
@g.step
|
|
async def path_b(ctx: StepContext[ExecutionState, None, int]) -> int:
|
|
return ctx.inputs * 3
|
|
|
|
@g.step
|
|
async def path_c(ctx: StepContext[ExecutionState, None, int]) -> int:
|
|
return ctx.inputs * 4
|
|
|
|
collect = g.join(reduce_list_append, initial_factory=list[int])
|
|
|
|
g.add(
|
|
g.edge_from(g.start_node).to(source),
|
|
# Multiple .to() destinations creates a broadcast
|
|
g.edge_from(source).to(path_a, path_b, path_c),
|
|
g.edge_from(path_a, path_b, path_c).to(collect),
|
|
g.edge_from(collect).to(g.end_node),
|
|
)
|
|
|
|
graph = g.build()
|
|
result = await graph.run(state=ExecutionState())
|
|
assert sorted(result) == [20, 30, 40]
|
|
|
|
|
|
async def test_implicit_broadcast_with_immediate_join():
|
|
"""Test broadcast that immediately joins by just manually adding multiple edges from a single node."""
|
|
g = GraphBuilder(state_type=ExecutionState, output_type=list[int])
|
|
|
|
@g.step
|
|
async def source(ctx: StepContext[ExecutionState, None, None]) -> int:
|
|
return 10
|
|
|
|
@g.step
|
|
async def path_a(ctx: StepContext[ExecutionState, None, int]) -> int:
|
|
return ctx.inputs * 2
|
|
|
|
@g.step
|
|
async def path_b(ctx: StepContext[ExecutionState, None, int]) -> int:
|
|
return ctx.inputs * 3
|
|
|
|
@g.step
|
|
async def path_c(ctx: StepContext[ExecutionState, None, int]) -> int:
|
|
return ctx.inputs * 4
|
|
|
|
collect = g.join(reduce_list_append, initial_factory=list[int])
|
|
|
|
g.add(
|
|
g.edge_from(g.start_node).to(source),
|
|
# Multiple .to() destinations creates a broadcast
|
|
g.edge_from(source).to(path_a),
|
|
g.edge_from(source).to(path_b),
|
|
g.edge_from(source).to(path_c),
|
|
g.edge_from(path_a).to(collect),
|
|
g.edge_from(path_b).to(collect),
|
|
g.edge_from(path_c).to(collect),
|
|
g.edge_from(collect).to(g.end_node),
|
|
)
|
|
|
|
graph = g.build()
|
|
result = await graph.run(state=ExecutionState())
|
|
assert sorted(result) == [20, 30, 40]
|
|
|
|
|
|
async def test_mixed_sequential_and_parallel_execution():
|
|
"""Test graph with both sequential and parallel sections."""
|
|
g = GraphBuilder(state_type=ExecutionState, output_type=str)
|
|
|
|
@g.step
|
|
async def step1(ctx: StepContext[ExecutionState, None, None]) -> int:
|
|
ctx.state.log.append('step1')
|
|
return 5
|
|
|
|
@g.step
|
|
async def step2(ctx: StepContext[ExecutionState, None, int]) -> list[int]:
|
|
ctx.state.log.append('step2')
|
|
return [ctx.inputs * 10, ctx.inputs * 20]
|
|
|
|
@g.step
|
|
async def parallel_step(ctx: StepContext[ExecutionState, None, int]) -> int:
|
|
ctx.state.log.append(f'parallel-{ctx.inputs}')
|
|
return ctx.inputs + 1
|
|
|
|
@g.step
|
|
async def step3(ctx: StepContext[ExecutionState, None, list[int]]) -> str:
|
|
ctx.state.log.append('step3')
|
|
return f'Result: {sum(ctx.inputs)}'
|
|
|
|
collect = g.join(reduce_list_append, initial_factory=list[int])
|
|
|
|
g.add(
|
|
g.edge_from(g.start_node).to(step1),
|
|
g.edge_from(step1).to(step2),
|
|
g.edge_from(step2).map().to(parallel_step),
|
|
g.edge_from(parallel_step).to(collect),
|
|
g.edge_from(collect).to(step3),
|
|
g.edge_from(step3).to(g.end_node),
|
|
)
|
|
|
|
graph = g.build()
|
|
state = ExecutionState()
|
|
result = await graph.run(state=state)
|
|
|
|
assert 'step1' in state.log
|
|
assert 'step2' in state.log
|
|
assert 'parallel-50' in state.log
|
|
assert 'parallel-100' in state.log
|
|
assert 'step3' in state.log
|
|
assert result == 'Result: 152' # (50+1) + (100+1) = 152
|
|
|
|
|
|
async def test_multiple_sequential_joins():
|
|
g = GraphBuilder(output_type=list[int])
|
|
|
|
@g.step
|
|
async def source(ctx: StepContext[None, None, None]) -> int:
|
|
return 10
|
|
|
|
@g.step
|
|
async def add_one(ctx: StepContext[None, None, int]) -> list[int]:
|
|
return [ctx.inputs + 1]
|
|
|
|
@g.step
|
|
async def add_two(ctx: StepContext[None, None, int]) -> list[int]:
|
|
return [ctx.inputs + 2]
|
|
|
|
@g.step
|
|
async def add_three(ctx: StepContext[None, None, int]) -> list[int]:
|
|
return [ctx.inputs + 3]
|
|
|
|
collect = g.join(reduce_list_extend, initial_factory=list[int], parent_fork_id='source_fork', node_id='collect')
|
|
mediator = g.join(reduce_list_extend, initial_factory=list[int], node_id='mediator')
|
|
|
|
# Broadcasting: send the value from source to all three steps
|
|
g.add(
|
|
g.edge_from(g.start_node).to(source),
|
|
g.edge_from(source).to(add_one, add_two, add_three, fork_id='source_fork'),
|
|
g.edge_from(add_one, add_two).to(mediator),
|
|
g.edge_from(mediator).to(collect),
|
|
g.edge_from(add_three).to(collect),
|
|
g.edge_from(collect).to(g.end_node),
|
|
)
|
|
|
|
graph = g.build()
|
|
result = await graph.run()
|
|
assert sorted(result) == [11, 12, 13]
|
|
|
|
|
|
async def test_early_termination_from_nested_generator():
|
|
"""Test that a generator wrapping an iteration can be terminated early."""
|
|
g = GraphBuilder()
|
|
g.add_edge(g.start_node, g.end_node)
|
|
graph = g.build()
|
|
|
|
async def stream_graph():
|
|
async with graph.iter() as run:
|
|
async for node in run: # pragma: no branch
|
|
yield node
|
|
|
|
gen = stream_graph()
|
|
async for _ in gen: # pragma: no branch
|
|
break
|
|
await gen.aclose()
|
|
|
|
|
|
async def test_tracked_task_send_after_sender_closed_is_swallowed():
|
|
"""A node task whose `send` lands after the run's stream sender is closed must not crash the run.
|
|
|
|
When a run is cancelled before reaching its end node, teardown closes
|
|
`iter_stream_sender` while node tasks may still be in flight. An in-flight
|
|
`send` to a closed sender raises `anyio.ClosedResourceError` (a closed
|
|
*receiver* would instead raise `BrokenResourceError`); both are benign during
|
|
teardown and must be swallowed rather than escape into the task group.
|
|
|
|
The real-world trigger is a teardown race that can't be reproduced reliably
|
|
through the public API, so this drives `_run_tracked_task` directly: closing
|
|
the sender up front guarantees the deterministic equivalent of the race (the
|
|
task finishing its `send` against an already-closed sender). Regression test
|
|
for #6146; without the fix this leaks `ClosedResourceError` out of the group.
|
|
"""
|
|
g = GraphBuilder(output_type=int)
|
|
|
|
ran = False
|
|
|
|
@g.step
|
|
async def produce(ctx: StepContext[None, None, None]) -> int:
|
|
nonlocal ran
|
|
ran = True
|
|
return 42
|
|
|
|
g.add(
|
|
g.edge_from(g.start_node).to(produce),
|
|
g.edge_from(produce).to(g.end_node),
|
|
)
|
|
graph = g.build()
|
|
|
|
next_task_id_counter = 0
|
|
|
|
def next_task_id() -> TaskID:
|
|
nonlocal next_task_id_counter
|
|
next_task_id_counter += 1
|
|
return TaskID(f'task:{next_task_id_counter}')
|
|
|
|
def next_node_run_id() -> NodeRunID: # pragma: no cover
|
|
# Not reached for a single leaf step, but required to construct the iterator.
|
|
return NodeRunID('node-run:0')
|
|
|
|
async with create_task_group() as task_group:
|
|
iterator = _GraphIterator(graph, None, None, task_group, next_node_run_id, next_task_id)
|
|
# Simulate run teardown closing the streams while a node task is still in flight.
|
|
# `send_nowait` checks the sender's own closed flag before the receiver's, so the
|
|
# in-flight `send` raises `ClosedResourceError` (not `BrokenResourceError`) here.
|
|
iterator.iter_stream_sender.close()
|
|
iterator.iter_stream_receiver.close()
|
|
task = GraphTask(produce.id, None, (), next_task_id())
|
|
task_group.start_soon(iterator._run_tracked_task, task) # pyright: ignore[reportPrivateUsage]
|
|
|
|
# Reaching here means the task group exited cleanly; the `ClosedResourceError`
|
|
# from the closed-sender `send` was swallowed instead of propagating.
|
|
assert ran, 'the node task should have run and attempted to send its result'
|
|
|
|
|
|
def test_run_sync():
|
|
"""`Graph.run_sync` mirrors `Graph.run` from synchronous code."""
|
|
g = GraphBuilder(input_type=int, output_type=int)
|
|
|
|
@g.step
|
|
async def increment(ctx: StepContext[None, None, int]) -> int:
|
|
return ctx.inputs + 1
|
|
|
|
@g.step
|
|
async def double(ctx: StepContext[None, None, int]) -> int:
|
|
return ctx.inputs * 2
|
|
|
|
g.add(
|
|
g.edge_from(g.start_node).to(increment),
|
|
g.edge_from(increment).to(double),
|
|
g.edge_from(double).to(g.end_node),
|
|
)
|
|
|
|
graph = g.build()
|
|
# First call infers the name from the calling frame.
|
|
assert graph.run_sync(inputs=3) == 8
|
|
assert graph.name == 'graph'
|
|
# Second call skips name inference because the name is already set.
|
|
assert graph.run_sync(inputs=4) == 10
|
|
assert graph.name == 'graph'
|