Follow-ups from #31 #34

New Issue

aep · 2026-10-16T09:12:43+03:00

aep commented

2026-10-16 09:12:43 +03:00

Отложенные находки финального ревью ветки feat/31-bug-and-attachments (PR #32, merge 6740ccf).

M-2 — test_hub_non_json_file_no_content_in_error подменяет HubClient.file: настоящий путь Http.send/FolderTransport с обрезанным 2xx тестом не проверен (зонд ревьюера: поведение верное). Источник: финальное ревью #31.
M-3 — любой 404 от /agent/files считается «файл удалён» (R-20): при откате хаба на версию без маршрута текст ошибки неточный; проверять no_file в ответе. Источник: финальное ревью #31.
M-4 — docstring HubError упоминает только refresh. Источник: финальное ревью #31.
M-5 — ответ Jira на загрузку вложения покрыт тестом только для []; добавить {} и [123]. Источник: финальное ревью #31.
M-6 — README: строка про timeout не говорит про 120 с на загрузку и про файл у хаба. Источник: финальное ревью #31.

Отложенные находки финального ревью ветки `feat/31-bug-and-attachments` (PR #32, merge 6740ccf). - **M-2** — `test_hub_non_json_file_no_content_in_error` подменяет `HubClient.file`: настоящий путь `Http.send`/`FolderTransport` с обрезанным 2xx тестом не проверен (зонд ревьюера: поведение верное). Источник: финальное ревью #31. - **M-3** — любой 404 от `/agent/files` считается «файл удалён» (R-20): при откате хаба на версию без маршрута текст ошибки неточный; проверять `no_file` в ответе. Источник: финальное ревью #31. - **M-4** — docstring `HubError` упоминает только refresh. Источник: финальное ревью #31. - **M-5** — ответ Jira на загрузку вложения покрыт тестом только для `[]`; добавить `{}` и `[123]`. Источник: финальное ревью #31. - **M-6** — README: строка про `timeout` не говорит про 120 с на загрузку и про файл у хаба. Источник: финальное ревью #31.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: aep/jira-agent#34