Skip to content

#35 Add discriminated union result - #37

Merged
dannygoodacre merged 18 commits into
mainfrom
35-discriminated-union-result-type
Sep 3, 2026
Merged

dannygoodacre merged 18 commits into
mainfrom
35-discriminated-union-result-type

Conversation

@dannygoodacre

@dannygoodacre dannygoodacre commented Sep 2, 2026 •

Copy link
Copy Markdown
Owner
  • Update Result to be a discriminated union record instead of a class.
  • Update all affected code to support the new Result.
  • Misc. code and test tidying.

@dannygoodacre dannygoodacre linked an issue Sep 2, 2026 that may be closed by this pull request
@dannygoodacre dannygoodacre self-assigned this Sep 2, 2026
@github-actions

github-actions Bot commented Sep 2, 2026 •

Copy link
Copy Markdown

Summary

Summary
Generated on: 09/03/2026 - 19:41:44
Coverage date: 09/03/2026 - 19:41:41 - 09/03/2026 - 19:41:43
Parser: MultiReport (4x Cobertura)
Assemblies: 5
Classes: 36
Files: 23
Line coverage: 94.6% (462 of 488)
Covered lines: 462
Uncovered lines: 26
Coverable lines: 488
Total lines: 1441
Branch coverage: 85.4% (123 of 144)
Covered branches: 123
Total branches: 144
Method coverage: Feature is only available for sponsors

Coverage

DannyGoodacre.Cqrs - 91.1%
Name Line Branch
DannyGoodacre.Cqrs 91.1% 80.5%
DannyGoodacre.Cqrs.CommandHandler`1 100%
DannyGoodacre.Cqrs.CommandHandler`2 100%
DannyGoodacre.Cqrs.CommandHandlerBase`2 100% 100%
DannyGoodacre.Cqrs.LoggerExtensions 100% 50%
DannyGoodacre.Cqrs.QueryHandler`2 96.1% 100%
DannyGoodacre.Cqrs.StateCommandHandler`1 37.5%
DannyGoodacre.Cqrs.StateCommandHandler`2 37.5%
DannyGoodacre.Cqrs.StateCommandHandlerBase`2 96.1% 100%
DannyGoodacre.Cqrs.TransactionCommandHandler`1 37.5%
DannyGoodacre.Cqrs.TransactionCommandHandler`2 37.5%
DannyGoodacre.Cqrs.TransactionCommandHandlerBase`2 97.7% 100%
DannyGoodacre.Cqrs.TypeExtensions 100% 92.8%
Microsoft.Extensions.DependencyInjection.ServiceCollectionExtensions 100% 100%
DannyGoodacre.Cqrs.EntityFrameworkCore - 100%
Name Line Branch
DannyGoodacre.Cqrs.EntityFrameworkCore 100% 93.7%
DannyGoodacre.Cqrs.EntityFrameworkCore.DbContextTransactionUnit`1 100% 91.6%
Microsoft.Extensions.DependencyInjection.ServiceCollectionExtensions 100% 100%
DannyGoodacre.Cqrs.Testing - 98.2%
Name Line Branch
DannyGoodacre.Cqrs.Testing 98.2% 100%
DannyGoodacre.Cqrs.Testing.CommandHandlerTestCore`2 100%
DannyGoodacre.Cqrs.Testing.LoggerMockExtensions 100%
DannyGoodacre.Cqrs.Testing.QueryHandlerTestBase`2 100%
DannyGoodacre.Cqrs.Testing.StateCommandHandlerTestCore`2 100%
DannyGoodacre.Cqrs.Testing.TransactionCommandHandlerTestCore`2 94.4% 100%
DannyGoodacre.Primitives - 98.3%
Name Line Branch
DannyGoodacre.Primitives 98.3% 95.4%
DannyGoodacre.Primitives.Conflict 100%
DannyGoodacre.Primitives.Conflict`1 100%
DannyGoodacre.Primitives.DomainError 100%
DannyGoodacre.Primitives.DomainError`1 100%
DannyGoodacre.Primitives.Error 100%
DannyGoodacre.Primitives.InternalError 100%
DannyGoodacre.Primitives.InternalError`1 100%
DannyGoodacre.Primitives.Invalid 100%
DannyGoodacre.Primitives.Invalid`1 100%
DannyGoodacre.Primitives.IResult 94.4% 92.8%
DannyGoodacre.Primitives.Result 100%
DannyGoodacre.Primitives.Result`1 100%
DannyGoodacre.Primitives.Success`1 100%
DannyGoodacre.Primitives.ValidationState 100% 100%
DannyGoodacre.Testing - 98.5%
Name Line Branch
DannyGoodacre.Testing 98.5% 83.3%
DannyGoodacre.Testing.LoggerExtensions 94.4% 50%
DannyGoodacre.Testing.TestBase 100% 85.7%

Full coverage report artifact

@dannygoodacre
dannygoodacre marked this pull request as ready for review September 3, 2026 19:29
@dannygoodacre
dannygoodacre merged commit 82b0e32 into main Sep 3, 2026
1 check passed
@dannygoodacre
dannygoodacre deleted the 35-discriminated-union-result-type branch September 3, 2026 19:42
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.

Discriminated union result type

1 participant