|
15 | 15 | """Unit tests for LlmAgent include_contents field behavior.""" |
16 | 16 |
|
17 | 17 | from google.adk.agents.llm_agent import LlmAgent |
| 18 | +from google.adk.agents.run_config import RunConfig |
18 | 19 | from google.adk.agents.sequential_agent import SequentialAgent |
19 | 20 | from google.genai import types |
20 | 21 | import pytest |
@@ -189,6 +190,153 @@ def simple_tool(message: str) -> dict: |
189 | 190 | assert len(mock_model.requests[0].config.tools) > 0 |
190 | 191 |
|
191 | 192 |
|
| 193 | +def test_model_input_context_is_sent_to_model_without_persisting_to_session(): |
| 194 | + mock_model = testing_utils.MockModel.create(responses=["Answer"]) |
| 195 | + agent = LlmAgent(name="test_agent", model=mock_model) |
| 196 | + runner = testing_utils.InMemoryRunner(agent) |
| 197 | + session = runner.session |
| 198 | + |
| 199 | + list( |
| 200 | + runner.runner.run( |
| 201 | + user_id=session.user_id, |
| 202 | + session_id=session.id, |
| 203 | + new_message=testing_utils.get_user_content("Question"), |
| 204 | + run_config=RunConfig( |
| 205 | + model_input_context=[ |
| 206 | + types.UserContent("Relevant context for this turn") |
| 207 | + ] |
| 208 | + ), |
| 209 | + ) |
| 210 | + ) |
| 211 | + |
| 212 | + assert testing_utils.simplify_contents(mock_model.requests[0].contents) == [ |
| 213 | + ("user", "Relevant context for this turn"), |
| 214 | + ("user", "Question"), |
| 215 | + ] |
| 216 | + assert testing_utils.simplify_events(runner.session.events) == [ |
| 217 | + ("user", "Question"), |
| 218 | + ("test_agent", "Answer"), |
| 219 | + ] |
| 220 | + |
| 221 | + |
| 222 | +def test_model_input_context_stays_before_user_message_after_tool_call(): |
| 223 | + def simple_tool(message: str) -> dict: |
| 224 | + return {"result": f"Tool processed: {message}"} |
| 225 | + |
| 226 | + mock_model = testing_utils.MockModel.create( |
| 227 | + responses=[ |
| 228 | + types.Part.from_function_call( |
| 229 | + name="simple_tool", args={"message": "payload"} |
| 230 | + ), |
| 231 | + "Answer", |
| 232 | + ] |
| 233 | + ) |
| 234 | + agent = LlmAgent(name="test_agent", model=mock_model, tools=[simple_tool]) |
| 235 | + runner = testing_utils.InMemoryRunner(agent) |
| 236 | + session = runner.session |
| 237 | + |
| 238 | + list( |
| 239 | + runner.runner.run( |
| 240 | + user_id=session.user_id, |
| 241 | + session_id=session.id, |
| 242 | + new_message=testing_utils.get_user_content("Question"), |
| 243 | + run_config=RunConfig( |
| 244 | + model_input_context=[ |
| 245 | + types.UserContent("Relevant context for this turn") |
| 246 | + ] |
| 247 | + ), |
| 248 | + ) |
| 249 | + ) |
| 250 | + |
| 251 | + assert testing_utils.simplify_contents(mock_model.requests[0].contents) == [ |
| 252 | + ("user", "Relevant context for this turn"), |
| 253 | + ("user", "Question"), |
| 254 | + ] |
| 255 | + assert testing_utils.simplify_contents(mock_model.requests[1].contents) == [ |
| 256 | + ("user", "Relevant context for this turn"), |
| 257 | + ("user", "Question"), |
| 258 | + ( |
| 259 | + "model", |
| 260 | + types.Part.from_function_call( |
| 261 | + name="simple_tool", args={"message": "payload"} |
| 262 | + ), |
| 263 | + ), |
| 264 | + ( |
| 265 | + "user", |
| 266 | + types.Part.from_function_response( |
| 267 | + name="simple_tool", |
| 268 | + response={"result": "Tool processed: payload"}, |
| 269 | + ), |
| 270 | + ), |
| 271 | + ] |
| 272 | + assert testing_utils.simplify_events(runner.session.events) == [ |
| 273 | + ("user", "Question"), |
| 274 | + ( |
| 275 | + "test_agent", |
| 276 | + types.Part.from_function_call( |
| 277 | + name="simple_tool", args={"message": "payload"} |
| 278 | + ), |
| 279 | + ), |
| 280 | + ( |
| 281 | + "test_agent", |
| 282 | + types.Part.from_function_response( |
| 283 | + name="simple_tool", |
| 284 | + response={"result": "Tool processed: payload"}, |
| 285 | + ), |
| 286 | + ), |
| 287 | + ("test_agent", "Answer"), |
| 288 | + ] |
| 289 | + |
| 290 | + |
| 291 | +def test_model_input_context_with_include_contents_none_sub_agent(): |
| 292 | + agent1_model = testing_utils.MockModel.create( |
| 293 | + responses=["Agent1 response: XYZ"] |
| 294 | + ) |
| 295 | + agent1 = LlmAgent(name="agent1", model=agent1_model) |
| 296 | + |
| 297 | + agent2_model = testing_utils.MockModel.create( |
| 298 | + responses=["Agent2 final response"] |
| 299 | + ) |
| 300 | + agent2 = LlmAgent( |
| 301 | + name="agent2", |
| 302 | + model=agent2_model, |
| 303 | + include_contents="none", |
| 304 | + ) |
| 305 | + sequential_agent = SequentialAgent( |
| 306 | + name="sequential_test_agent", sub_agents=[agent1, agent2] |
| 307 | + ) |
| 308 | + runner = testing_utils.InMemoryRunner(sequential_agent) |
| 309 | + session = runner.session |
| 310 | + |
| 311 | + list( |
| 312 | + runner.runner.run( |
| 313 | + user_id=session.user_id, |
| 314 | + session_id=session.id, |
| 315 | + new_message=testing_utils.get_user_content("Original user request"), |
| 316 | + run_config=RunConfig( |
| 317 | + model_input_context=[ |
| 318 | + types.UserContent("Relevant context for this turn") |
| 319 | + ] |
| 320 | + ), |
| 321 | + ) |
| 322 | + ) |
| 323 | + |
| 324 | + assert testing_utils.simplify_contents(agent1_model.requests[0].contents) == [ |
| 325 | + ("user", "Relevant context for this turn"), |
| 326 | + ("user", "Original user request"), |
| 327 | + ] |
| 328 | + assert testing_utils.simplify_contents(agent2_model.requests[0].contents) == [ |
| 329 | + ("user", "Relevant context for this turn"), |
| 330 | + ( |
| 331 | + "user", |
| 332 | + [ |
| 333 | + types.Part(text="For context:"), |
| 334 | + types.Part(text="[agent1] said: Agent1 response: XYZ"), |
| 335 | + ], |
| 336 | + ), |
| 337 | + ] |
| 338 | + |
| 339 | + |
192 | 340 | @pytest.mark.asyncio |
193 | 341 | async def test_include_contents_none_sequential_agents(): |
194 | 342 | """Test include_contents='none' with sequential agents.""" |
|
0 commit comments