search

webdriverio-community / wdio-cucumberjs-json-reporter

A WDIO reporter that creates CucumberJS JSON files for WebdriverIO
https://webdriver.io/docs/wdio-cucumberjs-json-reporter
MIT License
23 stars 32 forks source link

Add videos from the wdio-video-reporter to the report output #162

Open alexwalker1993 opened 1 year ago

alexwalker1993 commented 1 year ago

Is your feature request related to a problem? Please describe. I want to add the videos from the wdio-video-reporter to the report but it doesn't seem to be possible

Describe the solution you'd like The report to have the videos attached to the relevant tests

christian-bromann commented 1 year ago

Thanks for reporting!

Any contributions that implements this feature are highly appreciated. Don't expect this to be picked up by active contributors as they have their own priorities. If you depend on this bug to be fixed, your contribution is required. Please take a look into our contribution guidelines or join our Discord development server and let us know if you have any questions. Cheers!

mikezaschka commented 1 year ago

I do have the same requirement and will potentially look into this in the next couple of weeks.

christian-bromann commented 1 year ago

@mikezaschka any updates? Happy to help if anything is unclear.

isopropilick commented 10 months ago

I think it's a single weekend issue maybe? Can I take it?

christian-bromann commented 10 months ago

@isopropilick that would be awesome, I think changes need to be made in the video reporter. I just rewrote the reporter and am about to release a new major version soon.

isopropilick commented 10 months ago

@christian-bromann No problem! if your dev branch is public I would love to take a look and then return to this issue when the realase is made in order to use a tagged version of the video reporter to tackle this one.

isopropilick commented 10 months ago

Maybe a little JSON-based magic for metadata --- blob / arraybuffer for the transfer and/or store of the video --- and add the default base64 thinggy for images? IDK seems fun

christian-bromann commented 10 months ago

It is, go for it!

christian-bromann commented 10 months ago

No problem! if your dev branch is public I would love to take a look and then return to this issue when the realase is made

Actually I plan to do the release now.

isopropilick commented 10 months ago

@christian-bromann sweet I will look in to this this week!

AlfredoMoretta commented 7 months ago

hi @isopropilick any news about this topic? thank you in advance for the answer

christian-bromann commented 7 months ago

@AlfredoMoretta I think this bug is still up for grabs, any contributions would be much appreciated.