-
Notifications
You must be signed in to change notification settings - Fork 2.7k
MM-T346 Public link related to a deleted post should no longer open the file #6331
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @M-ZubairAhmed , I'm getting a failure when the case is run with other cases in the test file. It runs successfully by itself though. Please check.
@josephbaylon Hi, thanks for bringing this up, unfortunately my computer doesnt handle running more than one test simulataneously. Also i see that the problem seems to be in beforeEach. If you can let me know i will gladly be able to fix |
@M-ZubairAhmed I tried locally and just needs |
@josephbaylon build is failing due to some node error. Can you please check, all other are working fine, is there something i should do from here? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. All tests passed. Thanks @M-ZubairAhmed !!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @M-ZubairAhmed. LGTM, except for one request.
e2e/cypress/integration/files_and_attachments/upload_files_spec.js
Outdated
Show resolved
Hide resolved
…c.js Co-authored-by: Saturnino Abril <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @M-ZubairAhmed, LGTM!
…he file (#6331) * MM-T346 Public link related to a deleted post should no longer open the file * login as admin * typo * changes * changes * Update e2e/cypress/integration/files_and_attachments/upload_files_spec.js Co-authored-by: Saturnino Abril <[email protected]> * changes 1 Co-authored-by: Saturnino Abril <[email protected]>
Summary
This PR includes an end to end test for verifying the deleted post attachment are indeed deleted when we try to visit their public links
Ticket Link
MM-T346
Related Pull Requests
none
Screenshots
none