Working implementation for issue-1216
issue 1216: added functional test file
Revert "issue 1216: added functional test file"

This reverts commit 6b69db132c6b50aa3885c7556044694668eb75de.
isue-1216: added functional tests
Revert "isue-1216: added functional tests"

This reverts commit ac1f4420d05eb2927117bb9332a4c29da34d6201.
issue-1216: added functional tests
issue-1216: improved code quality
Fix for issue 1418: a newline apparently made the median wrong.
Fixed issue 1347 (seemingly infinite loading)
Fix issue #1295: Added a check to see if every group being collected had only one TA assigned to them.
Added @ealona's test cases.
Fix for issue 1295: DIsabled button if more than one TA is marking the same set of submissions.
Added check to second branch to make sure it fails for any user except for a TA.
implement eager loading for index action on main
do some formatting
Fix updating of annotation overall comment

This fixes #1431. The instance variable `@result` is always the new
result of the remark, not the original old result. The caller of the
template passes the old result correctly as a local variable `result`,
but the template is not using it.

Tested:
- local server
- rake test:unit
- rake test:functional
Add functional test to cover comment edit forms
fix errant parens
Fix for issue-1414: Grade doesn't get updated when grading under TA
implement su's suggestions