Task1 problems

[costea32]

<this.> is a explicit representation, you don't have to use it unless to remove ambiguity,

Clean up should be done using IUserHandler Methods, not directly on UserList

UserList is not tested, and no LINQ was used

[costea32]

NameCompare - misleading name - it acts as a "IsUserInHandlerList", and it only verifies by First/Last name

private UserHandlerProvider uh; is used only in TestInit, so there is no meaning in making him a class member

Data validation tests all use the same pattern - create helper method to reduce code duplication

As there is a "perfect" implementation - you already have specs, so you can remove tests that were written assuming different ways in which this could work, and only leave the appropriate ones

GetUsersByAge_OneUserFound - beside checking the count you should also check the actual user

GetUserByName_Basic - misleading test - name suggests it tests GetUserByName() method, but actually tests that user can be retrieved through the UserList

Review current tests, and check if they comply with expectations for Task1

[costea32]

Almost forgot - rename project and class to appropriate names, rename folder also