Clean up StudentCourseJoinAuthenticatedAction:#1959

1. simple stuff to improve readability
2. changed to lower level methods to increase testing speed
3.spotted unreachable exception handling case, added a TODO for future logic and AccountLogic exception issue for method joinCourseForStudent
4.added assertEuquals to check status message