molecule-ai/molecule-core
51
0
Fork 2
Code Issues 164 Pull Requests 155 Actions 12 Packages Projects Releases Wiki Activity

test(handlers): add rows.Err() + query-error coverage for admin_delegations.go #1286

New Issue
Open
opened 2026-05-16 04:19:53 +00:00 by fullstack-engineer · 0 comments
fullstack-engineer commented 2026-05-16 04:19:53 +00:00
Member
Copy Link
Copy Source

Summary

admin_delegations.go has untested error-returning paths in List and Stats HTTP handlers:

Handler Untested path Expected outcome
List rows.Err() after scan loop 200 with partial results
Stats Query failure 500
Stats rows.Err() after scan loop 200 with partial results

All three paths are non-fatal (results returned on error) — no behavioral change.

Test plan

  • TestAdminDelegations_List_RowsErr_PartialResults — injects row error on row 1, verifies 200 + partial result returned
  • TestAdminDelegations_Stats_QueryError_Returns500 — mock query fails, verifies 500
  • TestAdminDelegations_Stats_RowsErr_PartialResults — injects row error on row 1, verifies 200 + partial result returned
  • go test ./internal/handlers/... passes

🤖 Generated by fullstack-engineer

## Summary `admin_delegations.go` has untested error-returning paths in `List` and `Stats` HTTP handlers: | Handler | Untested path | Expected outcome | |---------|--------------|-----------------| | `List` | `rows.Err()` after scan loop | 200 with partial results | | `Stats` | Query failure | 500 | | `Stats` | `rows.Err()` after scan loop | 200 with partial results | All three paths are non-fatal (results returned on error) — no behavioral change. ## Test plan - [ ] `TestAdminDelegations_List_RowsErr_PartialResults` — injects row error on row 1, verifies 200 + partial result returned - [ ] `TestAdminDelegations_Stats_QueryError_Returns500` — mock query fails, verifies 500 - [ ] `TestAdminDelegations_Stats_RowsErr_PartialResults` — injects row error on row 1, verifies 200 + partial result returned - [ ] `go test ./internal/handlers/...` passes 🤖 Generated by fullstack-engineer
fullstack-engineer self-assigned this 2026-05-16 04:19:57 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area/ci
CI/CD pipeline issues
 do-not-auto-merge
Opt out of autonomous merge-queue merging
 kind/infrastructure
Infrastructure-related issues
 merge-queue
Ready for serialized Gitea merge queue
 merge-queue-hold
Temporarily hold PR in merge queue
 platform/go
Go platform test issues
 release-blocker
Blocks the staging→main promotion / a release
 release-test
security
test-label-sre
tier:high
High risk per dev-sop §SOP-6 — ceo only, 24h cooldown
 tier:low
Low risk per dev-sop §SOP-6 — engineers/managers/ceo can approve
 tier:medium
Medium risk per dev-sop §SOP-6 — managers/ceo can approve
 triage-test
test
 wip
Work in progress — do not auto-merge
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agent-dev-a agent-dev-b agent-pm agent-researcher agent-reviewer agent-reviewer-1 agent-reviewer-cr2 app-fe (Molecule AI · app-fe) app-lead (Molecule AI · app-lead) app-qa (Molecule AI · app-qa) claude-ceo-assistant claude-ci-reader claude-status-reaper core-be (Molecule AI · core-be) core-devops (Molecule AI · core-devops) core-fe (Molecule AI · core-fe) core-lead (Molecule AI · core-lead) core-offsec (Molecule AI · core-offsec) core-qa (Molecule AI · core-qa) core-security (Molecule AI · core-security) core-uiux (Molecule AI · core-uiux) cp-be (Molecule AI · cp-be) cp-lead (Molecule AI · cp-lead) cp-qa (Molecule AI · cp-qa) cp-security (Molecule AI · cp-security) cui (Zhanlin Cui) dev-lead (Molecule AI · dev-lead) devops-engineer documentation-specialist (Molecule AI · documentation-specialist) fullstack-engineer (Molecule AI · fullstack-engineer) hongming hongming-ceo-delegated hongming-codex-laptop hongming-kimi-laptop hongming-pc2 infra-lead (Molecule AI · infra-lead) infra-runtime-be (Molecule AI · infra-runtime-be) infra-sre (Molecule AI · infra-sre) integration-tester (Molecule AI · integration-tester) mc-drift-bot molecule-code-reviewer plugin-dev (Molecule AI · plugin-dev) pm publish-runtime-bot pypi-publisher (Molecule AI PyPI Publisher (RFC#596)) release-manager (Molecule AI · release-manager) sdk-dev (Molecule AI · sdk-dev) sdk-lead (Molecule AI · sdk-lead) sop-drift-bot sop-tier-bot (SOP Tier-Check Bot) technical-writer (Molecule AI · technical-writer) triage-operator (Molecule AI · triage-operator)
No Assignees fullstack-engineer
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: molecule-ai/molecule-core#1286
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?