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
[#11473] - Refactor wdio-allure-reporter package for better hooks handling #12852
Conversation
Code has been restructured across the wdio-allure-reporter package for enhanced handling of hooks in different situations. Additionally, constant names and util function names have been updated for increased clarity. The handling of hooks particularly for mocha tests has been revised, improving the accuracy of test reports. Signed-off-by: Lara, Manuel <lararojas.mr@gmail.com>
@christian-bromann is there an issue with pipelines, because i'm not getting those failures in my machine? I'm using a mac. Could you check that please? I'm getting errors in Bidi test cases |
@lararojasmr yes, I am currently working on fixing the |
I hope I did it right @christian-bromann if you have time, please confirm. |
v8: #12855 |
- Before Hook Failed (Duplicated) - Inner Step with fail status not propagate the fails details to test. - Other Validations
- Fixes test cases to correct validation after fixing report.ts
Fixing: Error: packages/wdio-allure-reporter/src/reporter.ts(396,23): error TS2451: Cannot redeclare block-scoped variable 'currentTest'. Error: packages/wdio-allure-reporter/src/reporter.ts(397,23): error TS2451: Cannot redeclare block-scoped variable 'currentTest'. Error: packages/wdio-allure-reporter/src/reporter.ts(401,17): error TS2304: Cannot find name 'setHistoryId'.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Hey lararojasmr 👋 Thank you for your contribution to WebdriverIO! Your pull request has been marked as an "Expensable" contribution. We've sent you an email with further instructions on how to claim your expenses from our development fund. Please make sure to check your spam folder as well. If you have any questions, feel free to reach out to us at expense@webdriver.io or in the contributing channel on Discord. We are looking forward to more contributions from you in the future 🙌 Have a nice day, |
Proposed changes
Code has been restructured across the wdio-allure-reporter package for enhanced handling of hooks in different situations. Additionally, constant names and util function names have been updated for increased clarity. The handling of hooks particularly for mocha tests has been revised, improving the accuracy of test reports.
Types of changes
Checklist
Backport Request
//: # (The current
main
branch is the development branch for WebdriverIO v9. If your change should be released to the current major version of WebdriverIO (v8), please raise another PR with the same changes against thev8
branch.)v9
and doesn't need to be back-portedReviewers: @webdriverio/project-committers