Skip to content

Commit 1641e86

Browse files
author
Valentina Bojan
committed
fix: fix lint
1 parent d35f2e6 commit 1641e86

2 files changed

Lines changed: 30 additions & 12 deletions

File tree

src/uipath_langchain/agent/guardrails/guardrail_nodes.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,9 @@ def _create_validation_command(
108108
if guardrail_result.result == GuardrailValidationResultType.PASSED:
109109
return Command(
110110
goto=success_node,
111-
update={"inner_state": {"guardrail_validation_result": guardrail_result.reason}},
111+
update={
112+
"inner_state": {"guardrail_validation_result": guardrail_result.reason}
113+
},
112114
)
113115

114116
if guardrail_result.result == GuardrailValidationResultType.VALIDATION_FAILED:

tests/agent/guardrails/test_guardrail_nodes.py

Lines changed: 27 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,9 @@ async def test_llm_success_pre_and_post(
9191
guardrail = MagicMock(spec=BuiltInValidatorGuardrail)
9292
guardrail.name = "Example"
9393
_patch_uipath(
94-
monkeypatch, result=GuardrailValidationResultType.PASSED, reason=""
94+
monkeypatch,
95+
result=GuardrailValidationResultType.PASSED,
96+
reason="validation passed",
9597
)
9698
node_name, node = create_llm_guardrail_node(
9799
guardrail=guardrail,
@@ -103,7 +105,9 @@ async def test_llm_success_pre_and_post(
103105
state = AgentGuardrailsGraphState(messages=[HumanMessage("payload")])
104106
cmd = await node(state)
105107
assert cmd.goto == "ok"
106-
assert cmd.update == {"inner_state": {"guardrail_validation_result": None}}
108+
assert cmd.update == {
109+
"inner_state": {"guardrail_validation_result": "validation passed"}
110+
}
107111

108112
@pytest.mark.asyncio
109113
@pytest.mark.parametrize(
@@ -162,7 +166,9 @@ async def test_agent_init_success_pre_and_post(
162166
guardrail = MagicMock(spec=BuiltInValidatorGuardrail)
163167
guardrail.name = "Example"
164168
fake = _patch_uipath(
165-
monkeypatch, result=GuardrailValidationResultType.PASSED, reason=""
169+
monkeypatch,
170+
result=GuardrailValidationResultType.PASSED,
171+
reason="validation passed",
166172
)
167173

168174
node_name, node = create_agent_init_guardrail_node(
@@ -176,7 +182,9 @@ async def test_agent_init_success_pre_and_post(
176182
state = AgentGuardrailsGraphState(messages=[HumanMessage("payload")])
177183
cmd = await node(state)
178184
assert cmd.goto == "ok"
179-
assert cmd.update == {"inner_state": {"guardrail_validation_result": None}}
185+
assert cmd.update == {
186+
"inner_state": {"guardrail_validation_result": "validation passed"}
187+
}
180188
assert fake.guardrails.last_text == "payload"
181189

182190
@pytest.mark.asyncio
@@ -239,7 +247,9 @@ async def test_agent_terminate_success_pre_and_post(
239247
guardrail = MagicMock(spec=BuiltInValidatorGuardrail)
240248
guardrail.name = "Example"
241249
fake = _patch_uipath(
242-
monkeypatch, result=GuardrailValidationResultType.PASSED, reason=""
250+
monkeypatch,
251+
result=GuardrailValidationResultType.PASSED,
252+
reason="validation passed",
243253
)
244254

245255
node_name, node = create_agent_terminate_guardrail_node(
@@ -257,7 +267,9 @@ async def test_agent_terminate_success_pre_and_post(
257267
)
258268
cmd = await node(state)
259269
assert cmd.goto == "ok"
260-
assert cmd.update == {"inner_state": {"guardrail_validation_result": None}}
270+
assert cmd.update == {
271+
"inner_state": {"guardrail_validation_result": "validation passed"}
272+
}
261273
assert fake.guardrails.last_text == str(agent_result)
262274

263275
@pytest.mark.asyncio
@@ -348,15 +360,15 @@ async def test_tool_success_pre_and_post(
348360
)
349361
cmd = await node(state)
350362
assert cmd.goto == "ok"
351-
assert cmd.update == {"inner_state": {"guardrail_validation_result": None}}
363+
assert cmd.update == {"inner_state": {"guardrail_validation_result": ""}}
352364
assert json.loads(fake.guardrails.last_text or "{}") == {"x": 1}
353365
else:
354366
state = AgentGuardrailsGraphState(
355367
messages=[ToolMessage(content="tool output", tool_call_id="call_1")]
356368
)
357369
cmd = await node(state)
358370
assert cmd.goto == "ok"
359-
assert cmd.update == {"inner_state": {"guardrail_validation_result": None}}
371+
assert cmd.update == {"inner_state": {"guardrail_validation_result": ""}}
360372
assert fake.guardrails.last_text == "tool output"
361373

362374
@pytest.mark.asyncio
@@ -509,7 +521,9 @@ async def test_evaluate_builtin_guardrail(self, monkeypatch):
509521
)
510522

511523
fake = _patch_uipath(
512-
monkeypatch, result=GuardrailValidationResultType.PASSED, reason=""
524+
monkeypatch,
525+
result=GuardrailValidationResultType.PASSED,
526+
reason="validation passed",
513527
)
514528

515529
guardrail = MagicMock(spec=BuiltInValidatorGuardrail)
@@ -532,12 +546,14 @@ def test_create_validation_command_success(self):
532546

533547
result = GuardrailValidationResult(
534548
result=GuardrailValidationResultType.PASSED,
535-
reason="",
549+
reason="validation passed",
536550
)
537551
command = _create_validation_command(result, "success_node", "failure_node")
538552

539553
assert command.goto == "success_node"
540-
assert command.update == {"inner_state": {"guardrail_validation_result": None}}
554+
assert command.update == {
555+
"inner_state": {"guardrail_validation_result": "validation passed"}
556+
}
541557

542558
def test_create_validation_command_failure(self):
543559
"""Test validation command creation for failed validation."""

0 commit comments

Comments
 (0)