[Sprint] Issues: Update unit and integration tests for ObjectId/Result<T> changes

[mpaulosky]

Problem

After handler and shared type changes (Issues #1 and #2), 12 Issue-domain test files will break. Tests need updating to match new Result return types and ObjectId ID handling.

Depends On

Depends on: Foundation issue, Issues handler refactor issue

Files to Modify

Unit Handler Tests

• ests/Unit.Tests/Handlers/Issues/GetIssueHandlerTests.cs
• ests/Unit.Tests/Handlers/Issues/ListIssuesHandlerTests.cs
• ests/Unit.Tests/Handlers/Issues/CreateIssueHandlerTests.cs
• ests/Unit.Tests/Handlers/Issues/UpdateIssueHandlerTests.cs
• ests/Unit.Tests/Handlers/Issues/DeleteIssueHandlerTests.cs
• ests/Unit.Tests/Handlers/Issues/UpdateIssueStatusHandlerTests.cs

Unit Endpoint Tests

• ests/Unit.Tests/Endpoints/IssueEndpointsTests.cs

Integration Tests

• ests/Integration.Tests/Handlers/GetIssueHandlerTests.cs
• ests/Integration.Tests/Handlers/ListIssuesHandlerIntegrationTests.cs
• ests/Integration.Tests/Handlers/CreateIssueHandlerTests.cs
• ests/Integration.Tests/Handlers/UpdateIssueHandlerIntegrationTests.cs
• ests/Integration.Tests/Handlers/DeleteIssueHandlerIntegrationTests.cs

Supporting

• ests/Unit.Tests/Builders/IssueBuilder.cs — update ID handling for ObjectId
• ests/Integration.Tests/Builders/IssueBuilder.cs — same

Acceptance Criteria

• All Issue handler unit tests updated to assert Result returns
• All Issue integration tests updated to use ObjectId-compatible ID setup
• Endpoint tests verify HTTP status codes (not just response bodies)
• All tests pass (green)
• Integration tests retain [Collection("Integration")] attribute

[github-actions]

🏗️ Squad Triage — Aragorn (Lead)

Issue: #82 — [Sprint] Issues: Update unit and integration tests for ObjectId/Result changes
Assigned to: Legolas (Frontend Dev)
Reason: Issue relates to frontend/UI work

@copilot evaluation: No strong capability match — routed to squad member.

Team roster:

• Aragorn (Lead) → label: squad:aragorn
• Legolas (Frontend Dev) → label: squad:legolas
• Sam (Backend Dev) → label: squad:sam
• Gimli (Tester) → label: squad:gimli
• Boromir (DevOps) → label: squad:boromir
• Frodo (Tech Writer) → label: squad:frodo
• Gandalf (Security Officer) → label: squad:gandalf
• Ralph (Work Monitor) → label: squad:ralph
• @copilot (🤖 Coding Agent) → label: squad:@copilot
• @copilot (Coding Agent) → label: squad:copilot

To reassign, remove the current squad:* label and add the correct one.

[github-actions]

📋 Assigned to Gimli (Tester)

Issue: #82 — [Sprint] Issues: Update unit and integration tests for ObjectId/Result changes

Gimli will pick this up in the next Copilot session.

For Copilot coding agent: If enabled, this issue will be worked automatically.
Otherwise, start a Copilot session and say:
Gimli, work on issue #82

[mpaulosky]

Tests updated by Gimli in PR #92 (merged to main). All 417 unit tests pass.