Skip to content

Add unit tests for DateRequestHandler#786

Merged
StuartFerguson merged 2 commits intomainfrom
copilot/add-daterequesthandler-tests
Feb 23, 2026
Merged

Add unit tests for DateRequestHandler#786
StuartFerguson merged 2 commits intomainfrom
copilot/add-daterequesthandler-tests

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 23, 2026

DateRequestHandler had no unit test coverage. Adds tests targeting the single Handle method to cover both the success and failure paths through the IApiClient.GetComparisonDates delegation.

New tests — RequestHandlers/DateRequestHandlerTests.cs

  • Handle_ReturnsSuccess_WhenApiClientSucceeds — verifies the success result and data is forwarded as-is from the API client
  • Handle_ReturnsFailure_WhenApiClientFails — verifies failure propagation without transformation

Both tests assert the API client is called exactly once via Verify.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • f.feedz.io
    • Triggering command: /usr/bin/dotnet dotnet test EstateManagementUI.BlazorServer.Tests/EstateManagementUI.BlazorServer.Tests.csproj --filter DateRequestHandlerTests (dns block)
    • Triggering command: /opt/hostedtoolcache/CodeQL/2.24.0/x64/codeql/csharp/tools/linux64/Semmle.Autobuild.CSharp /opt/hostedtoolcache/CodeQL/2.24.0/x64/codeql/csharp/tools/linux64/Semmle.Autobuild.CSharp (dns block)
    • Triggering command: /usr/bin/dotnet dotnet restore --no-dependencies /home/REDACTED/work/EstateManagementUI/EstateManagementUI/EstateManagementUI.sln --packages /home/REDACTED/work/EstateManagementUI/.codeql-scratch/dbs/csharp/working/packages /p:DisableImplicitNuGetFallbackFolder=true --verbosity normal /p:TargetFrameworkRootPath=/home/REDACTED/work/EstateManagementUI/.codeql-scratch/dbs/csharp/working/emptyFakeDotnetRoot /p:NetCoreTargetingPackRoot=/home/REDACTED/work/EstateManagementUI/.codeql-scratch/dbs/csharp/working/emptyFakeDotnetRoot /p:AllowMissingPrunePackageData=true (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>DateRequestHandler Unit Tests</issue_title>
<issue_description></issue_description>

<agent_instructions>Ensure a good level of code coverage as close to 100% as possible</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: StuartFerguson <16325469+StuartFerguson@users.noreply.github.com>
Copilot AI changed the title [WIP] Add unit tests for DateRequestHandler Add unit tests for DateRequestHandler Feb 23, 2026
Copilot AI requested a review from StuartFerguson February 23, 2026 19:44
@StuartFerguson StuartFerguson marked this pull request as ready for review February 23, 2026 19:47
@StuartFerguson StuartFerguson merged commit 23ec048 into main Feb 23, 2026
10 checks passed
@github-actions github-actions Bot deleted the copilot/add-daterequesthandler-tests branch April 25, 2026 01:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DateRequestHandler Unit Tests

2 participants