search

JanPalasek / smart-home-server

Server that is supposed to handle everything what smart home requires.
MIT License
0 stars 0 forks source link

Write tests #16

Open JanPalasek opened 4 years ago

JanPalasek commented 4 years ago

New unit tests should be written to cover repositories + services at least.

JanPalasek commented 4 years ago

AddPermissionsToUserAsync behaves like this: If we add permissions to user that already has it, it throws no error. It should be reexamined whether this is expected behaviour and same behaviour should be implemented in AddPermissionsToRoleAsync.

Furthermore many more tests should be written to cover at least the more difficult functionality. Basic GetAll etc probably do not need to be tested.