kindred/silo
1
1
Fork 0
Code Issues 30 Pull Requests Actions Packages Projects Releases Wiki Activity

test: file upload/download handler test coverage #76

New Issue
Closed
opened 2026-02-13 19:02:54 +00:00 by forbes · 0 comments
forbes commented 2026-02-13 19:02:54 +00:00
Owner
Copy Link

Ref: docs/ROADMAP.md § Appendix B / 1.4 Unit Test Suite

No tests exist for file handling endpoints. internal/api/file_handlers.go has 7 exported handlers with zero test coverage:

  • HandleUploadFile — multipart upload, checksum verification
  • HandleDownloadFile — download by revision number
  • HandleDownloadLatestFile — latest revision download
  • HandleListItemFiles — list file attachments
  • HandleDeleteItemFile — delete file association
  • HandleSetItemThumbnail — thumbnail upload
  • HandleAssociateItemFile — presigned upload association

Tests should mock MinIO storage and verify checksum validation, error responses for missing files, and correct content-type headers.

**Ref:** `docs/ROADMAP.md` § Appendix B / 1.4 Unit Test Suite No tests exist for file handling endpoints. `internal/api/file_handlers.go` has 7 exported handlers with zero test coverage: - [ ] `HandleUploadFile` — multipart upload, checksum verification - [ ] `HandleDownloadFile` — download by revision number - [ ] `HandleDownloadLatestFile` — latest revision download - [ ] `HandleListItemFiles` — list file attachments - [ ] `HandleDeleteItemFile` — delete file association - [ ] `HandleSetItemThumbnail` — thumbnail upload - [ ] `HandleAssociateItemFile` — presigned upload association Tests should mock MinIO storage and verify checksum validation, error responses for missing files, and correct content-type headers.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
forbes
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: kindred/silo#76
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?