Clean up AccountsLogic #1918

Fixes Issue #1918
Add tests to cover AccountsLogic
Do TODO, changed return type to void for some functions
Add assumption in getCourseInstitute
Add TODO in StudentsLogic - StudentAttribute should not be created  if course specified in it does not exist.
Move Account Level tests from LogicTest to AccountsLogic test.