Full test coverage for OperatorDomainService — fix dead code branches and missing edge cases#1506
Merged
StuartFerguson merged 6 commits intoFeb 20, 2026
GitHub Actions / Unit Test Results
succeeded
Feb 20, 2026 in 0s
1065 passed, 0 failed and 6 skipped
✅ TransactionProcessor.Aggregates.Tests/TestResults/aggregates-test-results.trx
464 tests were completed in 1s with 464 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| TransactionProcessor.Aggregates.Tests.ContractAggregateTests | 93✅ | 88ms | ||
| TransactionProcessor.Aggregates.Tests.EstateAggregateTests | 18✅ | 21ms | ||
| TransactionProcessor.Aggregates.Tests.FloatActivityAggregateTests | 5✅ | 22ms | ||
| TransactionProcessor.Aggregates.Tests.FloatAggregateTests | 8✅ | 28ms | ||
| TransactionProcessor.Aggregates.Tests.MerchantAggregateTests | 63✅ | 129ms | ||
| TransactionProcessor.Aggregates.Tests.MerchantDepositListAggregateTests | 18✅ | 63ms | ||
| TransactionProcessor.Aggregates.Tests.MerchantStatementAggregateTests | 17✅ | 26ms | ||
| TransactionProcessor.Aggregates.Tests.MerchantStatementForDateAggregateTests | 20✅ | 50ms | ||
| TransactionProcessor.Aggregates.Tests.OperatorAggregateTests | 13✅ | 26ms | ||
| TransactionProcessor.Aggregates.Tests.ReconciliationAggregateTests | 26✅ | 37ms | ||
| TransactionProcessor.Aggregates.Tests.SettlementAggregateTests | 26✅ | 64ms | ||
| TransactionProcessor.Aggregates.Tests.TransactionAggregateTests | 131✅ | 64ms | ||
| TransactionProcessor.Aggregates.Tests.VoucherAggregateTests | 26✅ | 31ms |
✅ TransactionProcessor.BusinessLogic.Tests/TestResults/buslogic-test-results.trx
457 tests were completed in 62s with 452 passed, 0 failed and 5 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| TransactionProcessor.BusinessLogic.Tests.DomainEventHandlers.ContractDomainEventHandlerTests | 5✅ | 5ms | ||
| TransactionProcessor.BusinessLogic.Tests.DomainEventHandlers.EstateDomainEventHandlerTests | 4✅ | 266ms | ||
| TransactionProcessor.BusinessLogic.Tests.DomainEventHandlers.MerchantDomainEventHandlerTests | 28✅ | 1⚪ | 36ms | |
| TransactionProcessor.BusinessLogic.Tests.DomainEventHandlers.MerchantStatementDomainEventHandlerTests | 12✅ | 60s | ||
| TransactionProcessor.BusinessLogic.Tests.DomainEventHandlers.OperatorDomainEventHandlerTests | 4✅ | 5ms | ||
| TransactionProcessor.BusinessLogic.Tests.DomainEventHandlers.TransactionDomainEventHandlerTests | 8✅ | 66ms | ||
| TransactionProcessor.BusinessLogic.Tests.DomainEventHandlers.VoucherDomainEventHandlerTests | 2✅ | 99ms | ||
| TransactionProcessor.BusinessLogic.Tests.Extensions.ExtensionsTests | 2✅ | 1ms | ||
| TransactionProcessor.BusinessLogic.Tests.Manager.FeeCalculationManagerTests | 12✅ | 10ms | ||
| TransactionProcessor.BusinessLogic.Tests.Manager.TransactionProcessorManagerTests | 30✅ | 67ms | ||
| TransactionProcessor.BusinessLogic.Tests.Manager.VoucherManagementManagerTests | 4✅ | 997ms | ||
| TransactionProcessor.BusinessLogic.Tests.Mediator.MediatorTests | 1✅ | 6s | ||
| TransactionProcessor.BusinessLogic.Tests.OperatorInterfaces.PataPawaPostPayProxyTests | 19✅ | 89ms | ||
| TransactionProcessor.BusinessLogic.Tests.OperatorInterfaces.PataPawaPrePayProxyTests | 20✅ | 724ms | ||
| TransactionProcessor.BusinessLogic.Tests.OperatorInterfaces.SafaricomPinlessProxyTests | 16✅ | 170ms | ||
| TransactionProcessor.BusinessLogic.Tests.OperatorInterfaces.VoucherManagementProxyTests | 10✅ | 8ms | ||
| TransactionProcessor.BusinessLogic.Tests.RequestHandler.FloatRequestHandlerTests | 2✅ | 13ms | ||
| TransactionProcessor.BusinessLogic.Tests.RequestHandler.SettlementRequestHandlerTests | 1✅ | 13ms | ||
| TransactionProcessor.BusinessLogic.Tests.RequestHandler.TransactionRequestHandlerTests | 4✅ | 13ms | ||
| TransactionProcessor.BusinessLogic.Tests.RequestHandler.VoucherManagementRequestHandlerTests | 2✅ | 10ms | ||
| TransactionProcessor.BusinessLogic.Tests.Services.ContractDomainServiceTests | 28✅ | 230ms | ||
| TransactionProcessor.BusinessLogic.Tests.Services.DomainServiceHelperTests | 4✅ | 1ms | ||
| TransactionProcessor.BusinessLogic.Tests.Services.EstateDomainServiceTests | 25✅ | 51ms | ||
| TransactionProcessor.BusinessLogic.Tests.Services.FloatDomainServiceTests | 10✅ | 37ms | ||
| TransactionProcessor.BusinessLogic.Tests.Services.MerchantDomainServiceTests | 55✅ | 2⚪ | 156ms | |
| TransactionProcessor.BusinessLogic.Tests.Services.MerchantStatementDomainServiceTests | 20✅ | 134ms | ||
| TransactionProcessor.BusinessLogic.Tests.Services.OperatorDomainServiceTests | 16✅ | 40ms | ||
| TransactionProcessor.BusinessLogic.Tests.Services.SettlementDomainServiceTests | 16✅ | 15s | ||
| TransactionProcessor.BusinessLogic.Tests.Services.StatementBuilderTests | 6✅ | 121ms | ||
| TransactionProcessor.BusinessLogic.Tests.Services.TransactionDomainServiceTests | 41✅ | 167ms | ||
| TransactionProcessor.BusinessLogic.Tests.Services.TransactionReceiptBuilderTests | 1✅ | 104ms | ||
| TransactionProcessor.BusinessLogic.Tests.Services.TransactionValidationServiceTests | 37✅ | 2⚪ | 120ms | |
| TransactionProcessor.BusinessLogic.Tests.Services.VoucherDomainServiceTests | 7✅ | 397ms |
✅ TransactionProcessor.DatabaseTests/TestResults/database-test-results.trx
22 tests were completed in 1086s with 22 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| TransactionProcessor.DatabaseTests.ContractEventTests | 16✅ | 3s | ||
| TransactionProcessor.DatabaseTests.MerchantEventTests | 6✅ | 778ms |
✅ TransactionProcessor.IntegrationTests/TestResults/integration-test-results.trx
9 tests were completed in 931s with 9 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| TransactionProcessor.IntegrationTests.Features.LogonTransactionFeature | 1✅ | 316s | ||
| TransactionProcessor.IntegrationTests.Features.MerchantFeature | 2✅ | 609s | ||
| TransactionProcessor.IntegrationTests.Features.OperatorFeature | 1✅ | 324s | ||
| TransactionProcessor.IntegrationTests.Features.ReconciliationFeature | 1✅ | 291s | ||
| TransactionProcessor.IntegrationTests.Features.RedeemVoucherFeature | 1✅ | 299s | ||
| TransactionProcessor.IntegrationTests.Features.SaleTransactionFeature | 1✅ | 322s | ||
| TransactionProcessor.IntegrationTests.Features.SettlementFeature | 1✅ | 300s | ||
| TransactionProcessor.IntegrationTests.Features.SettlementReportingFeature | 1✅ | 304s |
✅ TransactionProcessor.ProjectionEngine.Tests/TestResults/projection-test-results.trx
72 tests were completed in 801ms with 72 passed, 0 failed and 0 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| TransactionProcessor.ProjectionEngine.Tests.MerchantBalanceProjectionTests | 27✅ | 34ms | ||
| TransactionProcessor.ProjectionEngine.Tests.MerchantBalanceStateDispatcherTests | 9✅ | 98ms | ||
| TransactionProcessor.ProjectionEngine.Tests.MerchantBalanceStateExtensionsTests | 13✅ | 15ms | ||
| TransactionProcessor.ProjectionEngine.Tests.ProjectionHandlerTests | 8✅ | 106ms | ||
| TransactionProcessor.ProjectionEngine.Tests.VoucherProjectionTests | 10✅ | 13ms | ||
| TransactionProcessor.ProjectionEngine.Tests.VoucherStateDispatcherTests | 1✅ | 1ms | ||
| TransactionProcessor.ProjectionEngine.Tests.VoucherStateExtensionsTests | 4✅ | 1ms |
✅ TransactionProcessor.Tests/TestResults/tests-test-results.trx
47 tests were completed in 3s with 46 passed, 0 failed and 1 skipped.
| Test suite | Passed | Failed | Skipped | Time |
|---|---|---|---|---|
| TransactionProcessor.Tests.Common.ExtensionsTests | 2✅ | 11ms | ||
| TransactionProcessor.Tests.ControllerTests.TransactionControllerTests | 1⚪ | 1ms | ||
| TransactionProcessor.Tests.Factories.ModelFactoryTests | 43✅ | 125ms | ||
| TransactionProcessor.Tests.General.BootstrapperTests | 1✅ | 2s |
Loading