Skip to content

Commit da2a94a

Browse files
committed
test(cli): cover remaining cloud status error detail branch
Signed-off-by: phernandez <paul@basicmachines.co>
1 parent 7d4a8c6 commit da2a94a

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

src/basic_memory/cli/commands/project.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ def _format_cloud_index_status_error(error: Exception) -> str:
160160
"""Convert cloud lookup failures into concise user-facing text."""
161161
if isinstance(error, CloudAPIError):
162162
detail_message: str | None = None
163-
detail = error.detail.get("detail") if isinstance(error.detail, dict) else None
163+
detail = error.detail.get("detail")
164164
if isinstance(detail, str):
165165
detail_message = detail
166166
elif isinstance(detail, dict):

tests/cli/test_project_info_cloud_status.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -487,6 +487,12 @@ def test_format_cloud_index_status_error_prefers_cloud_api_detail():
487487
)
488488
== "HTTP 503: nested"
489489
)
490+
assert (
491+
project_cmd._format_cloud_index_status_error(
492+
CloudAPIError("fail", status_code=503, detail={"detail": {"detail": "nested-detail"}})
493+
)
494+
== "HTTP 503: nested-detail"
495+
)
490496
assert (
491497
project_cmd._format_cloud_index_status_error(CloudAPIError("fail", status_code=503))
492498
== "HTTP 503"

0 commit comments

Comments
 (0)