Fill in `pii_retirement: to_be_implemented`

[kdmccormick]

Problem

Since the annotations were added way back in 2019, three ProctoredExamSoftwareSecure* models in this package have been annotated with pii_retirement: to_be_implemented, with a comment saying "retirement to be implemented in https://openedx.atlassian.net/browse/EDUCATOR-4776". Links:

• https://github.com/openedx/edx-proctoring/blob/master/edx_proctoring/models.py#L691
• https://github.com/openedx/edx-proctoring/blob/master/edx_proctoring/models.py#L764
• https://github.com/openedx/edx-proctoring/blob/master/edx_proctoring/models.py#L810

to_be_implemented needs to be filled in with one of: ['retained', 'local_api', 'consumer_api', 'third_party']. Docs: https://docs.openedx.org/projects/openedx-proposals/en/latest/architectural-decisions/oep-0030-arch-pii-markup-and-auditing.html#docstring-annotations

Background

For a yet-to-be-determined reason, running pii_check on edx-platform master was not catching this. We are rewriting the quality checks to not use Paver and have noticed that when run locally, pii_check does indeed flag this issue. In order to get pii_check fully passing again, we'll need to resolve this violation.

code_annotations django_find_annotations --config_file .pii_annotations.yml --report_path /openedx/edx-platform/reports/pii --app_name lms --lint --coverage
Configured for report path: /openedx/edx-platform/reports/pii
Configured for source path: ./
Found safelist at .annotation_safe_list.yml. Reading.

Performing linting checks...

Search failed due to linting errors!
4 errors:
---------------------------------
celery_utils.FailedTask is annotated, but also in the safelist.
/openedx/venv/lib/python3.11/site-packages/edx_proctoring/models.py::edx_proctoring.ProctoredExamSoftwareSecureComment: "to_be_implemented" is not a valid choice for ".. pii_retirement:". Expected one of ['retained', 'local_api', 'consumer_api', 'third_party'].
/openedx/venv/lib/python3.11/site-packages/edx_proctoring/models.py::edx_proctoring.ProctoredExamSoftwareSecureReview: "to_be_implemented" is not a valid choice for ".. pii_retirement:". Expected one of ['retained', 'local_api', 'consumer_api', 'third_party'].
/openedx/venv/lib/python3.11/site-packages/edx_proctoring/models.py::edx_proctoring.ProctoredExamSoftwareSecureReviewHistory: "to_be_implemented" is not a valid choice for ".. pii_retirement:". Expected one of ['retained', 'local_api', 'consumer_api', 'third_party'].