From 7f0ac60ef1a6eef30e10a8a963e17ba497829f21 Mon Sep 17 00:00:00 2001
From: ancplua <anfh22@outlook.com>
Date: Tue, 12 May 2026 15:29:38 +0200
Subject: [PATCH] refactor(tests): tighten ExceptionHandlerSetup.Collector to
 private

Nothing outside ExceptionHandlerSetup needs the FakeLogCollector directly -
the four `_setup.Logger.Collector.GetSnapshot()` call sites reach it via
FakeLogger<T>'s own property, not the setup's redundant exposure. Marking
it private keeps the test fixture's surface area honest.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 PaperlessREST.Tests/Unit/ExceptionHandlerTests.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/PaperlessREST.Tests/Unit/ExceptionHandlerTests.cs b/PaperlessREST.Tests/Unit/ExceptionHandlerTests.cs
index 52d7c16..16c07dd 100644
--- a/PaperlessREST.Tests/Unit/ExceptionHandlerTests.cs
+++ b/PaperlessREST.Tests/Unit/ExceptionHandlerTests.cs
@@ -27,7 +27,7 @@ public ExceptionHandlerSetup()
 
 	public Mock<IProblemDetailsService> ProblemDetails { get; } = new();
 	public FakeLogger<GlobalExceptionHandler> Logger { get; }
-	public FakeLogCollector Collector { get; }
+	private FakeLogCollector Collector { get; }
 
 	public GlobalExceptionHandler CreateHandler() => new(ProblemDetails.Object, Logger);