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

Feature request: Add videos to Cypress' cucumber reports #1139

Open
3 tasks done
jessica-jorgenson opened this issue Dec 15, 2023 · 2 comments
Open
3 tasks done

Feature request: Add videos to Cypress' cucumber reports #1139

jessica-jorgenson opened this issue Dec 15, 2023 · 2 comments

Comments

@jessica-jorgenson
Copy link

Current behavior

When generating an HTML report, there is an attached image for screenshots for failing tests.

Desired behavior

When the Cypress has the config video: true, the HTML report should include the video attachment for passing or failing tests. (or just failing if this is enabled).
Similar to this issue here.

Versions

  • Cypress version: ^13.3.0
  • Preprocessor version: ^18.0.6
  • Node version: 18.18.2

Checklist

  • I've read the FAQ.
  • I've read instructions for logging issues.
  • I'm not using cypress-cucumber-preprocessor@4.3.1 (package name has changed and it is no longer the most recent version, see #689).
@badeball
Copy link
Owner

Cucumber messages, which the json and html report are products of, is a well-defined (somewhat at least) protocol. Its notion of attachments is fairly limited and attachments can only really be tied to a step. In Cypress however, a video is logically tied to a file.

So, in order to add a video, one would have to decide which step to attach it to. And this isn't as straight forward as one might think. There can be multiple failed tests and steps in a file, which of them deserves the video attachment? Bear in mind, attachments can't refer to each other, so adding it to multiple steps would blow of the report size.

@badeball
Copy link
Owner

Attachments not tied to a specific step, but a test run has been discussed in the past, ref: cucumber/messages#102 and cucumber/cucumber-js#1394. I suspect this is the most (perhaps only) viable solution. However, I don't know if / when this will get merged. It would also take some work on https://github.com/cucumber/react-components to display test-run attachments.

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

2 participants