[Test] Unit + Integration tests for DeleteCategoryHandler and DeleteStatusHandler

[mpaulosky]

Summary

Sprint 3 adds DeleteCategoryHandler and DeleteStatusHandler. This issue covers all test coverage for those handlers.

Test Files to Create

Unit Tests — Handler Layer

tests/Api.Tests.Unit/Handlers/Categories/DeleteCategoryHandlerTests.cs

• Handle_ValidId_SetsArchivedFieldsAndPersists — verifies Archived=true, ArchivedBy, ArchivedOn are set
• Handle_CategoryNotFound_ReturnsFailure
• Handle_RepositoryThrows_ReturnsFailure

tests/Api.Tests.Unit/Handlers/Statuses/DeleteStatusHandlerTests.cs

• Same three tests, mirrored for Status domain

Unit Tests — Endpoint Layer

tests/Api.Tests.Unit/Endpoints/CategoryEndpointsTests.cs (extend existing)

• DeleteCategory_AdminUser_Returns204
• DeleteCategory_NonAdmin_Returns403
• DeleteCategory_NotFound_Returns404

tests/Api.Tests.Unit/Endpoints/StatusEndpointsTests.cs (extend existing)

• Mirror above for Status

Integration Tests

tests/Api.Tests.Integration/Handlers/DeleteCategoryHandlerIntegrationTests.cs

• Full round-trip: create a category via handler, delete it, verify it's not returned in List
• Use TestContainers MongoDB setup (follow existing integration test pattern)

tests/Api.Tests.Integration/Handlers/DeleteStatusHandlerIntegrationTests.cs

• Same round-trip for Status

Test Tooling (match existing patterns)

• xUnit, FluentAssertions, NSubstitute (unit)
• TestContainers + MongoDbRunner (integration)
• Follow the existing *HandlerTests.cs pattern in Api.Tests.Unit/Handlers/Categories/

Acceptance Criteria

• All new unit tests pass locally
• All new integration tests pass locally (with TestContainers)
• Code coverage for DeleteCategoryHandler and DeleteStatusHandler is ≥ 80%
• CI pipeline stays green

Depends On

• [Feature] API: Add DeleteCategory handler and archive endpoint #120 (DeleteCategory API)
• [Feature] API: Add DeleteStatus handler and archive endpoint #121 (DeleteStatus API)

[github-actions]

📋 Assigned to Gimli (Tester)

Issue: #122 — [Test] Unit + Integration tests for DeleteCategoryHandler and DeleteStatusHandler

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 #122

[github-actions]

🏗️ Squad Triage — Aragorn (Lead)

Issue: #122 — [Test] Unit + Integration tests for DeleteCategoryHandler and DeleteStatusHandler
Assigned to: @copilot (Coding Agent)
Reason: 🟢 Good fit for @copilot — matches capability profile

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.