Skip to content

Commit 4e4d527

Browse files
refactor: format tool.execute calls for improved readability
- Adjust formatting of tool.execute calls in test_feedback.py for consistency - Enhance code clarity by aligning dictionary entries - All tests remain passing
1 parent 2f702d7 commit 4e4d527

File tree

1 file changed

+21
-15
lines changed

1 file changed

+21
-15
lines changed

tests/test_feedback.py

Lines changed: 21 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -77,11 +77,13 @@ def test_multiple_account_ids_validation(self) -> None:
7777
mock_response.raise_for_status = Mock()
7878
mock_request.return_value = mock_response
7979

80-
result = tool.execute({
81-
"feedback": "Great tools!",
82-
"account_id": ["acc_123456", "acc_789012"],
83-
"tool_names": ["test_tool"]
84-
})
80+
result = tool.execute(
81+
{
82+
"feedback": "Great tools!",
83+
"account_id": ["acc_123456", "acc_789012"],
84+
"tool_names": ["test_tool"],
85+
}
86+
)
8587
assert "total_accounts" in result
8688
assert result["total_accounts"] == 2
8789

@@ -197,11 +199,13 @@ def test_multiple_account_ids_execution(self) -> None:
197199
mock_response.raise_for_status = Mock()
198200
mock_request.return_value = mock_response
199201

200-
result = tool.execute({
201-
"feedback": "Great tools!",
202-
"account_id": ["acc_123456", "acc_789012", "acc_345678"],
203-
"tool_names": ["test_tool"]
204-
})
202+
result = tool.execute(
203+
{
204+
"feedback": "Great tools!",
205+
"account_id": ["acc_123456", "acc_789012", "acc_345678"],
206+
"tool_names": ["test_tool"],
207+
}
208+
)
205209

206210
# Check combined result structure
207211
assert result["message"] == "Feedback sent to 3 account(s)"
@@ -246,11 +250,13 @@ def mock_request_side_effect(*args, **kwargs):
246250
with patch("requests.request") as mock_request:
247251
mock_request.side_effect = mock_request_side_effect
248252

249-
result = tool.execute({
250-
"feedback": "Great tools!",
251-
"account_id": ["acc_123456", "acc_unauthorized"],
252-
"tool_names": ["test_tool"]
253-
})
253+
result = tool.execute(
254+
{
255+
"feedback": "Great tools!",
256+
"account_id": ["acc_123456", "acc_unauthorized"],
257+
"tool_names": ["test_tool"],
258+
}
259+
)
254260

255261
# Check combined result structure
256262
assert result["total_accounts"] == 2

0 commit comments

Comments
 (0)