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

[mpaulosky]

Problem

10 Comment-domain test files will break after handler and shared type changes (Issues #1 and #8).

Depends On

Depends on: Foundation issue, Comments handler refactor issue

Files to Modify

• ests/Unit.Tests/Handlers/Comments/GetCommentHandlerTests.cs
• ests/Unit.Tests/Handlers/Comments/ListCommentsHandlerTests.cs
• ests/Unit.Tests/Handlers/Comments/CreateCommentHandlerTests.cs
• ests/Unit.Tests/Handlers/Comments/UpdateCommentHandlerTests.cs
• ests/Unit.Tests/Handlers/Comments/DeleteCommentHandlerTests.cs
• ests/Unit.Tests/Endpoints/CommentEndpointsTests.cs
• ests/Integration.Tests/Handlers/GetCommentHandlerIntegrationTests.cs
• ests/Integration.Tests/Handlers/CreateCommentHandlerIntegrationTests.cs
• ests/Integration.Tests/Handlers/UpdateCommentHandlerIntegrationTests.cs
• ests/Integration.Tests/Handlers/DeleteCommentHandlerIntegrationTests.cs
• ests/Unit.Tests/Builders/CommentBuilder.cs

Acceptance Criteria

• All Comment handler unit tests updated for Result returns
• Integration tests updated for ObjectId ID setup
• All tests pass (green)
• Integration tests retain [Collection("Integration")] attribute

[github-actions]

📋 Assigned to Gimli (Tester)

Issue: #88 — [Sprint] Comments: 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 #88

[github-actions]

🏗️ Squad Triage — Aragorn (Lead)

Issue: #88 — [Sprint] Comments: 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.

[mpaulosky]

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