🐛♻️✅ Fix-uts #51

Merged

alexis.drai merged 6 commits from fix-uts into main

alexis.drai commented

Owner

There is no content yet.

alexis.drai added 6 commits

dcb9de97e5 🩹 Pass params by vals instead of by refs

if we pass parameters by reference, it must be because we want to
change them in the method and to keep those changes.
This seems like a different situation, so we should be able
to pass the objects by value.

f1404f829d 🐛 Add validation and delegation to Player.Remove()

4472c56354 ✅ Improve Update() test after bugfix

085e27122a ✏️ Fix typo

e3a86a67b9 ✅ Generalize from HashSets to IEnumerables or Collections

10690dc991 ♻️ Refactor, D.R.Y., use "expected" and "actual"

alexis.drai merged commit fd661d5b5c into main 2 years ago

alexis.drai deleted branch fix-uts

alexis.drai referenced this issue from a commit

Merge pull request '🐛♻️✅ Fix-uts' (#51) from fix-uts into main

continuous-integration/drone/push Build is passing

Details

The pull request has been merged as fd661d5b5c.

No reviewers

No Label

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: alexis.drai/dice_app#51