Skip to content
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.

Sign up for GitHub

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

Jump to bottom

add support not executed tests for TRX results #1578

Merged
merged 2 commits into from
Feb 10, 2022
Merged

add support not executed tests for TRX results #1578

merged 2 commits into from
Feb 10, 2022

Conversation

agentran
Copy link
Contributor

@agentran agentran commented Feb 8, 2022 •
edited by baev
Loading

Context

Checklist

@agentran agentran closed this Feb 8, 2022
@agentran agentran reopened this Feb 8, 2022
@baev baev added the type:improvement Change that improves some user experience but can't be considered as a new feature label Feb 8, 2022
@baev
Copy link
Member

baev commented Feb 8, 2022

@agentran could you please add a test for the change? Just for future regression testing

@agentran
Copy link
Contributor Author

agentran commented Feb 8, 2022

@baev Added to the base test shouldParseResults.

@omer-za
Copy link

omer-za commented Feb 9, 2022

@baev when it will be merged and when there's a plan to have a new release?

@baev baev merged commit 319d89e into allure-framework:master Feb 10, 2022
@baev baev changed the title parseStatus of TrxPlugin to support not executed tests by the IgnoreAttribute add support not executed tests for TRX results Feb 10, 2022
@baev
Copy link
Member

baev commented Feb 10, 2022 •
edited
Loading

@agentran @omer-za thanks!

when there's a plan to have a new release?

I'd say within a week

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@baev baev baev approved these changes

Assignees
No one assigned
Labels
theme:plugins type:improvement Change that improves some user experience but can't be considered as a new feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@agentran @baev @omer-za