Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trashing assets results in infinitely nested trash directories #2742

Closed
lilaconlee opened this issue Nov 8, 2018 · 6 comments
Closed

Trashing assets results in infinitely nested trash directories #2742

lilaconlee opened this issue Nov 8, 2018 · 6 comments
Assignees
Milestone

Comments

@lilaconlee
Copy link
Contributor

Current behavior:

Reported from Gitter, trashing assets and trying to download artifacts results in a file too long error:

screenshots/.Trash-0/files/27869827-fa54-46b7-83a5-a75b568c4ba9/.Trash-0/files/27869827-fa54-46b7-83a5-a75b568c4ba9/.Trash-0/files/27869827-fa54-46b7-83a5-a75b568c4ba9/.Trash-0/files/27869827-fa54-46b7-83a5-a75b568c4ba9/.Trash-0/files/27869827-fa54-46b7-83a5-a75b568c4ba9/.Trash-0/files/27869827-fa54-46b7-83a5-a75b568c4ba9/.Trash-0/files/27869827-fa54-46b7-83a5-a75b568c4ba9/.Trash-0/files/27869827-fa54-46b7-83a5-a75b568c4ba9/.Trash-0/files/27869827-fa54-46b7-83a5-a75b568c4ba9/.Trash-0/files/27869827-fa54-46b7-83a5-a75b568c4ba9/.Trash-0/files/27869827-fa54-46b7-83a5-a75b568c4ba9/.Trash-0/files/27869827-fa54-......file name too long

Desired behavior:

Should be able to trash files without recursively creating Trash directories. Alternative could be deleting files when running on CI.

Steps to reproduce:

Haven't been able to reproduce, but haven't successfully tried on the version of Ubuntu reported. Was unable to reproduce on 18.10.

Versions

Cypress 3.1.1
Ubuntu 16.04

@lilaconlee lilaconlee self-assigned this Nov 8, 2018
@lilaconlee lilaconlee added this to the Sprint 10 milestone Nov 8, 2018
@jennifer-shehane
Copy link
Member

Maybe related comment involving recursive Trash dir problem: #2392 (comment)

@jennifer-shehane
Copy link
Member

Also a related issue with permissions accessing Trash dir, not sure you want to tackle this with it though: #1281

@lilaconlee
Copy link
Contributor Author

Opted to delete files on Linux instead of trashing them. Seems like the lowest maintenance solution as trashing in Linux seems inconsistent, but lemme know if that doesn't seem like a reasonable trade-off.

@jennifer-shehane
Copy link
Member

The code for this is done, but this has yet to be released. We'll update this issue and reference the changelog when it's released.

@brian-mann
Copy link
Member

brian-mann commented Nov 19, 2018

Released in 3.1.2.

@electrofLy
Copy link

Why is the option to fully delete contents is not available on Windows as well? Our CI server (running on Windows) disk space is getting full as all videos of all projects are sent to the Recycle Bin, which results in ~50GB of videos for 1 week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants