Skip to content

Commit 2dd83df

Browse files
committed
test: address review feedback on manage_packages tests
1 parent 890be2d commit 2dd83df

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

Server/tests/test_manage_packages.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ def test_unknown_action_does_not_call_unity(self):
133133
mock_send.assert_not_called()
134134

135135
def test_action_matching_is_case_insensitive(self):
136-
"""Actions must be accepted regardless of capitalisation."""
136+
"""Actions must be accepted regardless of capitalisation and normalised to lowercase."""
137137
from services.tools.manage_packages import manage_packages
138138

139139
ctx = MagicMock()
@@ -145,7 +145,10 @@ def test_action_matching_is_case_insensitive(self):
145145
) as mock_send:
146146
mock_send.return_value = {"success": True, "message": "OK"}
147147
result = asyncio.run(manage_packages(ctx, action="LIST_PACKAGES"))
148+
148149
assert result["success"] is True
150+
sent_params = mock_send.call_args.args[1]
151+
assert sent_params["action"] == "list_packages"
149152

150153

151154
# =============================================================================

0 commit comments

Comments
 (0)