Issue1896

Issue1896
InstructorFeedbackSubmitPageUiTest
+ Complete TODO: add grace period test case
+ Remove TODO in testModifyData - seems okay to be done as a UI test to check visibility of the response at html level.