Populate completed_on when rejecting a ReviewAssignment. Migrate past rejected assignments accordingly. Fixes ietf-tools#2782. Commit ready for merge.

 - Legacy-Id: 16770

Author: rjsparks

ietf/doc/tests_review.py

@@ -473,6 +473,7 @@ def test_reject_reviewer_assignment(self):
 
         assignment = reload_db_objects(assignment)
         self.assertEqual(assignment.state_id, "rejected")
+        self.assertNotEqual(assignment.completed_on,None)
         e = doc.latest_event()
         self.assertEqual(e.type, "closed_review_assignment")
         self.assertTrue("rejected" in e.desc)

ietf/doc/views_review.py

@@ -335,6 +335,7 @@ def reject_reviewer_assignment(request, name, assignment_id):
         if form.is_valid():
             # reject the assignment
             review_assignment.state = ReviewAssignmentStateName.objects.get(slug="rejected")
+            review_assignment.completed_on = datetime.datetime.now()
             review_assignment.save()
 
             ReviewAssignmentDocEvent.objects.create(