Address PR comments

lms/djangoapps/grades/signals/signals.py

@@ -83,9 +83,9 @@ SUBSECTION_SCORE_CHANGED = Signal(
 )
 
 # Signal that indicates that a user's score for a subsection has been overridden.
-# This signal is generated when an admin changes a user's exam attempt state to
-# rejected or to verified from rejected. This signal may also be generated by any
-# other client using the GradesService to override subsections in the future.
+# This signal is generated when a user's exam attempt state is set to rejected or
+# to verified from rejected. This signal may also be sent by any other client
+# using the GradesService to override subsections in the future.
 SUBSECTION_OVERRIDE_CHANGED = Signal(
     providing_args=[
         'user_id',  # Integer User ID

lms/djangoapps/grades/tests/test_services.py

@@ -59,6 +59,7 @@ class GradesServiceTests(ModuleStoreTestCase):
         }
 
     def tearDown(self):
+        PersistentSubsectionGradeOverride.objects.all().delete()  # clear out all previous overrides
         self.signal_patcher.stop()
         self.id_patcher.stop()
         self.type_patcher.stop()
@@ -157,8 +158,6 @@ class GradesServiceTests(ModuleStoreTestCase):
     )
     @ddt.unpack
     def test_override_subsection_grade(self, override, expected):
-        PersistentSubsectionGradeOverride.objects.all().delete()  # clear out all previous overrides
-
         self.service.override_subsection_grade(
             user_id=self.user.id,
             course_key_or_id=self.course.id,