Files
George Weale f828667ee0 fix: import jinja2 lazily so it is not a required dependency
Co-authored-by: George Weale <gweale@google.com>
PiperOrigin-RevId: 961101849
2026-08-07 13:47:46 -07:00

401 lines
13 KiB
Python

# Copyright 2026 Google LLC
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
import importlib.util
import sys
from unittest import mock
from google.adk.agents.llm_agent import Agent
from google.adk.agents.llm_agent import InstructionProvider as LlmAgentInstructionProvider
from google.adk.agents.readonly_context import ReadonlyContext
from google.adk.sessions.session import Session
from google.adk.utils import instructions_utils
from google.adk.utils.instructions_utils import InstructionProvider
import pytest
from .. import testing_utils
class MockArtifactService:
def __init__(self, artifacts: dict):
self.artifacts = artifacts
async def load_artifact(self, app_name, user_id, session_id, filename):
if filename in self.artifacts:
return self.artifacts[filename]
else:
return None
async def _create_test_readonly_context(
state: dict = None,
artifact_service: MockArtifactService = None,
app_name: str = "test_app",
user_id: str = "test_user",
session_id: str = "test_session_id",
) -> ReadonlyContext:
agent = Agent(
model="gemini-2.5-flash",
name="agent",
instruction="test",
)
invocation_context = await testing_utils.create_invocation_context(
agent=agent
)
invocation_context.session = Session(
state=state if state else {},
app_name=app_name,
user_id=user_id,
id=session_id,
)
invocation_context.artifact_service = artifact_service
return ReadonlyContext(invocation_context)
@pytest.mark.asyncio
async def test_inject_session_state():
instruction_template = "Hello {user_name}, you are in {app_state} state."
invocation_context = await _create_test_readonly_context(
state={"user_name": "Foo", "app_state": "active"}
)
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context
)
assert populated_instruction == "Hello Foo, you are in active state."
@pytest.mark.asyncio
async def test_inject_session_state_without_placeholders_returns_template():
instruction_template = "A static instruction with no placeholders."
invocation_context = await _create_test_readonly_context(
state={"user_name": "Foo"}
)
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context
)
assert populated_instruction == instruction_template
@pytest.mark.asyncio
async def test_inject_session_state_with_artifact():
instruction_template = "The artifact content is: {artifact.my_file}"
mock_artifact_service = MockArtifactService(
{"my_file": "This is my artifact content."}
)
invocation_context = await _create_test_readonly_context(
artifact_service=mock_artifact_service
)
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context
)
assert (
populated_instruction
== "The artifact content is: This is my artifact content."
)
@pytest.mark.asyncio
async def test_inject_session_state_with_optional_state():
instruction_template = "Optional value: {optional_value?}"
invocation_context = await _create_test_readonly_context()
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context
)
assert populated_instruction == "Optional value: "
@pytest.mark.asyncio
async def test_inject_session_state_with_missing_state_raises_key_error():
instruction_template = "Hello {missing_key}!"
invocation_context = await _create_test_readonly_context(
state={"user_name": "Foo"}
)
with pytest.raises(
KeyError, match="Context variable not found: `missing_key`."
):
await instructions_utils.inject_session_state(
instruction_template, invocation_context
)
@pytest.mark.asyncio
async def test_inject_session_state_with_missing_artifact_raises_key_error():
instruction_template = "The artifact content is: {artifact.missing_file}"
mock_artifact_service = MockArtifactService(
{"my_file": "This is my artifact content."}
)
invocation_context = await _create_test_readonly_context(
artifact_service=mock_artifact_service
)
with pytest.raises(KeyError, match="Artifact missing_file not found."):
await instructions_utils.inject_session_state(
instruction_template, invocation_context
)
@pytest.mark.asyncio
async def test_inject_session_state_with_invalid_state_name_returns_original():
instruction_template = "Hello {invalid-key}!"
invocation_context = await _create_test_readonly_context(
state={"user_name": "Foo"}
)
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context
)
assert populated_instruction == "Hello {invalid-key}!"
@pytest.mark.asyncio
async def test_inject_session_state_with_invalid_prefix_state_name_returns_original():
instruction_template = "Hello {invalid:key}!"
invocation_context = await _create_test_readonly_context(
state={"user_name": "Foo"}
)
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context
)
assert populated_instruction == "Hello {invalid:key}!"
@pytest.mark.asyncio
async def test_inject_session_state_with_valid_prefix_state():
instruction_template = "Hello {app:user_name}!"
invocation_context = await _create_test_readonly_context(
state={"app:user_name": "Foo"}
)
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context
)
assert populated_instruction == "Hello Foo!"
@pytest.mark.asyncio
async def test_inject_session_state_with_multiple_variables_and_artifacts():
instruction_template = """
Hello {user_name},
You are {user_age} years old.
Your favorite color is {favorite_color?}.
The artifact says: {artifact.my_file}
And another optional artifact: {artifact.other_file}
"""
mock_artifact_service = MockArtifactService({
"my_file": "This is my artifact content.",
"other_file": "This is another artifact content.",
})
invocation_context = await _create_test_readonly_context(
state={"user_name": "Foo", "user_age": 30, "favorite_color": "blue"},
artifact_service=mock_artifact_service,
)
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context
)
expected_instruction = """
Hello Foo,
You are 30 years old.
Your favorite color is blue.
The artifact says: This is my artifact content.
And another optional artifact: This is another artifact content.
"""
assert populated_instruction == expected_instruction
@pytest.mark.asyncio
async def test_inject_session_state_with_empty_artifact_name_raises_key_error():
instruction_template = "The artifact content is: {artifact.}"
mock_artifact_service = MockArtifactService(
{"my_file": "This is my artifact content."}
)
invocation_context = await _create_test_readonly_context(
artifact_service=mock_artifact_service
)
with pytest.raises(KeyError, match="Artifact not found."):
await instructions_utils.inject_session_state(
instruction_template, invocation_context
)
@pytest.mark.asyncio
async def test_inject_session_state_artifact_service_not_initialized_raises_value_error():
instruction_template = "The artifact content is: {artifact.my_file}"
invocation_context = await _create_test_readonly_context()
with pytest.raises(ValueError, match="Artifact service is not initialized."):
await instructions_utils.inject_session_state(
instruction_template, invocation_context
)
@pytest.mark.asyncio
async def test_inject_session_state_with_optional_missing_artifact_returns_empty():
instruction_template = "Optional artifact: {artifact.missing_file?}"
mock_artifact_service = MockArtifactService(
{"my_file": "This is my artifact content."}
)
invocation_context = await _create_test_readonly_context(
artifact_service=mock_artifact_service
)
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context
)
assert populated_instruction == "Optional artifact: "
@pytest.mark.asyncio
async def test_inject_session_state_with_none_state_value_returns_empty():
instruction_template = "Value: {test_key}"
invocation_context = await _create_test_readonly_context(
state={"test_key": None}
)
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context
)
assert populated_instruction == "Value: "
@pytest.mark.asyncio
async def test_inject_session_state_with_optional_missing_state_returns_empty():
instruction_template = "Optional value: {missing_key?}"
invocation_context = await _create_test_readonly_context()
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context
)
assert populated_instruction == "Optional value: "
@pytest.mark.asyncio
async def test_inject_session_state_jinja2_basic_variable():
instruction_template = (
"Hello {{ user_name }}, you are in {{ app_state }} state."
)
invocation_context = await _create_test_readonly_context(
state={"user_name": "Foo", "app_state": "active"}
)
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context, use_jinja2=True
)
assert populated_instruction == "Hello Foo, you are in active state."
@pytest.mark.asyncio
async def test_inject_session_state_jinja2_conditional():
instruction_template = "{% if show_hint %}Hint: read the docs.{% endif %}"
invocation_context = await _create_test_readonly_context(
state={"show_hint": True}
)
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context, use_jinja2=True
)
assert populated_instruction == "Hint: read the docs."
@pytest.mark.asyncio
async def test_inject_session_state_jinja2_for_loop():
instruction_template = "{% for item in items %}{{ item }} {% endfor %}"
invocation_context = await _create_test_readonly_context(
state={"items": ["a", "b", "c"]}
)
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context, use_jinja2=True
)
assert populated_instruction == "a b c "
@pytest.mark.asyncio
async def test_inject_session_state_jinja2_artifact():
instruction_template = "Content: {{ artifact('my_file') }}"
mock_artifact_service = MockArtifactService({"my_file": "artifact data"})
invocation_context = await _create_test_readonly_context(
artifact_service=mock_artifact_service
)
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context, use_jinja2=True
)
assert populated_instruction == "Content: artifact data"
@pytest.mark.asyncio
async def test_inject_session_state_jinja2_undefined_variable_raises():
instruction_template = "Hello {{ missing_var }}!"
invocation_context = await _create_test_readonly_context()
with pytest.raises(Exception):
await instructions_utils.inject_session_state(
instruction_template, invocation_context, use_jinja2=True
)
@pytest.mark.asyncio
async def test_inject_session_state_jinja2_artifact_with_filter():
instruction_template = "Content: {{ artifact('my_file') | upper }}"
mock_artifact_service = MockArtifactService({"my_file": "artifact data"})
invocation_context = await _create_test_readonly_context(
artifact_service=mock_artifact_service
)
populated_instruction = await instructions_utils.inject_session_state(
instruction_template, invocation_context, use_jinja2=True
)
assert populated_instruction == "Content: ARTIFACT DATA"
def test_module_imports_without_jinja2_installed():
# Jinja2 ships only in the eval and test extras, but this module is on the
# import path of google.adk.agents, so a module-scope import of it would
# break every install that does not pull in those extras.
spec = importlib.util.find_spec("google.adk.utils.instructions_utils")
module = importlib.util.module_from_spec(spec)
with mock.patch.dict(sys.modules, {"jinja2": None}):
spec.loader.exec_module(module)
@pytest.mark.asyncio
async def test_inject_session_state_jinja2_without_jinja2_installed():
invocation_context = await _create_test_readonly_context()
with mock.patch.dict(sys.modules, {"jinja2": None}):
with pytest.raises(ImportError, match="pip install jinja2"):
await instructions_utils.inject_session_state(
"Hello {{ name }}", invocation_context, use_jinja2=True
)
def test_module_exposes_instruction_provider_alias():
assert instructions_utils.InstructionProvider is InstructionProvider
def test_llm_agent_reexports_same_instruction_provider():
# Existing importers rely on `from ...llm_agent import InstructionProvider`;
# it must remain the exact same object after moving the alias here.
assert LlmAgentInstructionProvider is InstructionProvider