webdriverio-community / wdio-html-reporter

Fork of wdio-html-format-reporter
MIT License
18 stars 27 forks source link

Capability to add video in HTML reporter #77

Closed arun04arun95 closed 1 year ago

arun04arun95 commented 2 years ago

New Feature

wdio-video-reporter library will help to generate video for the test. Need capability to attach and play the video from the master HTML test report.

rpii commented 2 years ago

I will investigate this sometime this week.

Rich

On Tue, Dec 7, 2021 at 3:26 AM Arun kumar P @.***> wrote:

New Feature

wdio-video-reporter library will help to generate video for the test. Need capability to attach and play the video from the master HTML test report.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/rpii/wdio-html-reporter/issues/77, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEABIEMQPNQ76HD44JNW6U3UPXVMRANCNFSM5JRABLTQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

rpii commented 2 years ago

Ok this was easier than it looked. nice-html-reporter is updated, but it requires a patch to wdio-video-reporter. see version 7.9.1 of nice-html-reporter. You can just patch your local copy of wdio-video-reporter after installing until the pull request is accepted . see the repo in the changelog. let me know if it works for you.

nair-sumesh commented 2 years ago

Video is being recorded and saved to the location. However, the generated HTML does not link it correctly. It adds the path suite-0-1/0-0/ in the url. Config used exactly same as described here

Expected Link file:///C:/Users/xxxxxxxxxxxxxx/spec/reports/html-reports/bootstrap--login--CHROME--01-27-2022--12-16-27-980.mp4

Actual generated Link file:///C:/Users/xxxxxxxxxxxxxx//spec/reports/html-reports/suite-0-1/0-0/bootstrap--login--CHROME--01-27-2022--12-16-27-980.mp4

rpii commented 2 years ago

Are you using the aggregated report?

On Thu, Jan 27, 2022 at 9:43 AM nair-sumesh @.***> wrote:

Video is being recorded and saved to the location. However, the generated HTML does not link it correctly. It adds the path suite-0-1/0-0/ in the url. Config used exactly same as described here https://www.npmjs.com/package/wdio-video-reporter

Expected Link

file:///C:/Users/xxxxxxxxxxxxxx/spec/reports/html-reports/bootstrap--login--CHROME--01-27-2022--12-16-27-980.mp4

Actual generated Link file:///C:/Users/xxxxxxxxxxxxxx//spec/reports/html-reports/suite-0-1/0-0 /bootstrap--login--CHROME--01-27-2022--12-16-27-980.mp4

— Reply to this email directly, view it on GitHub https://github.com/rpii/wdio-html-reporter/issues/77#issuecomment-1023482318, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEABIEI3J2I2FEZ6IC6APMLUYF7ZFANCNFSM5JRABLTQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you commented.Message ID: @.***>

rpii commented 2 years ago

Per the readme you likely need:

        outputDir: 'reports/html-reports/screenshots',

in the video reporter configuration

On Thu, Jan 27, 2022 at 9:43 AM nair-sumesh @.***> wrote:

Video is being recorded and saved to the location. However, the generated HTML does not link it correctly. It adds the path suite-0-1/0-0/ in the url. Config used exactly same as described here https://www.npmjs.com/package/wdio-video-reporter

Expected Link

file:///C:/Users/xxxxxxxxxxxxxx/spec/reports/html-reports/bootstrap--login--CHROME--01-27-2022--12-16-27-980.mp4

Actual generated Link file:///C:/Users/xxxxxxxxxxxxxx//spec/reports/html-reports/suite-0-1/0-0 /bootstrap--login--CHROME--01-27-2022--12-16-27-980.mp4

— Reply to this email directly, view it on GitHub https://github.com/rpii/wdio-html-reporter/issues/77#issuecomment-1023482318, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEABIEI3J2I2FEZ6IC6APMLUYF7ZFANCNFSM5JRABLTQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you commented.Message ID: @.***>

gromanas commented 2 years ago

Hello! I had the same issue as @arun04arun95. After I followed your instruction @rpii and indeed with 7.9.1 version of html-nice-reporter, I am now able to see the attached video player. I don't know if I should comment this here, but now I am facing an issue with .mp4. Actually, I am just getting in the outputDir a folder rawSeleniumVideoGrabs that includes only the images that have been taken during the test execution and no video included, although I am getting in logs:

 "Video" Reporter:

Generated:[]

Video reporter Done!

Generated:[
  "/Users/user/projects/game/reports/html-reports/screenshots/Manually-add-numbers-and-win-Jackpot-Tier--CHROME--02-09-2022--19-56-19-718.mp4"
]

Video reporter Done!

Generated:[]

Video reporter Done!

Thank you in advance!

arun04arun95 commented 1 year ago

Hey @rpii,

This feature enhancement is tested and working fine with my project. thanks for picking. Closing the ticket

wdio-video-reporter library will help to generate video for the test. Need capability to attach and play the video from the master HTML test report.