Closed exalate-issue-sync[bot] closed 3 days ago
Elaine Krauss commented: New command {{python manage.py delete_committee_reports}} added. An example of it running is included below, along with examples of the help text and various error messages.
!image-20240821-160836.png|width=1255,height=1122,alt="image-20240821-160836.png"! See image in Jira
From CR, [~accountid:712020:eaccd25d-427c-4e4d-a650-909ec0b31071] check out the DEV notes in the ticket and my comment in the PR. There is existing code for e2e tests that I think can be leveraged with a little refactoring.
Sending ticket back to In Progress
Matt Travers commented: For QA, here is a screenshot of the new command showing reports and transactions being deleted for test organization C99999999
!image-20240827-192853.png|width=1159,height=355,alt="image-20240827-192853.png"! See image in Jira
Passes CR, sending to QA.
Shelly Wise commented: QA review visual inspection from DEV showing deleted for test Committee ID C99999999
!image-20240827-223711.png|width=916,height=298,alt="image-20240827-223711.png"! See image in Jira
QA Review Completed. Moved to Stage Ready.
Automation for Jira commented: Sprint accepted by Paul Clark at Sprint Review on comment date.
From time-to-time, it is necessary to clear out all the reports of a test user. The code for doing this via the URL is set up for the C9999999 e2e committee account but cannot be expanded due to security reasons.
This can be done securely if implemented with a Django admin command.
QA Notes
A dev can clear out a committee account on DEV or STAGE to show that the new functionality is working.
DEV Notes
Be sure to reuse and leverage the existing clear reports code used by the e2e tests:
[https://github.com/fecgov/fecfile-web-api/blob/release/sprint-46/django-backend/fecfiler/reports/views.py#L144-L173|https://github.com/fecgov/fecfile-web-api/blob/release/sprint-46/django-backend/fecfiler/reports/views.py#L144-L173|smart-link]
Design
null
See full ticket and images here: FECFILE-1541