refactor(tests): tighten ExceptionHandlerSetup.Collector to private

[ANcpLua]

Summary

ExceptionHandlerSetup.Collector was public but nothing outside the class needs it — the four _setup.Logger.Collector.GetSnapshot() call sites in ExceptionHandlerTests reach the collector via FakeLogger<T>'s own property, not the setup's redundant exposure. Marking it private keeps the test fixture's surface area honest.

Test plan

• Confirmed all four usages go through _setup.Logger.Collector (i.e., FakeLogger<T>.Collector), not _setup.Collector
• CI Build & Test (backend) green (no behavioural change, accessibility only)

[coderabbitai]

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)

• PaperlessREST.Tests/Unit/ExceptionHandlerTests.cs is excluded by none and included by none

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 943234fc-f1a0-49b2-b55f-3f22f0968a06

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

Narrows the test fixture API surface by making ExceptionHandlerSetup.Collector private, since the tests already access log snapshots via FakeLogger<T>.Collector rather than through the setup object.

Changes:

• Changed ExceptionHandlerSetup.Collector from public to private in the test fixture.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.