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

test: database and item CRUD test coverage #75

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

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

Expand test coverage for the database layer. Existing tests in internal/db/items_test.go cover basics (10 tests) but are missing coverage for:

  • Database connection and transaction handling (error paths, rollback)
  • Item CRUD edge cases (duplicate part numbers, archived items, property validation)
  • Revision creation and retrieval (sequential numbering, rollback, comparison, status/label updates)
  • Project-item associations (tagging, untagging, project-filtered queries)

Existing test files:

  • internal/db/items_test.go (10 tests, 272 lines)
  • internal/db/relationships_test.go (10 tests, 278 lines)
  • internal/db/projects_test.go (5 tests, 119 lines)
  • internal/db/helpers_test.go (1 test, 15 lines)

Goal: Increase db package test coverage to cover all exported functions.

**Ref:** `docs/ROADMAP.md` § Appendix B / 1.4 Unit Test Suite Expand test coverage for the database layer. Existing tests in `internal/db/items_test.go` cover basics (10 tests) but are missing coverage for: - [ ] Database connection and transaction handling (error paths, rollback) - [ ] Item CRUD edge cases (duplicate part numbers, archived items, property validation) - [ ] Revision creation and retrieval (sequential numbering, rollback, comparison, status/label updates) - [ ] Project-item associations (tagging, untagging, project-filtered queries) **Existing test files:** - `internal/db/items_test.go` (10 tests, 272 lines) - `internal/db/relationships_test.go` (10 tests, 278 lines) - `internal/db/projects_test.go` (5 tests, 119 lines) - `internal/db/helpers_test.go` (1 test, 15 lines) **Goal:** Increase db package test coverage to cover all exported functions.
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#75
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?