Cleaning up HTTP results

[alexeyzimarev]

• Consolidated return types
• Use extensions instead of duplicated code
• Removed unnecessary generic constraints

[github-actions]

Test Results

37 tests   - 99   37 ✔️  - 97   2s ⏱️ - 3m 43s
  2 suites  - 46     0 💤 ±  0 
  2 files    - 46     0 ❌  -   2 

Results for commit 36bda3b. ± Comparison against base commit 134b66c.

This pull request removes 101 and adds 2 tests. Note that renamed tests count towards both.

Eventuous.Tests.Application.FunctionalServiceTests ‑ AmendEventAddsMeta
Eventuous.Tests.Application.FunctionalServiceTests ‑ ExecuteOnAnyForNewStream
Eventuous.Tests.Application.FunctionalServiceTests ‑ ExecuteOnExistingStream
Eventuous.Tests.Application.FunctionalServiceTests ‑ ExecuteOnNewStream
Eventuous.Tests.Application.StateWithIdTests ‑ ShouldGetIdForNew
Eventuous.Tests.AspNetCore.AggregateFactoryRegistrationTests ‑ ShouldCreateNewAggregateByResolve
Eventuous.Tests.AspNetCore.AggregateFactoryRegistrationTests ‑ ShouldCreateNewAggregateWithExplicitFunction
Eventuous.Tests.AspNetCore.AggregateFactoryRegistrationTests ‑ ShouldCreateTwoSeparateInstances
Eventuous.Tests.AspNetCore.Web.AggregateCommandsTests ‑ MapAggregateContractToCommandExplicitly<BookingResult>(tResult: BookingResult { State = , Success = False, Changes = , State =  })
Eventuous.Tests.AspNetCore.Web.AggregateCommandsTests ‑ MapAggregateContractToCommandExplicitly<Result>(tResult: Result { State = , Success = False, Changes =  })
…

Eventuous.Tests.Subscriptions.SequenceTests ‑ ShouldReturnFirstBefore(sequence: [CommitPosition { Position: 0, Sequence: 1, Timestamp: 2023-11-17T08:55:07.5975544+00:00 }, CommitPosition { Position: 0, Sequence: 2, Timestamp: 2023-11-17T08:55:07.5975544+00:00 }, CommitPosition { Position: 0, Sequence: 4, Timestamp: 2023-11-17T08:55:07.5975544+00:00 }, CommitPosition { Position: 0, Sequence: 6, Timestamp: 2023-11-17T08:55:07.5975544+00:00 }], expected: CommitPosition { Position: 0, Sequence: 2, Timestamp: 2023-11-17T08:55:07.5975544+00:00 })
Eventuous.Tests.Subscriptions.SequenceTests ‑ ShouldReturnFirstBefore(sequence: [CommitPosition { Position: 0, Sequence: 1, Timestamp: 2023-11-17T08:55:07.5975544+00:00 }, CommitPosition { Position: 0, Sequence: 2, Timestamp: 2023-11-17T08:55:07.5975544+00:00 }, CommitPosition { Position: 0, Sequence: 6, Timestamp: 2023-11-17T08:55:07.5975544+00:00 }, CommitPosition { Position: 0, Sequence: 8, Timestamp: 2023-11-17T08:55:07.5975544+00:00 }], expected: CommitPosition { Position: 0, Sequence: 2, Timestamp: 2023-11-17T08:55:07.5975544+00:00 })

[LockTar]

So far I can see this will work in our situation! Nice refactoring! Good that some files are gone now.