search

cypress-io / cypress

Fast, easy and reliable testing for anything that runs in a browser.
https://cypress.io
MIT License
46.72k stars 3.16k forks source link

saved screenshots and videos names be saved in a terminal friendly way #23510

Open jpita opened 2 years ago

jpita commented 2 years ago

What would you like?

that the saved screenshots and videos names be saved in a terminal friendly way so we can cmd+click them to immediately open it

Why is this needed?

minor annoyance to have to open finder to open the screenshot

Other

this name is not terminal friendly since it has spaces and its broken with the pixels description.

it should have the name without spaces and the pixels description separate from it image

nagash77 commented 2 years ago

@jpita Thanks for opening this feature request ticket. Unfortunately, I do not think this issue will get worked on in the near future. I have marked this ticket as a good first issue and hopefully someone from the community is willing to make a PR to address this.

jdouitsis commented 1 year ago

Is anyone working on this issue or would I be able to take a look?

jpita commented 1 year ago

it is not assigned to anyone , go for it

jdouitsis commented 1 year ago

Current status I've been working on swapping out whitespaces with underscores (_) but when I do that and try and cmd+click in VSCode, it breaks at the parenthesis for a failed test (path/to/screenshot/filename(failed).png)

Upon further research, VSCode doesn't support parentheses being in a file name by default.

Possible solution Instead of putting the failed or attempt x in parenthesis, put it after three underscores:

Replace path/to/screenshot/filename_(failed).png with path/to/screenshot/filename___failed.png

Thoughts?

jpita commented 1 year ago

Current status I've been working on swapping out whitespaces with underscores (_) but when I do that and try and cmd+click in VSCode, it breaks at the parenthesis for a failed test (path/to/screenshot/filename(failed).png)

Upon further research, VSCode doesn't support parentheses being in a file name by default.

Possible solution Instead of putting the failed or attempt x in parenthesis, put it after three underscores:

Replace path/to/screenshot/filename_(failed).png with path/to/screenshot/filename___failed.png

Thoughts?

If it works , go

DevJSter commented 1 year ago

saving screenshots and videos with terminal-friendly names in a project. The idea is to make it easier to open these files directly from the terminal using commands like cmd+click. The proposed solution is to remove spaces from the names and separate the pixel description. This change aims to improve the user experience and streamline the process of accessing these files.

jdouitsis commented 1 year ago

I currently don't have time to work on this, so if anyone else is available, please feel free to jump in.