Merge branch 'master' of https://github.com/TEAMMATES/repo into Issue1873
add test case to consider non-existent fs
Refactor code and adjust test cases
Format code
added test cases to check the condition when no questions are there for the user
Merge branch 'master' of https://github.com/TEAMMATES/repo into Issue1873
clean up InsFbResponseCommentDeleteAction
Issue1863
FeedbackSubmissionEditPageAction

+ Add test case closed session
+ Remove unnecessary conditional to check for null and throw exception since it will already be thrown by getDataBundle function.
+ Refactor some confusing code
Remove unused variable
Some refactoring of confusing code
clean up insFbResponseCommentEditAction
Clean up InstructorCourseInstructorEditSaveAction
Refactor and add missing test cases for updating instructor in InstructorsDb
added condition to avoid NullPointerException

course can become null and so NullPointerException will be thrown before
EntityDoesNotExistException is thrown
added test cases for better coverage
Refactor and modify test cases to be more detailed
Refactor code and remove excessive logging
Issue1896
InstructorFeedbackSubmitPageUiTest
+ Complete TODO: add grace period test case
Clean up InsFbResultsPageUiTest
Remove TODO in testModifyData - seems okay to be done as a UI test to check visibility of the response at html level.
Fix error in test cases, add test cases for confirm and authenticating join action into list of test cases