-
Notifications
You must be signed in to change notification settings - Fork 11
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
report different states to azure test plans #39
Conversation
@joerg1985 I appreciate you work @alex_neo/playwright-azure-reporter@1.6.0-beta.0 |
thank you, will test it in the next days and give feedback. |
All the enums are listed here, so we can map other values too:
|
@alexneo2003 could you please create a new beta with the latest fixes? |
@alex_neo/playwright-azure-reporter@1.6.0-beta.1 |
I had a look at the last runs of the pipeline and the results look good, this could be merged. The only thing to notice is that the 'Aborted' status does not update the state shown, but it will be listed in the details. PS: i think the 'colors' dependency could be removed or did i miss the usage in the code? |
This PR implements the change described in #38 without the option to configure the mapping.
I have no NodeJs IDE or the option to test this changes locally, so please review.