Skip to content

Update test coverage for Float Domain service#1515

Merged
StuartFerguson merged 2 commits intomasterfrom
copilot/update-test-coverage-float-service
Feb 21, 2026
Merged

Update test coverage for Float Domain service#1515
StuartFerguson merged 2 commits intomasterfrom
copilot/update-test-coverage-float-service

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 21, 2026

FloatDomainService lacked test coverage for failure and exception paths across all four service methods. RecordTransaction had no tests at all.

New tests added

CreateFloatForContractProduct

  • GetFloatFailedGetLatest<FloatAggregate> returns failure
  • SaveFailedAggregateService.Save returns failure
  • ExceptionThrown — exception propagated through try/catch

RecordCreditPurchase (FloatCommands)

  • GetFloatFailedGetLatest<FloatAggregate> returns failure

RecordCreditPurchase (FloatActivityCommands)

  • GetFloatActivityFailedGetLatest<FloatActivityAggregate> returns failure

RecordTransaction (all new)

  • TransactionRecorded — happy path
  • GetTransactionFailedGetLatest<TransactionAggregate> returns failure
  • GetFloatActivityFailedGetLatest<FloatActivityAggregate> returns failure
  • SaveFailedAggregateService.Save returns failure
  • ExceptionThrown — exception propagated through try/catch

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 TransactionProcessor.BusinessLogic.Tests/TransactionProcessor.BusinessLogic.Tests.csproj --filter FloatDomainService --no-build .Testing/TestData.cs (dns block)
    • Triggering command: /usr/bin/dotnet dotnet build TransactionProcessor.BusinessLogic.Tests/TransactionProcessor.BusinessLogic.Tests.csproj (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)

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>Update Test Coverage on Float Domain service</issue_title>
<issue_description>Each domain service method needs new tests added to cover all cases, including failure and exceptions.</issue_description>

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


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Update test coverage for Float Domain service Update test coverage for Float Domain service Feb 21, 2026
Copilot AI requested a review from StuartFerguson February 21, 2026 09:49
@StuartFerguson StuartFerguson marked this pull request as ready for review February 21, 2026 15:33
@StuartFerguson StuartFerguson merged commit 50431c6 into master Feb 21, 2026
10 checks passed
@github-actions github-actions Bot deleted the copilot/update-test-coverage-float-service branch April 23, 2026 03:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update Test Coverage on Float Domain service

2 participants