webdriverio-community / wdio-html-reporter

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

html-nice reporter not showing any scenario steps status in the master report in cucumber framework #65

Closed vdrulerz closed 3 years ago

vdrulerz commented 3 years ago

Everything seemed to be working well with @rpii/wdio-html-reporter but ever since I have started using html-nice reporter it is not showing scenario level status in the populated report

If you see below screenshot, there is nothing showing below the spec file and it's all blank

image

It should have ideally showed scenario level status like it used to show before.

Here is my wdio.conf.ts configuration for this report

image

image

image

rpii commented 3 years ago

I have looked at a cucumber report, and there appears to be no test steps in the json file. so they changed something. thats why metrics are coming up empty. show me one of your json files if it has any test steps in it.

vdrulerz commented 3 years ago

@rpii Hi I looked in to the the json files and it is not empty in my case it has steps data too. But still I am not able to see the scenario steps status in the html report it just shows spec name

image

vdrulerz commented 3 years ago

This seems to be an issue, before version 7 of WDIO this report was working fine looks like after refracturing this report in TS....cucumber framework is not supported at all

Can you investigate it once and see what exactly is the issue ??

rpii commented 3 years ago

So from what I can see the format has changed and the nesting of the suites and tests is different. I will look into this and get this fixed

vdrulerz commented 3 years ago

Thanks much @rpii.....will look forward to further updates from your side

rpii commented 3 years ago

Well, to fix this recursion needs to be used in the handlebars template to handle the nested suites structure. I am looking into this and doing a lot of debugging but it will likely not be a quick tix.

vdrulerz commented 3 years ago

@rpii Thanks for the heads up, as you pointed out that it's not a quick fix so will wait for further updates from your end

rpii commented 3 years ago

Please try this version out. It has been reworked and passes a basic cucumber feature test.

andreeamorariu commented 3 years ago

@rpii Does the fix work also for jasmine framework?

rpii commented 3 years ago

I dont use the Jasmine framework or have a test case for it. I have no reason to believe it wont. You can try it. if it fails send me the master-report.json file

On Fri, Aug 6, 2021 at 4:34 AM andreeamorariu @.***> wrote:

@rpii https://github.com/rpii Does the fix work also for jasmine framework?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/rpii/wdio-html-reporter/issues/65#issuecomment-894198671, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEABIEK7LYC5PZPC7LOWJUDT3PCBXANCNFSM5AUBFT5Q . 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&utm_campaign=notification-email .

andreeamorariu commented 3 years ago

I dont use the Jasmine framework or have a test case for it. I have no reason to believe it wont. You can try it. if it fails send me the master-report.json file On Fri, Aug 6, 2021 at 4:34 AM andreeamorariu @.***> wrote: @rpii https://github.com/rpii Does the fix work also for jasmine framework? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#65 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEABIEK7LYC5PZPC7LOWJUDT3PCBXANCNFSM5AUBFT5Q . 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&utm_campaign=notification-email .

Thank you. I used the new version and it's working! :)

rpii commented 3 years ago

If you have a small test case you can donate I will add it to the testing.

On Mon, Aug 9, 2021 at 3:16 AM andreeamorariu @.***> wrote:

I dont use the Jasmine framework or have a test case for it. I have no reason to believe it wont. You can try it. if it fails send me the master-report.json file … <#m-4559054302853301949> On Fri, Aug 6, 2021 at 4:34 AM andreeamorariu @.***> wrote: @rpii https://github.com/rpii https://github.com/rpii Does the fix work also for jasmine framework? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#65 (comment) https://github.com/rpii/wdio-html-reporter/issues/65#issuecomment-894198671>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEABIEK7LYC5PZPC7LOWJUDT3PCBXANCNFSM5AUBFT5Q . 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&utm_campaign=notification-email .

Thank you. I used the new version and it's working! :)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/rpii/wdio-html-reporter/issues/65#issuecomment-895106979, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEABIEMHAXWUFFCPD3HRMRLT36TFRANCNFSM5AUBFT5Q . 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&utm_campaign=notification-email .