-
Notifications
You must be signed in to change notification settings - Fork 0
Integration test for the rating write path #34
Copy link
Copy link
Open
Labels
area: dataEF Core, DbContext and migrations (Ssabba.Infrastructure)EF Core, DbContext and migrations (Ssabba.Infrastructure)area: domainEntities and rating maths (Ssabba.Domain)Entities and rating maths (Ssabba.Domain)type: choreMaintenance, refactors, toolingMaintenance, refactors, tooling
Milestone
Description
Metadata
Metadata
Assignees
Labels
area: dataEF Core, DbContext and migrations (Ssabba.Infrastructure)EF Core, DbContext and migrations (Ssabba.Infrastructure)area: domainEntities and rating maths (Ssabba.Domain)Entities and rating maths (Ssabba.Domain)type: choreMaintenance, refactors, toolingMaintenance, refactors, tooling
MatchRatingCalculatoris covered by unit tests and the schema is verified by applyingthe migrations, but the glue between them is not tested at all.
MatchQueries.CreateAsyncderives the community and format from the teams, snapshots therule set, writes
MatchAppearancerows, movesCommunityMember.Ratingand setsRatingAppliedAt. None of that is exercised against a real database.There is no infrastructure test project to put such a test in, which is the first thing
this needs.
service container in CI)
the rule set is snapshotted onto the match, and the season is picked up
player who is not a member of the community
RatingAppliedAtmakes a second application a no-opSplit out of #32.