Skip to content

Commit 5734e9a

Browse files
jopemachineclaude
andcommitted
test: remove trivial owner_access_key assertion test classes
Removed TestRestartSessionRequest and TestGetStatusHistoryRequest which only checked that owner_access_key was no longer present. Trivial attribute checks add no value over the schema definition itself. Also clean up unused imports for the removed test classes. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 269eafc commit 5734e9a

1 file changed

Lines changed: 0 additions & 18 deletions

File tree

tests/unit/common/dto/manager/session/test_request.py

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -25,12 +25,10 @@
2525
GetAbusingReportRequest,
2626
GetCommitStatusRequest,
2727
GetContainerLogsRequest,
28-
GetStatusHistoryRequest,
2928
GetTaskLogsRequest,
3029
ListFilesRequest,
3130
MatchSessionsRequest,
3231
RenameSessionRequest,
33-
RestartSessionRequest,
3432
ShutdownServiceRequest,
3533
StartServiceRequest,
3634
SyncAgentRegistryRequest,
@@ -343,15 +341,6 @@ def test_defaults(self) -> None:
343341
assert req.recursive is False
344342

345343

346-
class TestRestartSessionRequest:
347-
def test_defaults(self) -> None:
348-
# The request model carries no fields; ensure it still accepts an
349-
# empty body and that the removed ``owner_access_key`` is gone.
350-
req = RestartSessionRequest()
351-
assert not hasattr(req, "owner_access_key")
352-
assert req.model_dump() == {}
353-
354-
355344
class TestMatchSessionsRequest:
356345
def test_required(self) -> None:
357346
req = MatchSessionsRequest.model_validate({"id": "some-session"})
@@ -440,10 +429,3 @@ def test_task_id_alias(self) -> None:
440429
kid = uuid4()
441430
req = GetTaskLogsRequest.model_validate({"taskId": str(kid)})
442431
assert req.kernel_id == kid
443-
444-
445-
class TestGetStatusHistoryRequest:
446-
def test_defaults(self) -> None:
447-
req = GetStatusHistoryRequest()
448-
assert not hasattr(req, "owner_access_key")
449-
assert req.model_dump() == {}

0 commit comments

Comments
 (0)