test(unit): add handler unit tests for IBlogPostCacheService (#111)

[mpaulosky]

Closes #111

Working as Gimli (Tester)

Summary

Adds a new Unit.Tests project with TDD-red handler unit tests that document the expected IBlogPostCacheService contract after Sam's #110 handler refactor. Tests are marked with the TDD-red pattern — they exist as executable specifications but will not compile on sprint/5-redis-caching until #110 merges.

Also adds the Domain.Tests stub project (required by Gate 3 of the pre-push hook, removed in #104) and registers it in MyBlog.slnx.

Why TDD-red?

IBlogPostCacheService (introduced in #109) is only available on squad/109-cache-abstraction / squad/110-handler-refactor. The handler constructors on sprint/5 still use IMemoryCache + IDistributedCache. Adding Unit.Tests to MyBlog.slnx would fail Gate 2 on this branch, so the project is present on disk but not registered in the solution file until #110 merges.

Changes

File	What
tests/Unit.Tests/Unit.Tests.csproj	New test project (net10.0, xunit, NSubstitute, FluentAssertions)
tests/Unit.Tests/GlobalUsings.cs	Global usings including IBlogPostCacheService
tests/Unit.Tests/Handlers/GetBlogPostsHandlerTests.cs	3 tests: delegates to cache, fetch-on-miss, exception→fail
tests/Unit.Tests/Handlers/GetBlogPostByIdHandlerTests.cs	4 tests: id forwarded, cache-miss, null from repo, exception
tests/Unit.Tests/Handlers/CreateBlogPostHandlerTests.cs	3 tests: success+InvalidateAllAsync, no InvalidateById, repo throws
tests/Unit.Tests/Handlers/EditBlogPostHandlerTests.cs	3 tests: success invalidates both keys, NotFound, concurrency
tests/Unit.Tests/Handlers/DeleteBlogPostHandlerTests.cs	3 tests: success invalidates both keys, repo throws, concurrency
tests/Domain.Tests/Domain.Tests.csproj	Stub project (re-added for Gate 3)
tests/Domain.Tests/GlobalUsings.cs	Minimal usings for stub
MyBlog.slnx	Add Domain.Tests only (Unit.Tests excluded until #110 merges)

Depends On

• [Sprint 5] Extract ICacheService abstraction and centralize cache keys #109 (IBlogPostCacheService interface) — merged into sprint/5 via [Sprint 5] Refactor BlogPost handlers to use ICacheService #110
• [Sprint 5] Refactor BlogPost handlers to use ICacheService #110 (handler refactor) — must merge before Unit.Tests compiles

⚠️ This task was flagged as "needs review" — Unit.Tests does not compile on sprint/5-redis-caching base. Please have a squad member verify test contracts match Sam's #110 implementation before merging.

[github-actions]

🏗️ PR Added to Squad Triage Queue

This PR has been labeled with squad and added to the triage queue.

Next steps:

• The squad Lead will review and assign to an appropriate team member
• A squad:member label will be added after triage

If you know which squad member should handle this, you can add the appropriate squad:member label yourself.

[github-actions]

Test Results Summary

203 tests   203 ✅  14s ⏱️
  5 suites    0 💤
  5 files      0 ❌

Results for commit 05ac5fc.

♻️ This comment has been updated with latest results.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (sprint/5-redis-caching@239da3b). Learn more about missing BASE report.

Additional details and impacted files

@@                    Coverage Diff                    @@
##             sprint/5-redis-caching     #117   +/-   ##
=========================================================
  Coverage                          ?   70.38%           
=========================================================
  Files                             ?       43           
  Lines                             ?      672           
  Branches                          ?      111           
=========================================================
  Hits                              ?      473           
  Misses                            ?      147           
  Partials                          ?       52           

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull request overview

Adds new (currently TDD-red) unit-test specifications for the upcoming IBlogPostCacheService-based handler refactor, and reintroduces a stub Domain.Tests project to satisfy the repo’s pre-push Gate 3 expectations.

Changes:

• Add a new tests/Unit.Tests project with handler tests written against the future IBlogPostCacheService contract (intentionally not wired into MyBlog.slnx yet).
• Add a stub tests/Domain.Tests project and register it in MyBlog.slnx.
• Add supporting global usings for the new test projects.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
tests/Unit.Tests/Unit.Tests.csproj	New Unit test project targeting net10.0 with xUnit/NSubstitute/FluentAssertions references.
tests/Unit.Tests/GlobalUsings.cs	Adds global usings for Unit.Tests, including caching abstractions.
tests/Unit.Tests/Handlers/GetBlogPostsHandlerTests.cs	TDD-red tests describing expected caching delegation behavior for “get all” handler.
tests/Unit.Tests/Handlers/GetBlogPostByIdHandlerTests.cs	TDD-red tests describing expected caching delegation behavior for “get by id” query path.
tests/Unit.Tests/Handlers/CreateBlogPostHandlerTests.cs	TDD-red tests for create behavior + cache invalidation expectations.
tests/Unit.Tests/Handlers/EditBlogPostHandlerTests.cs	TDD-red tests for edit behavior, invalidation expectations, and concurrency error mapping.
tests/Unit.Tests/Handlers/DeleteBlogPostHandlerTests.cs	TDD-red tests for delete behavior, invalidation expectations, and concurrency error mapping.
tests/Domain.Tests/Domain.Tests.csproj	Stub Domain.Tests project (net10.0) to satisfy local Gate 3 requirements.
tests/Domain.Tests/GlobalUsings.cs	Minimal global usings for Domain.Tests stub.
MyBlog.slnx	Registers tests/Domain.Tests in the solution.