Public

Project privacy set to public. By default, its content is available to everyone (authenticated or not). Please note that more restrictive permissions might exist on some items.

stable

Clone or download

Read-only

feat(ttm): Remove attachment files from an execution

Download Browse

This patch introduces an improvement of the REST route PUT testmanagement_executions/id and insert a new optional key `deleted_file_ids` to remove files from an execution. For the moment, when the user gives a bad file id to delete, no exception is thrown. That can be done in a future patch if necessary. How to test: - You need to have an execution with attachment - In TTM, click on 'Edit comment' - You can click on the trash to mark the attachment as deleted - Click on "Save the comment" Expected result: - There should be not the file attachment This is part of story #22576 have multiple files uploaded with comments Change-Id: I793a9119a0b92c2fd29ebae48fdb17c95acd1f42

+325 −21
Benjamin Bouillot (ben.bouillot) 2021-09-28 12:19
Benjamin Bouillot (ben.bouillot) 2021-09-30 10:36
28f376c9b21e0f9e87a3595b52fb112e79a6c9f3
b3c5318373dfc4436a0fe85ed82f01c24b5563e3
git #tuleap/stable/28f376c9b21e0f9e87a3595b52fb112e79a6c9f3

Modified Files

Side by side diff
Inline diff
Name
M plugins/testmanagement/include/TestManagement/REST/ExecutionChangesExtractor.php +3 −2 Go to diff View file
M plugins/testmanagement/include/TestManagement/REST/FormattedChangesetValueForFileFieldRetriever.php +3 −2 Go to diff View file
M plugins/testmanagement/include/TestManagement/REST/v1/ExecutionFromAutomatedTestsUpdater.php +1 −0 Go to diff View file
M plugins/testmanagement/include/TestManagement/REST/v1/ExecutionsResource.class.php +4 −3 Go to diff View file
M plugins/testmanagement/scripts/testmanagement/po/fr.po +6 −0 Go to diff View file
M plugins/testmanagement/scripts/testmanagement/src/execution-collection/execution-rest-service.js +8 −1 Go to diff View file
M plugins/testmanagement/scripts/testmanagement/src/execution-collection/execution-rest-service.test.js +2 −1 Go to diff View file
M plugins/testmanagement/scripts/testmanagement/src/execution-collection/execution-service.js +26 −0 Go to diff View file
M plugins/testmanagement/scripts/testmanagement/src/execution-collection/execution-service.test.js +49 −0 Go to diff View file
M plugins/testmanagement/scripts/testmanagement/src/execution/execution-attachments/execution-attachments-component.js +19 −0 Go to diff View file
M plugins/testmanagement/scripts/testmanagement/src/execution/execution-attachments/execution-attachments-component.test.js +52 −0 Go to diff View file
M plugins/testmanagement/scripts/testmanagement/src/execution/execution-attachments/execution-attachments.tpl.html +41 −1 Go to diff View file
M plugins/testmanagement/scripts/testmanagement/src/execution/execution-detail-controller.js +15 −1 Go to diff View file
M plugins/testmanagement/scripts/testmanagement/src/execution/execution-detail-controller.test.js +19 −2 Go to diff View file
M plugins/testmanagement/tests/e2e/cypress/cypress/integration/campaign.spec.ts +9 −2 Go to diff View file
M plugins/testmanagement/tests/rest/TestManagement/ExecutionsTest.php +26 −0 Go to diff View file
M plugins/testmanagement/tests/unit/TestManagement/REST/ExecutionChangesExtractorTest.php +2 −0 Go to diff View file
M plugins/testmanagement/tests/unit/TestManagement/REST/ExecutionFromAutomatedTestsUpdaterTest.php +3 −0 Go to diff View file
M plugins/testmanagement/tests/unit/TestManagement/REST/FormattedChangesetValueForFieldsRetrieverTest.php +8 −6 Go to diff View file
M plugins/testmanagement/themes/BurningParrot/css/includes/_current-execution.scss +29 −0 Go to diff View file