Tests & cleanup #10

Merged

bpeskiewicz merged 15 commits from feature/cs into main

2026-04-24 19:01:01 +00:00

bpeskiewicz commented

2026-04-24 18:38:27 +00:00

Owner

No description provided.

bpeskiewicz added 14 commits

2026-04-24 18:38:27 +00:00

fix: correct nullability annotation on Result<T>.Value property e4571eeffb

feat: add implicit operator for Result<T> to enable clean syntax 98e37f1784

fix: use worst status code when combining errors in Result.Combine da0244673d

Changed the Combine method to use Math.Max to select the worst (highest)
status code when both results fail, instead of always using the first
error's status code. This better represents the combined error severity.

feat: map exception types to appropriate HTTP status codes in TryCatch ef4d4e6016

fix: add reflection safety check in ResultFilter for Value property e39458d3f7

refactor: remove unnecessary null-forgiving operators in AuthController 10431fab9a

Remove null-forgiving operators (!) from ErrorDetails access in AuthController.
While the compiler still generates nullability warnings due to interface design
limitations, the operators are logically unnecessary as ErrorDetails is guaranteed
to be non-null when IsFailure is true.

test: add Result factory method tests ef81b6cc56

test: add monad operation tests (Bind, Then, Map, Match) 5ccd75951c

test: add Combine and TryCatch tests with exception type mapping 065d0554b0

test: add implicit operator conversion tests 8a4724cc5a

test: add LINQ extension method tests (Select, Where, SelectMany) 310accfa79

test: add ResultFilter integration tests a4a6e5f0a6

docs: update plan-monad.md with implementation status 175bc088c5

refactor: centralize signed download links and tighten result/path coverage

Run Tests / test (pull_request) Failing after 1m20s

Details

Run Tests / lint (pull_request) Successful in 1m23s

Details